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

Missing documentation on how to add authenticated users #30

Open
junjun-zhang opened this issue May 24, 2021 · 3 comments · Fixed by #31
Open

Missing documentation on how to add authenticated users #30

junjun-zhang opened this issue May 24, 2021 · 3 comments · Fixed by #31
Assignees
Labels
documentation Improvements or additions to documentation

Comments

@junjun-zhang
Copy link

When running the ceibacli login command using the GitHub PAT generated from my account, I got an error message: User junjun-zhang doesn't have permissions to access the service. Checked the ceiba code and noticed that there is a Mongo DB collection authenticated_users, however, there seems to be no documentation on how to add users to the collection.

@felipeZ felipeZ self-assigned this May 25, 2021
@felipeZ felipeZ added bug Something isn't working documentation Improvements or additions to documentation and removed bug Something isn't working labels May 25, 2021
@felipeZ
Copy link
Member

felipeZ commented May 25, 2021

You are completely right! I am going to add the missing documentation!

@junjun-zhang
Copy link
Author

junjun-zhang commented May 27, 2021

Thanks @felipeZ for the quick response, however, the problem persists even I edited the users.txt file.

Had a look at the docker compose file, based on this line:
entrypoint: ceiba -m 172.17.0.2 -u admin -p ${MONGO_PASSWORD} -f users.txt

The users.txt file has to be within the docker container, not the one from the host. Basically it would require the docker image to be rebuilt in order to include the updated users.txt in the container. Alternatively, a volume can be mount that includes this file.

@felipeZ
Copy link
Member

felipeZ commented May 27, 2021

@junjun-zhang You are right about it. I just forgot about the volume, actually, I had opened issue #16 to acknowledge that issue but I forgot about it.

@felipeZ felipeZ reopened this Jun 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants