-
Notifications
You must be signed in to change notification settings - Fork 25
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
Feat/auth handler #1562
base: main
Are you sure you want to change the base?
Feat/auth handler #1562
Conversation
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.
Thanks for getting this started!
import java.util.regex.Matcher; | ||
import java.util.regex.Pattern; | ||
|
||
public class ContinuousAccessEvaluationClaims { |
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.
could we restrict this to package instead?
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.
Wouldn't be possible since it's used both under the ...http
and ...http.middleware
packages.
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Show resolved
Hide resolved
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Vincent Biret <[email protected]>
293575b
to
5fa1f82
Compare
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.
Great progress here! just a few minor comments.
* Helper class to extract the claims from the WWW-Authenticate header in a response. | ||
* https://learn.microsoft.com/en-us/entra/identity/conditional-access/concept-continuous-access-evaluation | ||
*/ | ||
public final class ContinuousAccessEvaluationClaims { |
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.
do we need this class to be public? can't we restrain it to package so we're not shipping additional API on this side?
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.
It's being used in the com.microsoft.kiota.http
package and com.microsoft.kiota.http.middleware
package.
Restricting it would make it only visible in the main http
package and not in the middleware
package and vice versa. Java doesn't have an equivalent of internal
in .NET
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
...ents/http/okHttp/src/main/java/com/microsoft/kiota/http/middleware/AuthorizationHandler.java
Outdated
Show resolved
Hide resolved
86923aa
to
08c471b
Compare
Co-authored-by: Vincent Biret <[email protected]>
2d9ccc7
to
c9f333c
Compare
Quality Gate passedIssues Measures |
closes microsoftgraph/msgraph-sdk-java#2096