TheDcoder

@kasper93 Agreed, it's hacky, but it's a quick fix for an edge case, I'm still in favour of merging this until upstream has this sorted out, it might take some...

> Personally I would expect yt-dlp to return an error I was thinking more along the lines of just ignoring the request since dumping the JSON should take precedence ideally....

@guidocella I'm using `yt-dlp` and it seems to work when I try it directly but unfortunately it doesn't work with `mpv --ytdl-raw-options-append=no-list-formats=` for whatever reason.

@kasper93 Well that would be an issue, but it's highly unlikely. We can modify the code to loop until it finds the right `{` but I think it's overkill for...

I opened an issue in upstream: https://github.com/yt-dlp/yt-dlp/issues/14378 > I think I got the same results yesterday but now --no-list-formats doesn't work even on the command line, what gives? Perhaps it...

@sfan5 I'm sorry but I have to disagree, option 4 is just accepting that we can't handle certain valid (as considered by youtube-dl) options with a very real downside of...

@dirkf That's an ideologically different take than what's currently implemented, I agree that having a separate yt-dl config for mpv has its benefits but it is my opinion that by...

@dirkf Pardon me, but I don't really understand what you're suggesting, it just sounds like a more complicated way to do what are already able to do in mpv currently...

That's great news, we can now work around this by explicitly passing that option in mpv... so a new PR? :thinking: Also since this is a fix in the forked...

I just realized that I forgot to include `load_oswap_crs` in the config, so I did that: ``` (waf) { coraza_waf { load_owasp_crs directives ` Include @coraza.conf-recommended Include @crs-setup.conf.example Include @owasp_crs/*.conf...