Skip to content

Fix Mattermost integration

Valery Sizov requested to merge mattermost_fixes into master

What does this MR do?

MM introduced started using a cookie in OAuth2 flow which broke our MM integration. This is the fix, and this also improves (a little) the error handling.

Are there points in the code the reviewer needs to double check?

Dunno

Why was this MR needed?

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

https://gitlab.com/gitlab-org/gitlab-ce/issues/31835

Edited by Valery Sizov

Merge request reports