Fetch: fixed setting of Content-Type header.

Previously, Content-Type was set unconditionally to
"text/plain;charset=UTF-8" when fetch request contained a string body.
This may overlap with user's Content-Type.

The fix is to set the header only if it was not set before.

This fixes #654 issue on Github.

1 files changed, 21 insertions, 0 deletions

diff --git a/nginx/t/js_fetch_objects.t b/nginx/t/js_fetch_objects.t
index edc3be57..c8001823 100644
--- a/nginx/t/js_fetch_objects.t
+++ b/nginx/t/js_fetch_objects.t
@@ -373,6 +373,27 @@ $t->write_file('test.js', <<EOF);
                 var body = await r.text();
                 return `\${body}: \${r.headers.get('Content-Type')}`;
              }, 'ABC: text/plain;charset=UTF-8'],
+            ['user content type', async () => {
+                var r = new Request("http://nginx.org",
+                                    {body: 'ABC',
+                                     headers: {'Content-Type': 'text/html'}});
+                return r.headers.get('Content-Type');
+             }, 'text/html'],
+            ['user content type from Headers()', async () => {
+                var h = new Headers();
+                h.append('Content-Type', 'text/html');
+                var r = new Request("http://nginx.org",
+                                    {body: 'ABC', headers: h});
+                return r.headers.get('Content-Type');
+             }, 'text/html'],
+            ['user content type deleted', async () => {
+                var h = new Headers();
+                h.append('Content-Type', 'text/html');
+                h.delete('Content-Type');
+                var r = new Request("http://nginx.org",
+                                    {body: 'ABC', headers: h});
+                return r.headers.get('Content-Type');
+             }, 'text/plain;charset=UTF-8'],
             ['GET body', () => {
                 try {
                     var r = new Request("http://nginx.org", {body: 'ABC'});