Skip to content

Make methodId required in the response#2

Open
nijel wants to merge 23 commits intocuchac:masterfrom
nijel:master
Open

Make methodId required in the response#2
nijel wants to merge 23 commits intocuchac:masterfrom
nijel:master

Conversation

@nijel
Copy link
Contributor

@nijel nijel commented Nov 6, 2018

It is present there even with methodSelectionAllowed and without that
the signature check fails when it's present.

This reverts commit 50d54c5.

I'm not really sure if I'm not doing something wrong, but even with methodSelectionAllowed the return payment information does contain methodId and with current code I always got signature validation error...

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.

1 participant