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.

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

Users should be persisted in separate context #250

Open
2 of 3 tasks
blcham opened this issue Oct 7, 2024 · 5 comments · Fixed by kbss-cvut/record-manager#87
Open
2 of 3 tasks

Users should be persisted in separate context #250

blcham opened this issue Oct 7, 2024 · 5 comments · Fixed by kbss-cvut/record-manager#87
Assignees

Comments

@blcham
Copy link

blcham commented Oct 7, 2024

Same way as it is done in fta-fmea tool.

A/C:

  • Person is saved in separate context
  • there is a migration script in migration directory to move Person entity from default context to the correct context (see example of such script here. I suggest to call it person-from-default-to-separate-context.ru.
  • change configuration of keycloak to put users to the separate context
@blcham
Copy link
Author

blcham commented Oct 7, 2024

It would be easier to implement after #208

@palagdan
Copy link
Collaborator

palagdan commented Dec 9, 2024

@blcham,

It’s a bit confusing. I checked the implementation in FTA-FMEA and noticed that some users (test1 and test2) are in a separate context:
http://context-with-users
Image

In the internal authorization, I created two users, and they are stored without a context:
Image

I’m not sure which option I should implement in RM:

Create a new context for each user.
Use a common context, like http://context-with-users.

@blcham
Copy link
Author

blcham commented Dec 10, 2024

@palagdan All users should be in one context.

@blcham
Copy link
Author

blcham commented Jan 10, 2025

@palagdan I transfered this issue to record-manager-ui and updated A/C.

Please configure docker-compose.yml so that keycloak-graphdb-user-replicator puts users into separate context. It should be tested locally.

see https://github.com/kbss-cvut/keycloak-graphdb-user-replicator

and parameter NAMESPACE and DB_SERVER_CONTEXT

@blcham
Copy link
Author

blcham commented Jan 10, 2025

@palagdan I created PR for last point in A/C, please revise and make sure that you understand how it works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants