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

RABBITMQ_ERLANG_COOKIE deprecation #7262

Closed
lukebakken opened this issue Feb 12, 2023 · 2 comments · Fixed by #7391
Closed

RABBITMQ_ERLANG_COOKIE deprecation #7262

lukebakken opened this issue Feb 12, 2023 · 2 comments · Fixed by #7391
Assignees

Comments

@lukebakken
Copy link
Collaborator

lukebakken commented Feb 12, 2023

Examples of user confusion:

In RabbitMQ Server, there is no warning when the RABBITMQ_ERLANG_COOKIE environment variable is used, but there is a warning printed when a CLI tool uses it:

https://github.com/rabbitmq/rabbitmq-server/blob/main/deps/rabbitmq_cli/lib/rabbitmq/cli/core/distribution.ex#L115-L137

Furthermore, it is not listed as deprecated by the docker image:

https://github.com/docker-library/rabbitmq/blob/master/docker-entrypoint.sh#L13-L29

Personally I think the deprecation warning should be removed and support for the existing RABBITMQ_ env vars preserved.

@dumbbell @michaelklishin ?

@michaelklishin
Copy link
Member

@lukebakken we can change the warning to day that environment variable use is not recommended over other options and keep what we currently support but not anything else beyond the already added default user credential vars. WDYT?

@tijoer
Copy link

tijoer commented Feb 13, 2023

I was using the "docker -e RABBITMQ_ERLANG_COOKIE=xxx" command mainly to setup a testing environment. The way to handle this in such a scenario is fine for me and in a production environment I would fall back to proper secrets management. However getting the warning that this feature will get removed soon is kind of a bummer, as this means that all tests WILL break soon.

PS: Thanks for your work on this :). Loving RabbitMQ so far.

lukebakken added a commit that referenced this issue Feb 22, 2023
mergify bot pushed a commit that referenced this issue Feb 23, 2023
Fixes #7262

(cherry picked from commit b0d2f94)
mergify bot pushed a commit that referenced this issue Feb 23, 2023
Fixes #7262

(cherry picked from commit b0d2f94)
(cherry picked from commit 29ba1c3)
mergify bot pushed a commit that referenced this issue Feb 23, 2023
Fixes #7262

(cherry picked from commit b0d2f94)
(cherry picked from commit 29ba1c3)
(cherry picked from commit 194db17)
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 a pull request may close this issue.

4 participants