search

GyrosOfWar / youtube-dl-rs

A youtube-dl wrapper for Rust
132 stars 40 forks source link

Added new protocol to enum : m3u8_native+https #52

Closed Francis0xf closed 1 year ago

Francis0xf commented 1 year ago

while trying to download this video: https://www.youtube.com/watch?v=MoKe4zvtNzA Serde panicked with the unknown variant 'm3u8_native+https' I added it to the protocol enum

BartWillems commented 1 year ago

I bumped into the same issue, would it also not make sense to add a catch-all unknown variant so it doesn't break in the future?

GyrosOfWar commented 1 year ago

I bumped into the same issue, would it also not make sense to add a catch-all unknown variant so it doesn't break in the future?

Yes, good point. Will do that.