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

Fix For issue #541 #546

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Aditya31398
Copy link

Issue #541
Added Check admin user exists before creating new admin user

Added Check admin user exists before creating new admin user
@welcome
Copy link

welcome bot commented Nov 5, 2022

Thanks so much for opening this pull request and for helping to improve SirixDB 🚀

@JohannesLichtenberger
Copy link
Member

I think the user is not created anymore (downloaded and checked the log):

2022-11-05T14:14:08.6137375Z     14:14:08.594 [vert.x-eventloop-thread-1] TRACE io.vertx.ext.web.RoutingContext - Calling the  handler
2022-11-05T14:14:08.6137830Z 
2022-11-05T14:14:08.6138140Z Integration test > Testing a simple query to get the node-key of a node STANDARD_ERROR
2022-11-05T14:14:08.6138852Z     Exception in thread "vert.x-eventloop-thread-1 @coroutine#11" io.vertx.core.impl.NoStackTraceThrowable: invalid_grant: Invalid user credentials

@JohannesLichtenberger
Copy link
Member

Would be awesome if you can fix this -- thanks so far for your effort :)

Added config credentials Command
@JohannesLichtenberger
Copy link
Member

@Aditya31398 still the same error, sadly

@Aditya31398
Copy link
Author

When I try locally the kcadm.sh command gives out error connection refused. This is the problem. I'll try to fix it

@Aminmalek
Copy link
Contributor

I had this issue just one time I think Its related to how docker save the credentials and why It just happen one time, I think you need to check the ENVs another time it seems both docker and the keycloak script attempts to save the user and they have conflicts with each other.

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.

None yet

3 participants