Skip to content

Commit

Permalink
Merge pull request #25 from ThisIsMissEm/feat/add-consideration-for-l…
Browse files Browse the repository at this point in the history
…ogo_uri

Add security consideration for logo_uri usage
  • Loading branch information
aaronpk authored Jan 10, 2025
2 parents bab0ab4 + 8fc35a6 commit 6158b5e
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions draft-parecki-oauth-client-id-metadata-document.md
Original file line number Diff line number Diff line change
Expand Up @@ -258,6 +258,12 @@ Authorization servers fetching the client metadata document and resolving URLs l
Authorization servers SHOULD limit the response size when fetching the client metadata document, as to avoid denial of service attacks against the authorization server by consuming excessive resources (memory, disk, database). The recommended maximum response size for client metadata documents is 5 kilobytes.


## Displaying Logos to End-Users

Authorization servers that wish to make use of the `logo_uri` property within client metadata document SHOULD prefetch the file at `logo_uri` and cache it for the cache duration of the client metadata document. This allows for moderation tools to verify the file contents (e.g., preventing usage of logos that look like other logos), as well as preventing the logo from being dynamically changed to confuse an end-user.

Caching of the `logo_uri` response can additionally prevent cross-domain tracking through the `logo_uri` being requested by the client, since the cached file would be served not from the remote URI but instead from a URI that the Authorization server trusts.

# IANA Considerations

## OAuth Authorization Server Metadata Registry
Expand Down

0 comments on commit 6158b5e

Please sign in to comment.