- Notifications
You must be signed in to change notification settings - Fork1.6k
fix: fosite refresh scope narrowing e2e#3403
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:master
Are you sure you want to change the base?
fix: fosite refresh scope narrowing e2e#3403
Uh oh!
There was an error while loading.Please reload this page.
Conversation
fd4ccdc toe9a1358Comparecodecovbot commentedJan 2, 2023 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Codecov Report
@@ Coverage Diff @@## master #3403 +/- ##==========================================- Coverage 76.74% 76.70% -0.04%========================================== Files 123 123 Lines 9022 9076 +54 ==========================================+ Hits 6924 6962 +38- Misses 1657 1672 +15- Partials 441 442 +1
Help us with your feedback. Take ten seconds to tell ushow you rate us. Have a feature suggestion?Share it here. |
This is a proof of conformance e2e test for the related issues.
Related issue(s)
ory/fosite#696
ory/fosite#718
Checklist
introduces a new feature.
contributing code guidelines.
vulnerability. If this pull request addresses a security vulnerability, I
confirm that I got the approval (please contact
security@ory.sh) from the maintainers to push
the changes.
works.
Further Comments
This is my first time working with cypress, and there may be a better way to handle the
refreshTokenBrowserScopeor I may have completely bungled it. The intent is for the flow to get an initial refresh token, narrow the scope, then broaden the scope again.