Skip to content

Conversation

@vszakats
Copy link
Contributor

No description provided.

@vszakats
Copy link
Contributor Author

vszakats commented Apr 2, 2024

Any interest in this?

@goggle
Copy link
Owner

goggle commented Apr 3, 2024

Yes, absolutely, thank you for that PR.
Maybe some more information would be helpful ( I know, I can do this on my own, but I didn't yet...).

  • Does the current method of retrieving subtitles not work anymore?
  • Are all the subtitles served via https now?
  • Maybe you can provide an example?

@vszakats
Copy link
Contributor Author

vszakats commented Apr 3, 2024

Yes, absolutely, thank you for that PR. Maybe some more information would be helpful ( I know, I can do this on my own, but I didn't yet...).

  • Does the current method of retrieving subtitles not work anymore?

I haven't tested the insecure method, sorry.

Using curl, the insecure token endpoint http://tp.srgssr.ch/ redirects
to HTTPS with 302 Moved Temporarily.

  • Are all the subtitles served via https now?

AFAIK all content has been served via HTTPS for several years now.

The first program we tuned on worked without issues with these
modifications. It also worked for another program. And just now
with another one. It seems fair to assume that if the endpoints work
for a few programs, it shall work for all.

  • Maybe you can provide an example?

'10 vor 10 vom 02.04.2024'.

@vszakats
Copy link
Contributor Author

vszakats commented Apr 4, 2024

The subtitle server identifies itself as Amazon S3 behind Cloudfront. It doesn't redirect HTTP to HTTPS, and both do work.

This is the subtitle URL for the program mentioned above:
https://subtitles.eai-general.aws.srf.ch/srf/384d603f-0393-47bf-90f5-ad5f23122ee1/episode/de/vod/vod.m3u8

@goggle goggle merged commit 0f1b817 into goggle:master Apr 4, 2024
@vszakats vszakats deleted the patch-1 branch April 4, 2024 08:12
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.

2 participants