Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 06:24
    anacrolix commented #657
  • Sep 18 22:19
    bsergean commented #657
  • Sep 18 18:06
    bsergean commented #657
  • Sep 18 11:01

    anacrolix on master

    goimports doing its thing Export request_strategy.GetRequ… Add choking/allowed fast checks… (compare)

  • Sep 18 06:35

    anacrolix on master

    Add some tests verifying reques… (compare)

  • Sep 18 05:16
    YenForYang synchronize #664
  • Sep 18 05:15
    YenForYang edited #664
  • Sep 18 05:10
    YenForYang edited #664
  • Sep 18 05:10
    YenForYang opened #664
  • Sep 18 04:37
    YenForYang opened #663
  • Sep 18 04:35
    YenForYang synchronize #662
  • Sep 18 04:26
    YenForYang opened #662
  • Sep 18 03:23

    anacrolix on master

    Update anacrolix/args for bool … (compare)

  • Sep 18 02:53

    anacrolix on master

    request_strategy.Peer.canReques… requesting: Remove some obsolet… (compare)

  • Sep 18 02:46
    anacrolix commented #657
  • Sep 18 02:44

    anacrolix on master

    bencode: get type of `big.Int` … (compare)

  • Sep 18 02:44
    anacrolix closed #651
  • Sep 18 02:43

    anacrolix on master

    bencode: simplify getting `mars… (compare)

  • Sep 18 02:43
    anacrolix closed #652
  • Sep 18 02:42

    anacrolix on master

    bencode: Simplify `(*Decoder).p… (compare)

Matt Joiner
@anacrolix
@lkslts64 no, there's no limit to the number of peers uploaded to simultaneously
Loukas Litsos
@lkslts64
ok thanks. Anyways, besides this, I recently developed my own bittorrent client in Go, just for learning/fun, so I ended up looking a lot your project to clarify some things. I was amazed by the quality of the code! I am relatively new in programming so this was the first Go project I digged into but again I don't think that others provide such code quality by leveraging all Go's features and idioms. So thanks for this experience!
Matt Joiner
@anacrolix
@lkslts64 I wish I could say it was all good, but a lot of it is inconsistent, and I'd do a lot differently in hindsight.
ctwyw
@ctwyw
Hi there, Is there any example to building dht server or tracker server ?
Matt Joiner
@anacrolix
@ctwyw you're welcome to peruse anacrolix/dht and the tracker-related parts of anacrolix/torrent.
Jian Zhu
@zhujian0805
i intermittently get those errors, not sure why
https://paste.ubuntu.com/p/bHxGzMWczg/
Jian Zhu
@zhujian0805
sometime it can download successfully, sometime not :(
ctwyw
@ctwyw
@anacrolix I`ve success to build server and client. It was success when use the same torrent file . but if my client use the magnet of that torrent , it was failure. how can I download by using magnet ?
I use [ torrent-magnet < 2.torrent ] to create the magnet , is this right ?
Matt Joiner
@anacrolix
@zhujian0805 sorry for the delay. what errors do you refer to? there are none in that log file
@ctwyw would your network have UDP issues? it's likely your DHT is failing to find peers, or not working at all. are you behind a NAT router? does the magnet link that is created contain any trackers? do you want to PM the link or share it here?
Yaroslav Kolomiiets
@yarikk
@anacrolix krpc.CompactIPv4NodeAddrs (or NodeAddr) does not have the essential To4() piece in its marshaller functions, unlike pp.CompactIp which does. Hence the need to do the dance at the use sites – in particular, the PEX implementation.
Yaroslav Kolomiiets
@yarikk
Yaroslav Kolomiiets
@yarikk
@anacrolix why do we need tt.DownloadAll()? In essence, it calls updatePiecePriority() for every piece. We could do it from onSetInfo() along with updatePieceCompletion() – what do you think?
Loukas Litsos
@lkslts64
@anacrolix Hi again, I can see that in the torrent package you don't make heavy usage of channels. Instead you prefer the sync package. That choice was made for performance reasons or you think that channels don't fit well in this particular application? What made you choose the sync package as the primary synchronization mechanism? Moreover, what would you do differently if you had to make this package again? Thanks in advance and sorry for this type of questions but I would like to hear your thoughts.
Matt Joiner
@anacrolix
@yarikk It's a convenience method from back when the client was first written. anacrolix/torrent does not download on start-up, that's intentional. You must raise the piece or file priorities yourself, or use a torrent.Reader to prompt the client to start downloading.
@lkslts64 Like a lot of people I was sucked in to Go partly due to channels. Channels only work in some simplified situations, they don't handle prioritization of events, they don't combine well with other forms of synchronization, and they don't perform as well (such as requiring huge numbers of goroutines in some situations). I don't think I would switch to channels if I were to do it again, in fact I'd probably use another language like Rust instead. FWIW, teh project was in Python in 2012, and quickly moved to Go.
Loukas Litsos
@lkslts64
@anacrolix Thats right I agree with you. I asked you because in my own BT client, I started up using channels and then realized that the complexity of the code was huge. I was so frustrated by the annoying deadlocks that were happening merely because of the need of bidirectional communication between the channels (N connection goroutines communicating bidirectionally with 1 torrent goroutine). Moreover, channels don't fit well when a connection wants to decide which pieces to download so I ended up using the sync package too and it was much more simplified that way. On the other hand, my implementation avoids having a global lock which improves scalability and also channels cooperate well with the timers and tickers in the time package. But, in conclusion if I were to do it again, I woudn't choose them for sure, the sync package is way more flexible. Another man that fell into the channel overuse trap...
Akshay Ashok
@Akshay090
How can we use a web UI to interact with this library, i saw confluence ( interact with Http) so was wondering if there is some website ui to interact with this ?
Matt Joiner
@anacrolix
@Akshay090 there are, they're separate and/or private projects, see https://github.com/anacrolix/torrent#downstream-projects
ctwyw
@ctwyw
@anacrolix Hi again, I can already download the magnet. these is another question. I build serval server on the PublicIP4 and then build two client behind the NAT router . but could not see them in the DHT node. UNPN is start already. the clients can not share with each other.but the server work well yet. What can I do in this case ?
Listening on 0.0.0.0:63333
Nodes in table: 5 good, 5 total
Ongoing transactions: 0
Server node ID: e0cb86882625a25b8a83002f21cd1f5c46*****

b# node id                                  addr                 anntok last query last response cf
0  2572c85bdb9e52254e629285f7eb329075e***** *.*.*.*:40001  20     never      46s ago       0
0  1a6e0b237b73a79091efd6ca8572c06c9fa***** *.*.*.*:40002  20     never      45s ago       0
1  ae614ab3fc2288a8607fb5264e1cef6d7b4***** *.*.*.*:40002   20     never      45s ago       0
1  9d415d2479966acd83c3b3446cbb97a209f***** *.*.*.*:40002 20     never      45s ago       0
1  9730ca99ccae7eb8d243846063fe951326f***** *.*.*.*:40002    20     never      45s ago       0
ctwyw
@ctwyw
I saw this in the torrent status
error announcing: reading from socket: read udp4 192.168.199.229:64718->*:40002: i/o timeout
Yaroslav Kolomiiets
@yarikk
@anacrolix , I just noticed anacrolix/torrent@c1d189e, how much of a performance gain did the change allows to get? I'm also exploring ways to speed it up
Matt Joiner
@anacrolix
replied in github
imadr
@imadr
Hello, is there a way to specify what directory each torrent is downloaded to, instead of putting them all in ClientConfig.DataDir?
Matt Joiner
@anacrolix
Yes, provide a custom storage at the Client level, or to each Torrent when you add it.
Yaroslav Kolomiiets
@yarikk

@imadr, indeed, like this:

    spec := torrent.TorrentSpecFromMetaInfo(meta)
    compl := storage.NewMapPieceCompletion()
    spec.Storage = storage.NewMMapWithCompletion(dstDir, compl)
    tt, _, err := client.AddTorrentSpec(spec)

(there are options of storage and completion backends to choose from)

imadr
@imadr
thanks!
imadr
@imadr
I'm not able to delete the torrent files even after it finishes, the error from os.RemoveAll is "Access is denied."
Here is an example https://pastebin.com/raw/gKLkq8bk
Also this only happens on windows
Matt Joiner
@anacrolix
@imadr you are not closing the mmap storage before trying to remove the files. On Windows you cannot remove files that are being held open.
imadr
@imadr
It's working, thx
imadr
@imadr
Oops nevermind I talked too soon, it's still saying access denied. I also closed the PieceCompletion for good measure but same, am I doing it wrong?
c := torrentStorage.NewMapPieceCompletion()
mmap := torrentStorage.NewMMapWithCompletion(torrentPath, c)
s.Storage = mmap
...
c.Close()
mmap.Close()
t.Drop()
Matt Joiner
@anacrolix
You should release resources in the reverse order that you allocate them. I'll need more info to help further, can you check for errors returned from those clean-up functions? Could you add a time.Sleep after the clean-up before trying to remove the files in case there's a timing bug? Also run with the race detector enabled.
imadr
@imadr
Here is what I tried https://pastebin.com/raw/EYebt4vr same access denied error and no errors when closing the resources
I also tried with go run -race main.go but nothing changed
Matt Joiner
@anacrolix
@imadr I did some thorough static analysis on MacOS. No files remained memory mapped in the directory you're deleting. I was using github.com/anacrolix/torrent v1.19.2 with your code.
gitpubber
@gitpubber
Matt joiner , I have client with DataDir set to a path
But,i have this case where i need to download torrent to different path
how can i do?
I know of storage.ClientImpl but i am not proficient to do it
gitpubber
@gitpubber
hey,thank you very much
gitpubber
@gitpubber
@anacrolix If i enable webseed in config,is it enough?
Or should I do anything extra to enable webseeding?
@anacrolix I must get pieces completed and total no of pieces .Is there any function for it?
Ok,I got about pieces. no need to answer that
Matt Joiner
@anacrolix
@gitpubber nothing extra should be required for webseeding
AlexSpain
@AlexSpain
hi all
how I can disable ipv6