Modify redirect_uri to force HTTPS for v2

Crivaledaz / Mattermost-LDAP

This module provides an external LDAP authentication in Mattermost for the Team Edition (free).

MIT License

359 stars 71 forks source link

Modify redirect_uri to force HTTPS for v2 #48

Closed grvsmth closed 2 years ago

grvsmth commented 4 years ago

The Mattermost server seems to be returning bare http urls, even if there is no http url in the config.json. If we are using https we need to modify them.

Crivaledaz commented 4 years ago

Same as PR #47.

grvsmth commented 4 years ago

Thanks, Denis! For some reason, when I had HTTPS in the site_url it was still returning HTTP. I will try and reproduce it.

Crivaledaz commented 2 years ago

I close this PR, since there is no update for 2 years. Besides, HTTPS is working with the docker-compose implementation, it just needs to be set up.

If you think your patch is always relevant, please let me know and give me more details to allow me to reproduce your issue.

Feel free to open an issue if you still does not succeed to set up HTTPS.