-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Panic #7
Comments
Hi, can you post the lines above as well, so I can see which cmd triggered your error? Thanks. Quickfix would be changing line 291
Anyway I'll integrate more data checks soon, thanks for reporting this one :) |
Will try that and revert. Also I am noticing delay in music playback, after I select the video and the video starts playing on the Raspberry Pi. The delay is in the range 15 - 20 seconds. Is it normal? |
Yea the script uses youtube-dl, which can take around 15 secs to start and fetch the video stream URL. Because there is no open API for obtaining the stream link from a video ID, we have to rely on youtube-dl here for now. |
after Quickfix, the following build errors occurs: _/root/gotubecast./main.go:293: cannot use data["currentTime"].(string) (type string) as type int in assignment |
`panic: interface conversion: interface {} is nil, not string
goroutine 1 [running]:
main.genericCmd(0x1b, 0x0, 0x11d82360, 0xb, 0x11d84348, 0x1, 0x1)
/home/pi/go/src/github.com/CBiX/gotubecast/main.go:291 +0x40a0
main.decodeBindStream(0x74c6bd10, 0x11dfa4d0, 0x11dfa4d0, 0x74c6bd10)
/home/pi/go/src/github.com/CBiX/gotubecast/main.go:233 +0x104
main.main()
/home/pi/go/src/github.com/CBiX/gotubecast/main.go:191 +0x11d4`
The text was updated successfully, but these errors were encountered: