KRATool: Update error message to include keysize 168 - #5415
Conversation
Updates the error message for source payload wrapping key size validation to accurately reflect that 168 is an accepted value (for DES3/DESede only). The error message now lists all valid values (128, 168, 192, 256) with a note explaining that 168 is DES3-specific and 192 is recommended for better PKCS#11 compatibility. This addresses reviewer feedback that the previous error message was misleading when operators entered a valid DES3 keysize of 168. Assisted-by: Claude DOGTAG-4555
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
This "part 2" PR has been pre-approved by @jmagne |
Updates the error message for source payload wrapping key size validation to accurately reflect that 168 is an accepted value (for DES3/DESede only). The error message now lists all valid values (128, 168, 192, 256) with a note explaining that 168 is DES3-specific and 192 is recommended for better PKCS#11 compatibility.
This addresses reviewer feedback that the previous error message was misleading when operators entered a valid DES3 keysize of 168.
Assisted-by: Claude
DOGTAG-4555