Skip to content
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.

Sign up for GitHub

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

Jump to bottom

oauth: change app name #1990

Merged
merged 2 commits into from
Dec 16, 2024
Merged

oauth: change app name #1990

merged 2 commits into from
Dec 16, 2024

Conversation

ntarocco
Copy link
Contributor

  • Change app name to avoid collisions with the cern_openid in invenio-oauthclient. Signals on_identity_* won't be called in Invenio module.

@ntarocco
oauth: change app name
6872010 
* Change app name to avoid collisions with the cern_openid in
  invenio-oauthclient. Signals on_identity_* won't be called in Invenio
  module.
@ntarocco ntarocco requested a review from zzacharo December 16, 2024 15:29
@ntarocco
Copy link
Contributor Author

To be tested in sandbox. When deploying, the only required change is to add/replace the redirect url in the CERN SSO app.

zzacharo
zzacharo previously approved these changes Dec 16, 2024
View reviewed changes
@ntarocco ntarocco merged commit 3f95458 into CERNDocumentServer:main Dec 16, 2024
1 check passed
@ntarocco ntarocco deleted the fix-oauth-e-groups branch December 16, 2024 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zzacharo zzacharo zzacharo left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ntarocco @zzacharo