Skip to content

Add attachment content signature transform - #60

Open
NathanBerthier wants to merge 2 commits into
benoist:masterfrom
pennylane-hq:add-attachment-content-signature-transform
Open

Add attachment content signature transform#60
NathanBerthier wants to merge 2 commits into
benoist:masterfrom
pennylane-hq:add-attachment-content-signature-transform

Conversation

@NathanBerthier

@NathanBerthier NathanBerthier commented Sep 2, 2025

Copy link
Copy Markdown

@benoist

benoist commented Sep 2, 2025

Copy link
Copy Markdown
Owner

Hi,

That link doesn't work so I can verify :)

@NathanBerthier
NathanBerthier force-pushed the add-attachment-content-signature-transform branch from 6b69a7d to 2b09307 Compare September 2, 2025 08:25
@benoist

benoist commented Sep 2, 2025

Copy link
Copy Markdown
Owner

Also do you have an example on what XML this can sign

@NathanBerthier

Copy link
Copy Markdown
Author

Also do you have an example on what XML this can sign

Hello 👋 Thank you for the quick reply 😃

I did open the PR as a draft to do some testing first, but I'll write some specs and provide example of XML later

@NathanBerthier
NathanBerthier marked this pull request as ready for review September 2, 2025 12:55
@NathanBerthier

Copy link
Copy Markdown
Author

@benoist I've added some specs and an example of unsigned xml, let me know if you need more context

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