-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Core: Allow customizing OAuth scope #6616
Conversation
3c9963e
to
64c1da5
Compare
core/src/test/java/org/apache/iceberg/rest/TestRESTCatalog.java
Outdated
Show resolved
Hide resolved
0e925ed
to
05f6763
Compare
05f6763
to
4b5ee1a
Compare
response.token(), | ||
response.issuedTokenType(), | ||
credential, | ||
parent.scope()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When this is called in fromTokenExchange
, the scope passed there is not passed in here. I think that this method needs to have a scope
argument. Then the fromTokenResponse
above can pass parent.scope()
and fromTokenExchange
can pass the scope it was passed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've simplified this to actually and removed the scope
parameter from fromTokenExchange
/fromCredential
as I think we need to take the scope
from the parent
in those cases
4b5ee1a
to
909d3bb
Compare
In #6169 we're planning to re-use
AuthSession
but we need the ability to use ascope
, which is different from the defaultOAuth2Properties.CATALOG_SCOPE