-
Notifications
You must be signed in to change notification settings - Fork 17
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
Epic/cognito/develop #186
Open
yasima-csiro
wants to merge
83
commits into
dev
Choose a base branch
from
epic/cognito/develop
base: dev
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Epic/cognito/develop #186
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
- Add two Spring profiles, one for DB based and one for Cognito based users
* disassociated GORM domain objects from grails and dynamically register based on profile merged UserService implementations from cognito develop branch using base domain objects * moved domain object into new package and manually config as GORM artifacts for the gorm profile * Fix gorm user service issues Co-authored-by: Bruce Hyslop <[email protected]>
…ure/aws-credentials-update Add AWS default credentials support
* initial commit for plugin based userdetails * Progress update gorm / cognito plugins: - Add proper domain classes for gorm - Move authorised systems / mobile key controllers to exclusive to gorm impl - Fix transitive dependency resolution in base plugin Still TODO: - Remove GORM usage from Controllers / Grails services * Move all GORM code into GORM impl
…ile based config from plugin
* Userdetails Cognito: - Migrate signup page #122 - Updates to my profile page #127 - Delegate existing user service to an implementation provided by a Spring profile determined bean - Add *Record classes to mirror existing domain classes so that services aren't inherently coupled to GORM - Remove some direct GORM usage from Controller actions * Add cognito password reset branch * Add cognito password reset * disassociated GORM domain objects from grails and dynamically register based on profile merged UserService implementations from cognito develop branch using base domain objects * Add cognito password reset * Add cognito password reset * Add create user account * Fix logout issue * Add banner config * Fix reset password related issues * moved domain object into new package and manually config as GORM artifacts for the gorm profile * Fix gorm user service issues * Add MFA set up * Fix issues * enable travis build * Fix failing test cases * Update domain * Ignore failing tests * Remove bulk user upload and export users admin features * Update user update functionality * Fix breadcrumbs * Fix issues * Update email label * Update wordings * Add logs * Remove logs * Add Capcha for reset password * Clean up missed merge additions, add MFA service call return types * Move GORM password reset into GORM user service, remove unused updateUser method * Add assets directories for apps * Fix for unit tests, extract authorised system lookup * Move plugin Spring beans to separate @configuration file, fix Bootstrap syntax Co-authored-by: Simon Bear <[email protected]> Co-authored-by: Bruce Hyslop <[email protected]>
- Update IUserService to return PagedResults for various methods - Add a paginate tag that can work with next page tokens instead of offset indicies - Update controllers to accept PagedResults - Remove unused role and userRole views
- Update CognitoUserService for IUserService PagedResults - Fix return type of cognitoIdpClient bean in Application.groovy
…ure/gsp-user-id Add id field to UserRecord
…ure/roles-as-groups Use cognito groups for roles
- Extract password operations from IUserService to IPasswordOperations to fix circular dependency between PasswordService and UserService - Fix GORM saving userId by changing userId to a read only property on UserRecord - Remove @EqualsAndHashCode from domain classes - Remove displayName from base UserRecord as it's unused
- Add password policy to model of createAccount and resetPassword views - Add separate userdetails external config files for gorm and congito for easier development switching
…ix/gorm-issues Extract password operations and other cleanup
* Refactor web services #141 Co-authored-by: Simon Bear <[email protected]>
…-of-password-contains-special-characters # Conflicts: # userdetails-gorm/src/test/groovy/au/org/ala/userdetails/gorm/PropertyControllerSpec.groovy # userdetails-plugin/grails-app/services/au/org/ala/userdetails/PasswordService.groovy
…admin-reset-of-password-contains-special-characters #180 fix symbol character set used in temporary password generation
…ilInUse Fix isEmailInUse check to check users from both cognito pool and CAS db.
…ame_to_email Update username to email
2ec2275
to
aea4148
Compare
* Userdetails grails6 upgrade * Userdetails grails6.1.2 upgrade * Grails 6.2.0 and further Grails 6 updates * Update plugins * Update plugin versions * Update plugins * Fix grails 6.2.0 upgrade issues * Add ehcache, bump mysql driver, move hibernate-jcache out of boilerplate grails block * Bump Grails Gradle Plugin to 6.2.0 --------- Co-authored-by: dewmini <[email protected]> Co-authored-by: Simon Bear <[email protected]>
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.