Oauth2 pure rest - Googe Summer of Code 2017 #4
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Tests fixed
Issue -> spring-core, spring-tx and spring-beans show up in packaged omod's lib folder
…pplicationContext.xml
next : intercept URL's
Demo for token endpoint and accessing FHIR resources.
What works 1) added login and authorization Authorization Code Grant Type 2) Configure Authorization Endpoint 3) Add AuthorizationController (currently handling error) What does not work 1) custom login page needs to be provided to intercept calls to Authorization Endpoint 2) spring-security-taglib jar is present in packaged omod, but gives error when module is installed Error : https://stackoverflow.com/questions/16099840/spring-security-taglib-cannot-be-found
dependency for spring security tags added
Blocker : https://trello.com/c/uCsUCXvd/81-userservice This will be resumed later
-> rename client_id to id
Client updated to persist fields from BaseOpenmrsData into oauth2_client table
update Client hibernate annotations to match liquibase schema definitions
…PI layer works smoothly
tests modified
…ds Parametrized> (used by Hibernate) to Set<String> (used by Spring Security) tests fixed
…test dataset (addition of uuid)
…ontext.getUserService().getUserbyUsername with Context.authenticate() instead
…entManagementController.java
Replace "Exception" with the specific exception that is raised in ClientManagementController.java
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.