plugins.kick: fix 403 error and VOD URL matcher #6384
Merged
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.
Closes #6325
See #6325 (comment) (credits to @Hakkin)
This seems to fix the 403 response just fine, so let's add the header to the entire HTTP session when initializing the plugin. Alternatively, the header could be added by the custom
KickAdapter
, but then the adapter would also need to be mounted on other (sub-)domains where the HLS stream data is accessed from. Or it could be set on all API requests and HLSStream instances, but that would be a bit redundant and therefore unnecessary.In addition to fixing the 403 responses, this PR also fixes the VOD URL matcher. Surprisingly though, some VODs I checked still returned 403 when accessing the HLS playlist, which made me believe that the header didn't work. Turns out that those VODs are not accessible on their site either.