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

Use scope claim, not scp #3

Merged
merged 3 commits into from
Oct 3, 2020
Merged

Use scope claim, not scp #3

merged 3 commits into from
Oct 3, 2020

Conversation

jbasney
Copy link
Member

@jbasney jbasney commented Oct 2, 2020 •
edited
Loading

I just did a blind query/replace of scope for scp to update the demo for our current profile in an attempt to fix issue #2.

jbasney added 3 commits October 2, 2020 15:30
@jbasney
Update scitokens_protect.py
a8de58e 
check for scope, not scp
@jbasney
Update app.py
4a81b96 
scope, not scp
@jbasney
Update app.js
c8dc33e 
scope not scp
@jbasney jbasney requested a review from djw8605 October 2, 2020 20:35
@djw8605
Copy link
Contributor

djw8605 commented Oct 3, 2020

I'm going to merge and run some tests.

@djw8605 djw8605 merged commit c8dc33e into scitokens:master Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@djw8605 djw8605 Awaiting requested review from djw8605

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
@jbasney @djw8605