Skip to content

Comments

Add SCEP ChallengePassword to EncryptionBlock#137

Merged
rene merged 2 commits intolf-edge:mainfrom
milan-zededa:scep-challenge-password
Feb 16, 2026
Merged

Add SCEP ChallengePassword to EncryptionBlock#137
rene merged 2 commits intolf-edge:mainfrom
milan-zededa:scep-challenge-password

Conversation

@milan-zededa
Copy link
Contributor

Commit 35ae108 added support for certificate enrollment using the SCEP protocol.
SCEP profiles can optionally include a ChallengePassword for one-time
or shared-secret authorization during certificate enrollment.
Since this is a sensitive value, it must be encrypted inside the CipherBlock.

This commit fixes an oversight: the ChallengePassword was not included in
EncryptionBlock (the proto message encrypted in CipherBlock.CipherData).

Commit 35ae108 added support for certificate enrollment using the SCEP protocol.
SCEP profiles can optionally include a ChallengePassword for one-time
or shared-secret authorization during certificate enrollment.
Since this is a sensitive value, it must be encrypted inside the CipherBlock.

This commit fixes an oversight: the ChallengePassword was not included in
EncryptionBlock (the proto message encrypted in CipherBlock.CipherData).

Signed-off-by: Milan Lenco <milan@zededa.com>
No functional changes; generated code and assets only.

Signed-off-by: Milan Lenco <milan@zededa.com>
@milan-zededa milan-zededa force-pushed the scep-challenge-password branch from 0a5aecd to cddf7d1 Compare February 11, 2026 17:16
@milan-zededa
Copy link
Contributor Author

@uncleDecart Could you please approve this small PR? I need to merge this to move forward with EVE and cloud implementation.

@uncleDecart
Copy link
Member

@milan-zededa , apologies, slipped through my fingers, although I don't think my approve would give you ability to merge it 😢

@milan-zededa
Copy link
Contributor Author

@milan-zededa , apologies, slipped through my fingers, although I don't think my approve would give you ability to merge it 😢

True, even with our powers combined we are still not strong enough to press that merge button :((

@uncleDecart
Copy link
Member

@milan-zededa , apologies, slipped through my fingers, although I don't think my approve would give you ability to merge it 😢

True, even with our powers combined we are still not strong enough to press that merge button :((

We need superpowers, @eriknordmark, could you help us?

@rene rene merged commit 2d9b92e into lf-edge:main Feb 16, 2026
4 checks passed
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.

3 participants