Parsing BIP21 incorrectly

[BullishNode]

Green apparently doesn't parse BIP21 correctly, considering payjoin data to be invalid address instead of stripping it. Reported by a BULL wallet user.

[ram0verflow]

I had this same issue with blockstream app. Is payjoin send something you'd consider, or is it out of scope? Right now pj gets dropped and the invoice pays as a normal transaction.
If it's welcome, can I take this up? I'd just need a steer on how much has to live in gdk, since the PSBT round-trip can't happen purely in the app.

[ardier16]

@BullishNode thanks for the report, and apologies for the delayed response!

The app now accepts BIP21 payment requests containing an optional pj parameter. Since Payjoin is not currently supported, the parameter is ignored and the payment proceeds as a normal on-chain transaction.

Full Payjoin support is tracked separately in #241. Regarding the offer to contribute, development is currently handled internally, so we do not have a pull-request pipeline for this work.

We’ll close this parsing issue as fixed.