jyapayne/Nim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Pass headers and body correctly to FetchOptions (#19884) [backport]

Browse source 
* Pass headers to FetchOptions

Don't pass body if method is HttpGet or HttpHead

* Syntax fixes

* Restart CI
This commit is contained in:
Jake Leahy 2022-06-13 16:03:40 +10:00 • committed by GitHub
commit 8fa2c0b532
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
lib/std/jsfetch.nim
View file

@ -90,8 +90,9 @@ func newfetchOptions*(metod: HttpMethod; body: cstring;
headers: Headers = newHeaders()): FetchOptions = headers: Headers = newHeaders()): FetchOptions =
## Constructor for `FetchOptions`. ## Constructor for `FetchOptions`.
result = FetchOptions( result = FetchOptions(
body: body, mode: cstring($mode), credentials: cstring($credentials), cache: cstring($cache), referrerPolicy: cstring($referrerPolicy), body: if metod notin {HttpHead, HttpGet}: body else: nil,
keepalive: keepalive, redirect: cstring($redirect), referrer: referrer, integrity: integrity, mode: cstring($mode), credentials: cstring($credentials), cache: cstring($cache), referrerPolicy: cstring($referrerPolicy),
keepalive: keepalive, redirect: cstring($redirect), referrer: referrer, integrity: integrity, headers: headers,
metod: (case metod metod: (case metod
of HttpHead: "HEAD".cstring of HttpHead: "HEAD".cstring
of HttpGet: "GET".cstring of HttpGet: "GET".cstring