Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

public_baseurl spelling confusion #5887

Closed
cueman opened this issue Aug 20, 2019 · 1 comment
Closed

public_baseurl spelling confusion #5887

cueman opened this issue Aug 20, 2019 · 1 comment
Labels
z-bug (Deprecated Label) z-p2 (Deprecated Label)

Comments

@cueman
Copy link

cueman commented Aug 20, 2019

Description

If you do not set public_baseurl and configure synapse for password reset emails, you get an error message that tells you it is missing "public_base_url" - with an extra underscore in there.

I went looking for public_base_url because of this, and then realised it should be public_baseurl.

There are two places where there is a reference to public_base_url, and both should be changed to public_baseurl:

  • synapse/config/emailconfig.py
  • synapse/rest/well_known.py

Steps to reproduce

  • do not set public_baseurl in homeserver.yaml
  • configure synapse for password reset emails
  • restart synapse
  • logs provide message:

Password resets emails are configured to be sent from this homeserver due to a partial 'email' block. However, the following required keys are missing: public_base_url

Version information

  • Homeserver: private server
  • Version: 1.3.1 and can be seen in master for tag v1.3.1 and the current tip of branch develop

  • Install method:

ubuntu apt install as described here:

  • Platform: ubuntu 18.04
@neilisfragile neilisfragile added z-bug (Deprecated Label) z-p2 (Deprecated Label) labels Aug 23, 2019
@richvdh
Copy link
Member

richvdh commented Aug 27, 2019

fixed by #5909

@richvdh richvdh closed this as completed Aug 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
z-bug (Deprecated Label) z-p2 (Deprecated Label)
Projects
None yet
Development

No branches or pull requests

3 participants