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

Replace Keto Authorization with External HTTP Authorization #864

Merged
merged 11 commits into from
Jul 10, 2020
Merged

Replace Keto Authorization with External HTTP Authorization #864

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
83d3a65
Replace Keto Auth with external HTTP Auth
woop Jul 8, 2020
ecfa725
Update dependencies for Auth Java client in Auth module
woop Jul 8, 2020
78429bb
Allow JavaDoc linting to be disabled for Auth module
woop Jul 8, 2020
5d11b43
Add exclusion for DefaultAPI that causes failure in javadoc lint
woop Jul 8, 2020
36de119
Remove AuthUtil
woop Jul 8, 2020
e91eb61
Clean up dependencies
woop Jul 8, 2020
cf7fe41
Improve class for comment for HTTPAuthorizationProvider
woop Jul 8, 2020
069541d
Fix casing for HTTPAuthProvider
woop Jul 8, 2020
ee786c3
Update OpenAPI specification to support resources for auth
woop Jul 9, 2020
825b07f
Update External HTTP Authorization Provider to use a more generalized…
woop Jul 10, 2020
65b535c
Update formatting
woop Jul 10, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Update formatting
  • Loading branch information
@woop
woop committed Jul 10, 2020
commit 65b535c75bcbf7492c15147663de9b44bf4a3986
5 changes: 3 additions & 2 deletions auth/src/main/java/feast/auth/authorization/HttpAuthorizationProvider.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,8 +103,9 @@ public AuthorizationResult checkAccessToProject(String projectId, Authentication
/**
* Extract a context object to send as metadata to the authorization service
*
* @param authentication Spring Security Authentication object
* @return Returns a context object that will be serialized and sent as metadata to the authorization service
* @param authentication Spring Security Authentication object
* @return Returns a context object that will be serialized and sent as metadata to the
* authorization service
*/
private Object getContext(Authentication authentication) {
// Not implemented yet, left empty
Expand Down