Error: Too many redirects (from miniget used by ytdl-core)

[DevYukine]

2|Senpai   | Error: Too many redirects
2|Senpai   |     at ClientRequest.<anonymous> (/root/Senpai-Bot/Senpai-Bot-Discord/node_modules/miniget/lib/index.js:59:19)
2|Senpai   |     at Object.onceWrapper (events.js:316:30)
2|Senpai   |     at emitOne (events.js:115:13)
2|Senpai   |     at ClientRequest.emit (events.js:210:7)
2|Senpai   |     at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:565:21)
2|Senpai   |     at HTTPParser.parserOnHeadersComplete (_http_common.js:116:23)
2|Senpai   |     at TLSSocket.socketOnData (_http_client.js:454:20)
2|Senpai   |     at emitOne (events.js:115:13)
2|Senpai   |     at TLSSocket.emit (events.js:210:7)
2|Senpai   |     at addChunk (_stream_readable.js:266:12)
2|Senpai   |     at readableAddChunk (_stream_readable.js:253:11)
2|Senpai   |     at TLSSocket.Readable.push (_stream_readable.js:211:10)
2|Senpai   |     at TLSWrap.onread (net.js:585:20)

i get this error often in the last time and im not quit sure how to handle properly since i cant catch it property (seems to be thrown in the miniget lib or smth) and maybe you could tell me if its an network issue or something like that

[fent]

Looks similar to the error from #187, but that one is too many reconnects. So possibly, this is a separate issue.

Do you have an example video where this happens?

[DevYukine]

oh not at the moment, i will try to get one ASAP to help you investigate the issue :)

[juusolain]

I got the same while testing, and it appears that if the video does not exist, this happens:

   at ClientRequest.<anonymous> (/home/juusolain/CordBot/node_modules/miniget/lib/index.js:59:19)
   at Object.onceWrapper (events.js:316:30)
   at emitOne (events.js:115:13)
   at ClientRequest.emit (events.js:210:7)
   at HTTPParser.parserOnIncomingClient [as onIncoming] (_http_client.js:565:21)
   at HTTPParser.parserOnHeadersComplete (_http_common.js:116:23)
   at TLSSocket.socketOnData (_http_client.js:454:20)
   at emitOne (events.js:115:13)
   at TLSSocket.emit (events.js:210:7)
   at addChunk (_stream_readable.js:266:12)

This is probably the same thing

[DevYukine]

oh yea that would make sense since you get redirected if the video was claimed/deleted if i remember correct

[FireController1847]

Ahh it's happening still @fent pls fix kthx <3

[arjndr]

Yep, still happening sometimes.

[fent]

Are there any videos where this consistently happens?

[jimbuck]

My unit tests are now failing with this: https://www.youtube.com/watch?v=oVXg7Grp1W8

But I can override them with an environment variable and this worked: https://www.youtube.com/watch?v=OLxY0HDakRk

[mandazi]

It is happening with this video: https://www.youtube.com/watch?v=PimVw0LomLs

Using ytb2mp3:
ytb2mp3/ytb2mp3#2

[fent]

It is happening with this video: https://www.youtube.com/watch?v=PimVw0LomLs

Ok I'm able to replicate with itag 22 of this. Will investigate.

Using ytb2mp3:
ytb2mp3/ytb2mp3#2

This is using an outdated version of ytdl-core btw. You can see in the package-lock.json file. The error still happens with the video above, but you should always be using the latest version or you might run into other errors.