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

Fix HTTPS #1159

Merged
merged 3 commits into from
May 10, 2023
Merged

Fix HTTPS #1159

merged 3 commits into from
May 10, 2023

Conversation

fsouza
Copy link
Owner

@fsouza fsouza commented May 10, 2023

Make sure the gRPC multiplexing works with HTTPS.

Fixes #1157.

fsouza added 2 commits May 9, 2023 21:30
Also remove the listener parameter, as that's better. One thing that
bothers me right now is the duplicated certificate handling, but we can
improve that later.

This should fix HTTPS support.

Closes #1157.
@fsouza fsouza force-pushed the fix-https branch 2 times, most recently from e56354b to af02b5c Compare May 10, 2023 01:35
This should prevent breakages in the future.
@fsouza fsouza merged commit effd073 into main May 10, 2023
@fsouza fsouza deleted the fix-https branch May 10, 2023 01:41
fsouza added a commit that referenced this pull request May 27, 2023
This is a regression introduced in #1159: when default parameters are
used, the external URL would be empty, breaking resumable uploads.

Fixes #1166.
Fixes #1170.
fsouza added a commit that referenced this pull request May 27, 2023
This is a regression introduced in #1159: when default parameters are
used, the external URL would be empty, breaking resumable uploads.

Fixes #1166.
Fixes #1170.
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.

SSL issue with 1.45
1 participant