-
Notifications
You must be signed in to change notification settings - Fork 451
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
Unable to update OAuth scope for authenticated ORCIDs #10819
Labels
Enhancement:1:Minor
A new feature or improvement that can be implemented in less than 3 days.
Milestone
Comments
ewhanson
added a commit
to ewhanson/pkp-lib
that referenced
this issue
Jan 31, 2025
ewhanson
added a commit
to ewhanson/ojs
that referenced
this issue
Jan 31, 2025
ewhanson
added a commit
to ewhanson/pkp-lib
that referenced
this issue
Jan 31, 2025
ewhanson
added a commit
to ewhanson/ojs
that referenced
this issue
Jan 31, 2025
ewhanson
added a commit
to ewhanson/ojs
that referenced
this issue
Jan 31, 2025
ewhanson
added a commit
to ewhanson/ops
that referenced
this issue
Jan 31, 2025
ewhanson
added a commit
to ewhanson/omp
that referenced
this issue
Jan 31, 2025
Hey @bozana, could you have a look at these PRs (noted in the issue description above)? Thanks! |
Hi @ewhanson, I took a look and I have just a few questions for pkp-lib PR, the others are fine. |
ewhanson
added a commit
to ewhanson/pkp-lib
that referenced
this issue
Feb 7, 2025
ewhanson
added a commit
to ewhanson/ojs
that referenced
this issue
Feb 7, 2025
ewhanson
added a commit
to ewhanson/ops
that referenced
this issue
Feb 7, 2025
ewhanson
added a commit
to ewhanson/omp
that referenced
this issue
Feb 7, 2025
ewhanson
added a commit
that referenced
this issue
Feb 7, 2025
ewhanson
added a commit
to pkp/ojs
that referenced
this issue
Feb 7, 2025
ewhanson
added a commit
to pkp/ops
that referenced
this issue
Feb 7, 2025
ewhanson
added a commit
to pkp/omp
that referenced
this issue
Feb 7, 2025
Merged! Thanks @bozana! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Enhancement:1:Minor
A new feature or improvement that can be implemented in less than 3 days.
Describe the issue
As background, there are two different API types for ORCID, the public and member APIs. They each make use of different OAuth scopes when authenticating users. If OJS users authenticate their ORCID via OAuth with the ORCID service, they are giving permission to a specific scope depending on which API version the journal is using. If a journal switches to use the member API in order to deposit items (submissions/reviews) with ORCID, previously authenticated ORCIDs stored in OJS will not have the correct permissions to make the deposit because they were previously authorized with a more limited scope.
Describe the solution
OJS needs to be able to handle the switchover from the public API to the member API gracefully.
The following will accomplish this:
TODOs, PRs, Notes
PRs
pkp-lib
: pkp/pkp-lib#10819 Unable to update OAuth scope for authenticated ORCIDs #10872ojs
: pkp/pkp-lib#10819 Unable to update OAuth scope for authenticated ORCIDs ojs#4632ops
: pkp/pkp-lib#10819 Unable to update OAuth scope for authenticated ORCIDs ops#857omp
: pkp/pkp-lib#10819 Unable to update OAuth scope for authenticated ORCIDs omp#1822Notes
Important
This issue changes the default cache driver from
opcache
tofile
. If you have previously manually set the cachedefault
toopcache
, you will need to set it tofile
for certain features to work properly. This issue introduces the use of unique jobs, which require use of thefile
cache driver.The text was updated successfully, but these errors were encountered: