rust-payjoin
rust-payjoin copied to clipboard
Consider dropping `MAX_CONTENT_LENGTH` checks
since body is already allocated, and presumably does not exceed the app's limits by the time this code is run, maybe it's better to just remove
MAX_CONTENT_LENGTHaltogether?
Originally posted by @nothingmuch in https://github.com/payjoin/rust-payjoin/pull/745#discussion_r2135585926
also this https://github.com/payjoin/rust-payjoin/pull/745#discussion_r2136628205
remaining TODOs per @nothingmuch :
- (DONE) making the receiver's request body handling stricter
- the same change to the sender
- enforcing limits on payjoin-cli receiver, which currently uses body.collect()
- same thing for payjoin-cli sender, which uses &response.bytes() (from reqwest)