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

3PID invites should include the configured web client instance / homeserver url in the email #8863

Closed
Half-Shot opened this issue Dec 2, 2020 · 1 comment · Fixed by #8930
Assignees

Comments

@Half-Shot
Copy link
Collaborator

Half-Shot commented Dec 2, 2020

Related element-hq/element-meta#1281

Some context in https://matrix.to/#/!oBWlVutBAdQYcYUsAc:sw1v.org/$pnwt-VMwwf3pC7_QET_e2jk_ojfRAPTNZ9fj2ag-Uio?via=jki.re&via=matrix.org&via=vector.modular.im

(This also applies to Sydent, but the work is 50/50 across both projects)

Synapse currently uses the identity server to send 3pid invites out to users (so it sends an email when an email address is invited to a room). However, the templates do not include and do not have a way to include the target web client instance. Instead, all invites include app.element.io.

This is a problem if you are running your own custom Element / Synapse deployment and wish to send invited users to your specific instance, especially so if your instance is not federated.

There are several ways to resolve this:

  • Invites could be sent by Synapse (and thus, the template could be configured by the owner).
  • Synapse could send the config value of web_client_location to sydent, and the template could use that.
  • Synapse could send the homeserver URL to sydent, and sydent could include a parameter to target the right homeserver. This would require Bring back the hs_url parameter and document it element-hq/element-web#5469
@Half-Shot Half-Shot changed the title 3PID invites should include the web client instance / homeserver url in the email 3PID invites should include configured web client instance / homeserver url in the email Dec 2, 2020
@Half-Shot Half-Shot changed the title 3PID invites should include configured web client instance / homeserver url in the email 3PID invites should include the configured web client instance / homeserver url in the email Dec 2, 2020
@clokep
Copy link
Contributor

clokep commented Dec 11, 2020

Synapse could send the config value of web_client_location to sydent, and the template could use that.

It seems like the identity service spec would allow additional config to be sent:

In addition to the request parameters specified below, an arbitrary number of other parameters may also be specified. These may be used in the invite message generation described below.

Given that, I think this is the easiest way forward. There is a config option for Synapse to know where to point to (web_client_location), ~~but that is deprecated~.So I'm not sure where this should be from instead.~~ so that seems like the way forward.

Edit: I have no idea why I thought it was deprecated. 🤷

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants