Skip to content

OAUTH_UPDATE_PICTURE_ON_LOGIN documentation #533

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

kaytwo
Copy link

@kaytwo kaytwo commented May 7, 2025

feature landed in dev, presumably slated for v0.6.8 release.

@Classic298
Copy link
Contributor

Hello @kaytwo

Can you add two more infos to this PR please?

-> OAUTH_PICTURE_CLAIM must NOT be an empty string when you wanna use OAUTH_UPDATE_PICTURE_ON_LOGIN

Otherwise, if OAUTH_PICTURE_CLAIM is set to an empty string, the profile picture for all users will always be this default person icon

Please add this to your docs of OAUTH_UPDATE_PICTURE_ON_LOGIN as a " - Note: ...."

And if you want, please also add to the OAUTH_PICTURE_CLAIM docs, that setting OAUTH_PICTURE_CLAIM to an empty string "" has this behaviour (of always using the default profile icon logo for everyone)

🙏

@Classic298
Copy link
Contributor

Super! Thanks :)

Makes the docs 1% better by mentioning dependencies between env vars

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.

2 participants