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

allow to init some user from environment variable to init some users without using the UI #135

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

StudioEtrange
Copy link

@StudioEtrange StudioEtrange commented Oct 21, 2024

allow to init some user from environment variable to init some users without using the UI
it is very confortable when we want to fully automatize ownfoil deployment without any humain intervention

using environment variable
USER_ADMIN_NAME
USER_ADMIN_PASSWORD
to create/update an admin user

using environment variable
USER_GUEST_NAME
USER_GUEST_PASSWORD
to create/update a regular user

…without using the UI

Using environment variable 
USER_ADMIN_NAME
USER_ADMIN_PASSWORD
to create/update an admin user 

Using environment variable 
USER_GUEST_NAME
USER_GEST_PASSWORD
to create/update a regular user
Add USER_ADMIN_NAME and USER_GUEST_NAME environment variable sample
@StudioEtrange
Copy link
Author

Anybody ? :)

@a1ex4
Copy link
Owner

a1ex4 commented Dec 19, 2024

Yes this is definitely something I want to merge, but there are problems with the code, all the user management code should be in the auth module, not in the main app.
Also the code to add an user is duplicated, the auth module should be refactored to separate a function that can be reused.

@StudioEtrange
Copy link
Author

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 this pull request may close these issues.

2 participants