Skip to content

Accept smaller key files#570

Merged
thomasnordquist merged 1 commit into
thomasnordquist:masterfrom
PawelDefee:patch-1
Aug 11, 2021
Merged

Accept smaller key files#570
thomasnordquist merged 1 commit into
thomasnordquist:masterfrom
PawelDefee:patch-1

Conversation

@PawelDefee

Copy link
Copy Markdown
Contributor

I can generate a valid key file for my Mosquitto setup that fails the minimum size check of 128, for example this kind of key:

-----BEGIN PRIVATE KEY-----
MC4CAQAwBQYDK2VwBCIEIFXWXY9yVJRKhZRfLz/oaAcjmOzX/2El+QRU+/1Npyfe
-----END PRIVATE KEY-----

I can generate a valid key file for my Mosquitto setup that fails the minimum size check of 128, for example this kind of key:

-----BEGIN PRIVATE KEY-----
MC4CAQAwBQYDK2VwBCIEIFXWXY9yVJRKhZRfLz/oaAcjmOzX/2El+QRU+/1Npyfe
-----END PRIVATE KEY-----
@PawelDefee

PawelDefee commented Jun 30, 2021

Copy link
Copy Markdown
Contributor Author

To clarify, this is a ed25519 (elliptic cryptography) private key file. Those are very short compared to asymmetric keys even though they are considered more secure.

@thomasnordquist thomasnordquist left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done!

@thomasnordquist thomasnordquist merged commit e4add31 into thomasnordquist:master Aug 11, 2021
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