Mirror/bittorrent-tracker
1
0
Fork 0
mirror of https://github.com/webtorrent/bittorrent-tracker.git synced 2024-12-14 03:16:24 +00:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Fixes for PR #39

Browse Source 
- emit 'warning' instead of 'error' because it's not a fatal error,
just a bad tracker response.
- made the try block more concise so it only catches the errors from
`string2compact`
This commit is contained in:
Feross Aboukhadijeh 2014-10-06 23:14:09 -07:00
parent 7e081f88d1
commit 07850cdc13
1 changed files with 12 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
client.js
View File

@ -352,13 +352,15 @@ Tracker.prototype._requestUdp = function (requestUrl, opts) {
incomplete: msg.readUInt32BE(12)
})
var addrs
try {
compact2string.multi(msg.slice(20)).forEach(function (addr) {
self.client.emit('peer', addr)
})
addrs = compact2string.multi(msg.slice(20))
} catch (err) {
return self.client.emit('error', err)
return self.client.emit('warning', err)
}
addrs.forEach(function (addr) {
self.client.emit('peer', addr)
})
break
case 2: // scrape
@ -482,13 +484,15 @@ Tracker.prototype._handleResponse = function (requestUrl, data) {
if (Buffer.isBuffer(data.peers)) {
// tracker returned compact response
var addrs
try {
compact2string.multi(data.peers).forEach(function (addr) {
self.client.emit('peer', addr)
})
addrs = compact2string.multi(data.peers)
} catch (err) {
return self.client.emit('error', err)
return self.client.emit('warning', err)
}
addrs.forEach(function (addr) {
self.client.emit('peer', addr)
})
} else if (Array.isArray(data.peers)) {
// tracker returned normal response
data.peers.forEach(function (peer) {