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

Add optional HTTP origin parameter to Session. #43

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nosmo
Copy link

@nosmo nosmo commented Jan 25, 2021

Interally within WMF we do not access the API servers via external
hostnames, but we need to access the APIs for a given wiki via our
internal appserver endpoint. Currently this is not possible as using
the appserver hostname directly doesn't set a valid Host header for a
wiki. This change adds an origin parameter to the Session object
which allows the host to be used as the HTTP Host header while
connecting to the IP or hostname of the given origin.

Interally within WMF we do not access the API servers via external
hostnames, but we need to access the APIs for a given wiki via our
internal appserver endpoint. Currently this is not possible as using
the appserver hostname directly doesn't set a valid Host header for a
wiki. This change adds an `origin` parameter to the Session object
which allows the host to be used as the HTTP Host header while
connecting to the IP or hostname of the given `origin`.
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.

1 participant