-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
plugins.wwenetwork: new api layout #5559
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
plugins.wwenetwork: new api layout #5559
Conversation
bastimeyer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the PR.
Since this site requires having an account, I can't verify these changes.
Please don't forget to update the plugin matcher tests. As you can see, the tests have failed.
|
There is no audio in the output file. |
|
@MaxosI As you can see from my logs I have streamlink finding audio tracks, and hence the final output with audio |
Muxing external audio tracks requires FFmpeg. FFmpeg is looked up automatically in the dirs of the user's
Please fix the linting error and provide a full debug log of both live streams and VODs, so I can see that the changes actually fix the plugin. Alternatively, one of you can send me auth data, so I can see for myself. As commented, the plugin is in a rough shape due to not being updated in a long time, so if I had auth data, I can have a look at rewriting and cleaning up the plugin. |
|
@bastimeyer And for live stream: |
bastimeyer
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks, @kitanoyoru
closes #5503
before changes:
after