Skip to content

Make it possible to specify encrypted values in rabbitmq conf (backport #11989)#11992

Merged
michaelklishin merged 6 commits intov4.0.xfrom
mergify/bp/v4.0.x/pr-11989
Aug 14, 2024
Merged

Make it possible to specify encrypted values in rabbitmq conf (backport #11989)#11992
michaelklishin merged 6 commits intov4.0.xfrom
mergify/bp/v4.0.x/pr-11989

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify Bot commented Aug 13, 2024

This is an extension of an existent advanced.config feature:

default_user = bunnies-444
default_pass = encrypted:F/bjQkteQENB4rMUXFKdgsJEpYMXYLzBY/AmcYG83Tg8AOUwYP7Oa0Q33ooNEpK9
definitions.tls.cacertfile = /tmp/tls-gen/basic/result/ca_certificate.pem
definitions.tls.certfile   = /tmp/tls-gen/basic/result/client_sunnyside_certificate.pem
definitions.tls.keyfile    = /tmp/tls-gen/basic/result/client_sunnyside_key.pem
definitions.tls.password   = encrypted:0GhvdGjqQ/EAPqr27/PtLj04ikzC7p4IBmM6X7Vqk4fDJGrCnXf5VV4MTAOyPQV5

This is an automatic backport of pull request #11989 done by Mergify.

This makes possible to specify an encrypted
value in rabbitmq.conf using a prefix.

For example, to specify a default user password
as an encrypted value:

``` ini
default_user = bunnies-444
default_pass = encrypted:F/bjQkteQENB4rMUXFKdgsJEpYMXYLzBY/AmcYG83Tg8AOUwYP7Oa0Q33ooNEpK9
```

``` erl
[
  {rabbit, [
      {config_entry_decoder, [
             {passphrase, <<"bunnies">>}
       ]}
    ]}
].
```

(cherry picked from commit 1c7e590)
'ctl encode' is unfortunately name and targets
advanced.config commands.

This introduce a command that targets 'rabbitmq.conf'
values and has a more specific name.

Eventually 'ctl encode' will be aliased and deprecated,
although we still do not have an aliasing mechanism
and it won't be in scope for 4.0.

(cherry picked from commit c2fdd73)
(cherry picked from commit e1490c6)
@michaelklishin michaelklishin merged commit 90b8f8a into v4.0.x Aug 14, 2024
@michaelklishin michaelklishin deleted the mergify/bp/v4.0.x/pr-11989 branch August 14, 2024 01:33
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.

1 participant