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

Jump to bottom

739 implement ory integration #740

Closed
wants to merge 232 commits into from
Closed

739 implement ory integration #740

wants to merge 232 commits into from

Conversation

Bdegraaf1234
Copy link
Collaborator

@Bdegraaf1234 Bdegraaf1234 commented Oct 24, 2023 •
edited
Loading

Description: Stage 1 implementation of Ory based identity management.

  • Adds support for 2FA and social signin

Organizational users can log in using Ory-kratos.

Fixes #739 partially

Checklist:

  • The Main workflow has succeeded
  • The Gatling tests have passed
  • I have logged into the portal running locally with default admin credentials
  • I have updated the README files if this change requires documentation update
  • I have commented my code, particularly in hard-to-understand areas

@Bdegraaf1234 Bdegraaf1234 linked an issue Oct 24, 2023 that may be closed by this pull request
Implement ory integration #739
Open
9 tasks
@Bdegraaf1234 Bdegraaf1234 force-pushed the 739-implement-ory-integration branch from 875cdfc to 7afd9c7 Compare November 13, 2023 13:30
@Bdegraaf1234 Bdegraaf1234 changed the base branch from master to dev November 21, 2023 11:00
@Bdegraaf1234 Bdegraaf1234 self-assigned this Nov 21, 2023
@Bdegraaf1234
extremely basic login via radar-ory
33b67b0
@Bdegraaf1234
working ory kratos auth proof of concept
4c90a09
@Bdegraaf1234
iteratively try to validate tokens, no need to mark the kratos header
419b719
@Bdegraaf1234
all verify functions should be marked 'suspend'
6e4eab9
@Bdegraaf1234
refactor: the runblocking seems needed, this should probablye be remo…
9678f26 
…ved later...
@Bdegraaf1234
import dependencies for ktor http requests
ed72c61
@Bdegraaf1234
Revert "Rename .java to .kt"
0f48e66 
This reverts commit 3dc71eaf755db9d13948affbdf4964fa71ff35c3.
@Bdegraaf1234
Parallel login working
732d7e4
@Bdegraaf1234
Create a kratos account from managementportal
c98434d
@Bdegraaf1234
refactor with better design
1fde2aa
@Bdegraaf1234
2fa required for ory login
f2776ae
@Bdegraaf1234
config files should have valid identityServer.serverUrl
e60567c
@Bdegraaf1234
fix test initialization (provide serverUrl)
6f8c633
@Bdegraaf1234
add proper coroutine support in the relevant tests, catch errors in i…
57e766f 
…dentity saving for now
@Bdegraaf1234
logout via makeshift button
2c76f7a
@Bdegraaf1234
Rename .java to .kt
c2b4baf
@Bdegraaf1234
last file in radar auth to kotlin
e529b21
@Bdegraaf1234
Rename .java to .kt
2d1409a
@Bdegraaf1234
first conversions, tests passing
ae6b161
@Bdegraaf1234
Rename .java to .kt
bf6fbf4
@Bdegraaf1234
finished config folder
257154e
@Bdegraaf1234
Rename .java to .kt
1151c3b
@Bdegraaf1234
finished config folder
261a461
@Bdegraaf1234
CustomRevisionListener needs to be a @component
ca26023
@Bdegraaf1234
tests passing
2dfefca
@Bdegraaf1234
Rename .java to .kt
06b90f5
@Bdegraaf1234 Bdegraaf1234 force-pushed the 739-implement-ory-integration branch from 4d6bf37 to c512ecb Compare January 26, 2024 10:45
peyman-mohtashami and others added 18 commits January 29, 2024 08:36
@peyman-mohtashami
Merge origin
69a99e8
@peyman-mohtashami
Fix UI issues - login, logout buttons and kratosId in user edit dialog
8ea418f
@Bdegraaf1234
set kratos public url to the right url since the prod profile gets pr…
af1f5bb 
…iority over e2e-test-prod on k3s (but not in spring bootrun)
@Bdegraaf1234
improved logging
b599b63
@Bdegraaf1234
working prototype: mp login, appconfig login, no appconfig api access
9b802bf
@Bdegraaf1234
qol
59d92f7
@Bdegraaf1234
API access working
404711a
@Bdegraaf1234
better logging
5b2cde8
@Bdegraaf1234
send emails from management-portal instead of kratos
36db36f
@Bdegraaf1234
fixes for kratos-ui integration in dev setups
4e25e5a
@Bdegraaf1234
revert csrf().disable(), this was not needed and a security risk
9872329
@peyman-mohtashami
Add environment variables to Dockerfile
848d507
@peyman-mohtashami
Add environment variables to Dockerfile
7f3c67f
@peyman-mohtashami
Make kratos id optional in user dialog
8765f89
@Bdegraaf1234
dynamically link the token endpoint
15694aa
@Bdegraaf1234
Merge branch '739-implement-ory-integration' of https://github.com/RA…
5e8473d 
…DAR-base/ManagementPortal into 739-implement-ory-integration
@Bdegraaf1234
fix dockerfile
2fe83fa
@Bdegraaf1234
kratos-ui instead of kratos
83c26d1
@Bdegraaf1234
Copy link
Collaborator Author

moved to #793

@Bdegraaf1234 Bdegraaf1234 deleted the 739-implement-ory-integration branch July 24, 2024 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peyman-mohtashami peyman-mohtashami Awaiting requested review from peyman-mohtashami peyman-mohtashami will be requested when the pull request is marked ready for review peyman-mohtashami is a code owner

@pvannierop pvannierop Awaiting requested review from pvannierop pvannierop will be requested when the pull request is marked ready for review pvannierop is a code owner

@yatharthranjan yatharthranjan Awaiting requested review from yatharthranjan yatharthranjan will be requested when the pull request is marked ready for review yatharthranjan is a code owner

@mpgxvii mpgxvii Awaiting requested review from mpgxvii mpgxvii will be requested when the pull request is marked ready for review mpgxvii is a code owner

Assignees

@Bdegraaf1234 Bdegraaf1234

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Implement ory integration
2 participants
@Bdegraaf1234 @peyman-mohtashami