Skip to content

Cuttlefish 3.5.0 #14364

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

Merged
merged 1 commit into from
Aug 11, 2025
Merged

Cuttlefish 3.5.0 #14364

merged 1 commit into from
Aug 11, 2025

Conversation

michaelklishin
Copy link
Collaborator

@michaelklishin michaelklishin commented Aug 11, 2025

This version forces prefixed binaries
(such as encrypted:TkQbjiVWtUJw3Ed/hkJ5JIsFIyhruKII6uKPXogfvDyMXGH1qQK3hVqshFolLN0S) to have alphanumeric prefixes ([a-zA-Z0-9_]+).

This allows us to tell a generated password value
with a colon from an tagged binary, for example:

# this is a not an encrypted value, or, in Cuttlefish speak,
# not a tagged one, so parse it has a "regular" password (binary)
default_password = 'abd:def07df783'
# this is an encrypted/tagged value,
# so parse it as a tagged tuple,
# {encrypted, <<"GhC4J5lh2DUkbdyKO0aMI8aYJ54mwe4eEWzou4yRFAHMF82IbD6cRiYAiBa8UIzR">>)
default_pass = encrypted:GhC4J5lh2DUkbdyKO0aMI8aYJ54mwe4eEWzou4yRFAHMF82IbD6cRiYAiBa8UIzR

If a value of, say, default_pass or ssl_options.password cannot be parsed as a tagged value, it will be parsed as a regular binary, because rabbit.schema
specifies multiple types as supported.

References #14233.

This version forces prefixed binaries
(such as encrypted:TkQbjiVWtUJw3Ed/hkJ5JIsFIyhruKII6uKPXogfvDyMXGH1qQK3hVqshFolLN0S)
to have alphanumeric prefixes ([a-zA-Z0-9_]+).

This allows us to tell a generated password value
with a colon from an tagged binary.

If a value of, say, default_pass or ssl_options.password
cannot be parsed as a tagged value, it will be
parsed as a regular binary, because rabbit.schema
specifies multiple types as supported.

References #14233.
@michaelklishin michaelklishin added this to the 4.2.0 milestone Aug 11, 2025
@mergify mergify bot added the make label Aug 11, 2025
@michaelklishin michaelklishin merged commit 5f69116 into main Aug 11, 2025
283 checks passed
@michaelklishin michaelklishin deleted the mk-bump-cuttlefish branch August 11, 2025 18:15
michaelklishin added a commit that referenced this pull request Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant