Require salt in Rfq privateData JSON schema

TBD54566975 / tbdex

56 stars 25 forks source link

Require salt in Rfq privateData JSON schema #306

Closed diehuxx closed 4 months ago

diehuxx commented 4 months ago

@KendallWeihe pointed out that we should make salt a required field in Rfq.privateData because the privateData is impossible to validate -- and therefore useless -- without the salt.

TODO: validate in at least two implementations:

[ ] tbdex-js PR
[ ] tbdex-kt PR
[ ] tbdex-swift

KendallWeihe commented 4 months ago

Should salt have required set to Y in the spec README? (screen shot below)