Skip to content

T26 factory pattern #37

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 9 commits into
base: main
Choose a base branch
from
Open

T26 factory pattern #37

wants to merge 9 commits into from

Conversation

dolsysmith
Copy link
Collaborator

This branch implements a separate API endpoint for requesting tokens. To review, just test the following functionality on https://test.orcid.library.gwu.edu and on the orcid-integration-test EC2 instance:

  • Confirm that you are still able to connect an ORCiD sandbox account by browsing to the URL above.
  • Log onto the EC2 instance.
  • Create an API key by running the following command: docker exec -it orcid-integration-token-api-1 flask create-api-key [email protected]. (You can use your email address instead of mine.)
  • Request the token for your ORCiD using the following CURL command, substituting the API key for the masked version, and your ORCiD for the masked ORCiD:
curl --header "Authorization: Apikey XXXX-XX-XXX-XXXX"  localhost:8081/api/get-token?orcid=XXXX-XXXX-XXXX-XXXX
  • Confirm that an access token is returned.
  • Run the following commands to extract all the tokens from the test database:
docker exec -it orcid-integration-flask-app-1 flask serialize-db /tmp/token-dump.json
docker cp orcid-integration-flask-app-1:/tmp/token-dump.json ./data/
  • Run cat ./data/token-dump.json and confirm that the access token returned by the API matches the most recent access token associated with your ORCiD in the database.

@dolsysmith dolsysmith requested a review from kerchner April 30, 2025 14:33
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.

1 participant