Skip to content
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

Bug-fix: Use SNI with media origin #78

Merged
merged 1 commit into from
Oct 11, 2023

Conversation

davidjwbbc
Copy link
Contributor

Description of issue

It was noticed recently that the Azure sample media streams we have been using on rt.5g-mag.com are now returning a 403 forbidden.

The streams were changed to similar streams from https://rdmedia.bbc.co.uk/ however this then generated a 502 error. This is due to an unknown SSL handshake failure.

Remedy

The servers being used by rdmedia.bbc.co.uk are general CDN servers, so it was postulated that the "rdmedia.bbc.co.uk" name was not being passed in the SNI allowing the CDN node to determine which server it is being a reverse proxy of. A quick test with OpenSSL tools confirmed this.

The solution is to add proxy_ssl_server_name on; to the nginx http configuration to make it send an SNI when proxying requests.

With the fix in place on the Linode instance we get a normal response from rdmedia.bbc.co.uk.

@davidjwbbc davidjwbbc added the bug Something isn't working label Oct 11, 2023
@davidjwbbc davidjwbbc requested a review from dsilhavy October 11, 2023 13:11
@davidjwbbc davidjwbbc self-assigned this Oct 11, 2023
Copy link
Contributor

@dsilhavy dsilhavy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@davidjwbbc davidjwbbc merged commit 3fb6d24 into 5G-MAG:development Oct 11, 2023
@davidjwbbc davidjwbbc deleted the fix/use-SNI-with-proxy branch October 11, 2023 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants