Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

messaging and payload references do not get verified [1] #148

Open
michielbdejong opened this issue Oct 5, 2022 · 3 comments
Open

messaging and payload references do not get verified [1] #148

michielbdejong opened this issue Oct 5, 2022 · 3 comments

Comments

@michielbdejong
Copy link
Member

michielbdejong commented Oct 5, 2022

https://github.com/pondersource/peppol-php/blob/54b8850/nextcloud-app/peppolnext/lib/PonderSource/WSSec/Signature.php#L122-L124

michielbdejong pushed a commit that referenced this issue Oct 5, 2022
@michielbdejong michielbdejong added this to the milestone 3 milestone Oct 5, 2022
@michielbdejong michielbdejong changed the title messaging reference does not get verified messaging and payload references do not get verified Oct 5, 2022
@michielbdejong
Copy link
Member Author

@yasharpm any idea how to debug this? How can it be that the signature for $envelope->getBody() is OK but the ones for $envelope->getHeader()->getMessaging() and for $payload are not?

@yasharpm
Copy link
Collaborator

Envelope body is empty so canonicalization can hardly affect it. Whilst messaging has a big body. Payload should be straight forward though. Is it phase4 to peppolnext? Or peppolnext to peppolnext?

@michielbdejong
Copy link
Member Author

phase4 to peppolnext. You can reproduce it with https://github.com/pondersource/peppol-php#running-in-docker-testnet or I can show you in a screenshare session if you want

@yasharpm yasharpm changed the title messaging and payload references do not get verified messaging and payload references do not get verified [1] Nov 29, 2022
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

No branches or pull requests

2 participants