Skip to content

Commit 71b7764

Browse files
committed
Fetch: introduced port and protocol helper macros.
1 parent 0a80dab commit 71b7764

File tree

3 files changed

+10
-6
lines changed

3 files changed

+10
-6
lines changed

nginx/ngx_js_fetch.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -558,7 +558,7 @@ ngx_js_ext_fetch(njs_vm_t *vm, njs_value_t *args, njs_uint_t nargs,
558558
http->max_response_body_size = http->conf->max_response_body_size;
559559

560560
#if (NGX_SSL)
561-
if (u.default_port == 443) {
561+
if (ngx_js_https(&u)) {
562562
http->ssl = http->conf->ssl;
563563
http->ssl_verify = http->conf->ssl_verify;
564564
}
@@ -1425,7 +1425,7 @@ ngx_js_request_constructor(njs_vm_t *vm, ngx_js_request_t *request,
14251425

14261426
u->url.len = request->url.len;
14271427
u->url.data = request->url.data;
1428-
u->default_port = 80;
1428+
u->default_port = NGX_JS_HTTP_DEFAULT_PORT;
14291429
u->uri_part = 1;
14301430
u->no_resolve = 1;
14311431

@@ -1441,7 +1441,7 @@ ngx_js_request_constructor(njs_vm_t *vm, ngx_js_request_t *request,
14411441
{
14421442
u->url.len -= 8;
14431443
u->url.data += 8;
1444-
u->default_port = 443;
1444+
u->default_port = NGX_JS_HTTPS_DEFAULT_PORT;
14451445
#endif
14461446

14471447
} else {

nginx/ngx_js_http.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,10 @@
1212

1313

1414
#define NGX_JS_HOST_MAX_LEN 256
15+
#define NGX_JS_HTTP_DEFAULT_PORT 80
16+
#define NGX_JS_HTTPS_DEFAULT_PORT 443
17+
18+
#define ngx_js_https(u) ((u)->default_port == NGX_JS_HTTPS_DEFAULT_PORT)
1519

1620

1721
typedef struct ngx_js_http_s ngx_js_http_t;

nginx/ngx_qjs_fetch.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -280,7 +280,7 @@ ngx_qjs_ext_fetch(JSContext *cx, JSValueConst this_val, int argc,
280280
ngx_qjs_external_max_response_buffer_size(cx, external);
281281

282282
#if (NGX_SSL)
283-
if (u.default_port == 443) {
283+
if (ngx_js_https(&u)) {
284284
http->ssl = ngx_qjs_external_ssl(cx, external);
285285
http->ssl_verify = ngx_qjs_external_ssl_verify(cx, external);
286286
}
@@ -662,7 +662,7 @@ ngx_qjs_request_ctor(JSContext *cx, ngx_js_request_t *request,
662662
ngx_memzero(u, sizeof(ngx_url_t));
663663

664664
u->url = request->url;
665-
u->default_port = 80;
665+
u->default_port = NGX_JS_HTTP_DEFAULT_PORT;
666666
u->uri_part = 1;
667667
u->no_resolve = 1;
668668

@@ -678,7 +678,7 @@ ngx_qjs_request_ctor(JSContext *cx, ngx_js_request_t *request,
678678
{
679679
u->url.len -= 8;
680680
u->url.data += 8;
681-
u->default_port = 443;
681+
u->default_port = NGX_JS_HTTPS_DEFAULT_PORT;
682682
#endif
683683

684684
} else {

0 commit comments

Comments
 (0)