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 ENV option for authentication and authorization #41

Closed
wants to merge 3 commits into from

Conversation

eintopf
Copy link

@eintopf eintopf commented Nov 19, 2015

I added options for authentication / authorization (via CASSANDRA_* ENV).

@eintopf
Copy link
Author

eintopf commented Nov 19, 2015

I don't get why CI is failing? What is the issue?

@tianon
Copy link
Member

tianon commented Jan 22, 2016

I think this is probably going to lead down a similar path as the discussion in #39 (comment) -- namely, a lot of authenticators are likely to require even more configuration, so it's probably easier for users who need a custom authenticator to create their own configuration file with the necessary bits, as described in the comment thread I've linked.

@tianon
Copy link
Member

tianon commented Oct 3, 2018

I'm sorry, but honestly if we're going to do anything more with environment variables it would likely be in the opposite direction (which we won't do due to the backwards compatibility implications) -- modifying a YAML file via shell script is very fragile, and setting complex configuration in this way is not great. For custom configuration, the only/best recommendation is to provide an appropriate cassandra.yaml file to your running instance (via a new image with FROM / COPY in a short Dockerfile, via Docker Configs, via bind mount, etc).

Closing in favor of #160 + docker-library/docs#1317.

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

2 participants