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

requests start to run against public_baseurl #8232

Closed
verymilan opened this issue Jan 23, 2019 · 2 comments
Closed

requests start to run against public_baseurl #8232

verymilan opened this issue Jan 23, 2019 · 2 comments
Labels
T-Other Questions, user support, anything else

Comments

@verymilan
Copy link

I am running synapse with an nginx reverse-proxy.
Since the documentation states "The public-facing base URL for the client API (not including _matrix/...)", i configured the public_baseurl to a subdomain pointing directly to the box where synapse is running on.
I don't know if it is a weird coincidence but since the latest release of riot-web, new logins started to fail because they got redirected to the public_baseurl and ended in a 405.

Stangely enough, i changed this setting to the main url (with the reverse-proxy at /_matrix) and it seems to work just fine again.
At this point i don't know if i may broke something i am not aware of yet or if i simply fixed a bad setting in my synapse configuration, so i'd appreciate clarification on what just happened / what this setting is for - i had it like that for maybe a year or so without issues before.

Regards! :3

@dbkr
Copy link
Member

dbkr commented Jan 24, 2019

Interesting - so this was a misconfiguration of the public_baseurl in synapse: it should be the URL that clients use to access synapse, so if you have a reverse proxy, the URL that hits the reverse proxy, ie. your new configuration is correct.

The change that caused this will be matrix-org/matrix-react-sdk#2384 which went out with v0.17.9. This honours the URL that synapse now sends in the login response. This URL comes from the public_baseurl field, so setting it to a non-publicly accessible field will cause things to break.

I don't think we anticipated this kind of misconfiguration when doing this change, this probably should have been mentioned in some release notes or similar. I'll try & clarify the docs for public_baseurl though.

@kittykat
Copy link
Contributor

I'm going to close this for now as it sounds like an issue in synapse. Please re-open or file a new issue with more details if you feel differently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Other Questions, user support, anything else
Projects
None yet
Development

No branches or pull requests

4 participants