Skip to content

Add information on the newly supported OAuth flows#1484

Merged
BenCookie95 merged 8 commits into
masterfrom
oauth-updates
Jan 6, 2026
Merged

Add information on the newly supported OAuth flows#1484
BenCookie95 merged 8 commits into
masterfrom
oauth-updates

Conversation

@BenCookie95

Copy link
Copy Markdown
Contributor

Summary

We added support for the following in v11.2:

  • DCR
  • Public clients
  • PKCE for public and confidential clients

Ticket Link

https://mattermost.atlassian.net/browse/MM-65810
https://mattermost.atlassian.net/browse/MM-65811

@github-actions

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA 235d1c0

@github-actions

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA ceca66c

@github-actions

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA 7e4b0d5

@cwarnermm cwarnermm added 1: Dev Review Requires review by a core commiter 2: Editor Review Requires review by an editor labels Dec 2, 2025

@esarafianou esarafianou left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's awesome, thanks @BenCookie95. Approving but just a minor comment that I'd prefer we address before merging.

| Client Type | Authorization Code Flow | Implicit Flow | PKCE Required |
|---------------------|-------------------------|---------------|----------------------|
| Public Client | Supported | Supported | YES (code flow only) |
| Confidential Client | Supported | Supported | Optional |

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we add that implicit flow for confidential clients is strongly not recommended even if it's is currently supported?

@cwarnermm cwarnermm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @BenCookie95!

@cwarnermm cwarnermm removed the 2: Editor Review Requires review by an editor label Dec 9, 2025
@github-actions

github-actions Bot commented Dec 9, 2025

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA 4b4373d

@github-actions

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA 54241db

@github-actions

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA 19dca78

@github-actions

github-actions Bot commented Jan 6, 2026

Copy link
Copy Markdown

Newest code from mattermost has been published to preview environment for Git SHA 5e8bb77

@BenCookie95
BenCookie95 merged commit 9632680 into master Jan 6, 2026
4 checks passed
@BenCookie95
BenCookie95 deleted the oauth-updates branch January 6, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1: Dev Review Requires review by a core commiter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants