Is it allowed to create a GET request with body? about fetch

[annevk]

No, I think we should amend step 27 to also run its substeps when temporaryBody is non-null, agreed?

(How do you want to be acknowledged? See https://fetch.spec.whatwg.org/#acknowledgments for options, but anything goes.)

[yutakahirano]

Looks good.

Please write as "平野裕 (Yutaka Hirano)". Thanks!

[yutakahirano]

Oh, I didn't notice extracting init's body doesn't work when it's null (and temporaryBody is non-null). How about the following?

(27) If init's body member is present:

1. Let stream and Content-Type be the result of extracting init's body member.
2. Set temporaryBody to stream.
3. If Content-Type is non-null and r's request's header list contains no header named Content-Type, append Content-Type/Content-Type to r's Headers object. Rethrow any exception.

(28) If temporaryBody is non-null and request's method is GET or HEAD, throw a TypeError.

[annevk]

Good point. However, to avoid impacting any passed in readable streams, I opted for reversing those two steps and adding a conditional to your step 28.

[annevk]

(The idea is to make failure as side effect free as it can be.)

[wanderview]

Nice catch! Gecko bug: https://bugzilla.mozilla.org/show_bug.cgi?id=1154268