Skip to content

use webm as format for opus audio streams#199

Merged
TobiGr merged 6 commits intoTeamNewPipe:devfrom
kapodamy:yt-webm-opus
Nov 24, 2019
Merged

use webm as format for opus audio streams#199
TobiGr merged 6 commits intoTeamNewPipe:devfrom
kapodamy:yt-webm-opus

Conversation

@kapodamy
Copy link
Contributor

@kapodamy kapodamy commented Sep 23, 2019

opus suffix is only used if the container format is ogg but youtube delivers the audio in webm format.
This PR updates the itags in youtube extractor for webm

  • [ ✔ ] I carefully read the contribution guidelines and agree to them.
  • [ ✖ ] I did test the API against NewPipe.
  • [ ✔ ] I agree to ASAP create a PULL request for NewPipe for making in compatible when I changed the api.

@kapodamy kapodamy changed the title use webm as suffix in opus audio streams use webm as format for opus audio streams Sep 24, 2019
@theScrabi
Copy link
Member

I kind of messed this up when I didn't distinguish between stream and container format. This could one day be cleared up.

@kapodamy
Copy link
Contributor Author

kapodamy commented Oct 1, 2019

ok

@kapodamy kapodamy mentioned this pull request Oct 2, 2019
2 tasks
@TobiGr
Copy link
Contributor

TobiGr commented Nov 24, 2019

Sorry for the delay. We should have merged this right after @theScrabi's approval.

@TobiGr
Copy link
Contributor

TobiGr commented Nov 24, 2019

@kapodamy Can you rebase it please?

@TobiGr TobiGr merged commit b6d3252 into TeamNewPipe:dev Nov 24, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants