fixup: disallow P2SH scripts in outputs

P2SH outputs have larger dust requirements than native SegWit (and
Taproot) scripts; we disallow the creation of them on funding txs as a
they're problematic for relay when allowing dust limits below 546 sats
(see lightning#894)

Suggested-by: @t-bast

Author: niftynei

02-peer-protocol.md

@@ -269,7 +269,7 @@ The sending node:
 
 The receiving node:
   - MUST add the specified output to the transaction
-  - MUST accept P2SH, P2WSH, P2WPKH, P2PKH, P2TR `script`s
+  - MUST accept P2WSH, P2WPKH, P2PKH, P2TR `script`s
   - MAY fail the negotiation if `script` is non-standard
   - MUST fail the negotiation if:
     - the `serial_id` is already included in the transaction