Skip to content

Relax client credentials grant requirement to SHOULD#253

Merged
jeswr merged 2 commits intomainfrom
relax-client-credentials-grant-requirement
Apr 23, 2026
Merged

Relax client credentials grant requirement to SHOULD#253
jeswr merged 2 commits intomainfrom
relax-client-credentials-grant-requirement

Conversation

@jeswr
Copy link
Copy Markdown
Member

@jeswr jeswr commented Apr 23, 2026

Summary

Updates the Solid-OIDC text to change support for OAuth 2.0 Client Credentials Grant from MUST to SHOULD in the Authorization Servers section.

Why

The original goal of adding Client Credentials language was to reflect a way script-based authentication can be done on Solid servers today.

Because not all Solid servers currently implement Client Credentials, this requirement has been downgraded from MUST to SHOULD to keep that path strongly recommended without making it universally mandatory.

Changes

  • In index.bs, changed:
    • Authorization Servers MUST support ...
    • to Authorization Servers SHOULD support ...

Copilot AI review requested due to automatic review settings April 23, 2026 01:32

This comment was marked as outdated.

@jeswr jeswr requested review from acoburn and uvdsl April 23, 2026 01:38
@jeswr
Copy link
Copy Markdown
Member Author

jeswr commented Apr 23, 2026

Since this is a class 3 change, does this PR need to be open for 10 days; or does that not apply since we are currently targeting the ED rather than the TR?

@elf-pavlik
Copy link
Copy Markdown
Member

elf-pavlik commented Apr 23, 2026

Given that it was added recently (~40 days ago) in

and it only adjusts requirement level to better reflect state of implementations, I think we can just go ahead and merge it.

@jeswr jeswr merged commit 7353ed3 into main Apr 23, 2026
1 check passed
@jeswr jeswr deleted the relax-client-credentials-grant-requirement branch April 23, 2026 12:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants