Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR should fix the annoying comments parsing error discussed in #455 in NewPipe by converting the
responseBody
into a "valid" format by escaping specific characters. When using the mobile user-agent theytInitialData
is formatted with chars as\x
, which can't be parsed correctly.Test NewPipe apk (just changed the implementation link to this PR's branch): app-debug.zip
PS: Out of some reason I can't reply to the linked issue, so I'm responding to your question here, @TobiGr :
I've done some research on the headers and so. As said above the reason for this character mess-up is the mobile User-Agent which is required to get the necessary token - without or normal/desktop User-Agent it's formatted well but does contain a different token used for the desktop way, explained in detail in #457 (but the desktop version has also its problems because of POST and CORS protection).