Skip to content

Deprecated PKCS8 and PasswordBasedMACAlgorithms cleanup#724

Open
bsriramprasad wants to merge 1 commit intodevelopmentfrom
deprecated-security-features-cleanup
Open

Deprecated PKCS8 and PasswordBasedMACAlgorithms cleanup#724
bsriramprasad wants to merge 1 commit intodevelopmentfrom
deprecated-security-features-cleanup

Conversation

@bsriramprasad
Copy link
Copy Markdown
Contributor

@bsriramprasad bsriramprasad commented Feb 13, 2026

  • Feature missed to be marked deprecated, though its related feature and associated interfaces are deprecated.
    • PKCS8RSAKeyPairUpload- capability deprecated
    • UploadKeyPairInPKCS8 - interface deprecated
    • PKCS8 capability - NOT marked deprecated

-PasswordBasedMACAlgorithms

  • Not marked deprecated in WSDL
  • Marked deprecated in spec

Copy link
Copy Markdown
Member

@HansBusch HansBusch left a comment

Choose a reason for hiding this comment

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

In the specification annex I suggest to write the original capability text in the table. Stating deprecated doesn't make much sense as the title of the annex already tells that this is about deprecated capabilities.

@bsriramprasad
Copy link
Copy Markdown
Contributor Author

bsriramprasad commented Mar 31, 2026

@HansBusch

In the specification annex I suggest to write the original capability text in the table.

Ref: Security.xml

  • What is the original capability text to include? I thought PasswordBasedMACAlgorithms OR PKCS8 itself are the original capabilities and the remarks column for those are empty that I try to fill in?

Copy link
Copy Markdown
Contributor

@kieran242 kieran242 left a comment

Choose a reason for hiding this comment

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

@bsriramprasad I would have approved these changes that I have reviewed.
I will await the required changes from @HansBusch before approving.

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.

5 participants