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

Added ACKEE_ANONYMOUS Environmental Variable For Allowing Users To View The Ackee UI Without Logging In #278

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

Zozman
Copy link

@Zozman Zozman commented Jul 24, 2021

As described in Issue #276, if a user has their instance of Ackee behind a reverse proxy, then it is possible that they might want to handle authentication on the proxy level instead of on the Ackee level. Therefore this PR adds a ACKEE_ANONYMOUS variable that when set to true will automatically generate a session when opening the UI instead of prompting the user for credentials. This even makes it possible to run an instance of Ackee without a ACKEE_USERNAME and ACKEE_PASSWORD even set.

@vercel
Copy link

vercel bot commented Jul 24, 2021

Someone is attempting to deploy a commit to a Personal Account owned by @electerious on Vercel.

@electerious first needs to authorize it.

@electerious
Copy link
Owner

Thanks for the PR!

There was already a similar experiment, but we couldn't find a way to protect the UI while keeping the /api endpoint public. ackee-tracker can't track visits when the endpoint isn't public and a public endpoint without a password protection means that everyone can modify and see your data.

#243 (comment)

@Zozman
Copy link
Author

Zozman commented Jul 24, 2021

Could we mark tokens then to know if they had been generated with or without a password and then only let the /api endpoint's more sensitive operations work if the token was generated with credentials?

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