unit.git/src, branch archive/knownConditionTrueFalse

Socket: removed useless port < 1 check.

2022-06-15T20:14:50+00:00

In src/nxt_sockaddr.c::nxt_job_sockaddr_inet_parse() there is a check
that port > 0 then there is a check that port < 1 || port > 65535, well
we _know_ it can't be less than 1.

Port: removed useless msg->cancelled == 0 checks.

2022-06-15T20:14:50+00:00

In src/nxt_port_socket.c::nxt_port_read_msg_process() msg->cancelled is
set to 0 and is not touched again.

However there are several checks for it being == 0 which are _always_
true, so remove them.

I'm assuming that this is functioning as intended and that setting
msg->cancelled to 0 is correct.

msg->cancelled was set 0 unconditionally in commit e227fc9 ("Introducing
application and port shared memory queues."), so I guess the now
redundant checks were simply missed for removal.

Var: relocated nxt_var_is_const() and nxt_var_raw().

2022-06-15T06:27:50+00:00

No functional changes.

Removing unused tracking fields and functions.

2022-06-07T05:59:45+00:00

The message tracking is unused since 1d84b9e4b459 commit.

This fixes the issue found by Coverity (CID 376263).

Router: removed unused code in nxt_router_conf_error().

2022-06-07T05:43:38+00:00

No functional changes.

Summary: Var: removing all async stuff.

2022-06-02T01:36:35+00:00

No functional changes.

HTTP: generalized uri encoding.

2022-05-19T13:18:25+00:00

No functional changes.

Node.js: fixed ES modules format in loader.mjs.

2022-06-02T10:48:27+00:00

Before Node.js v16.14.0 the "format" value in defaultResolve
was ignored so error was hidden.  For more information see:
https://github.com/nodejs/node/pull/40980

Logging a NULL pointer instead of passing it in the memcpy().

2022-06-01T15:40:34+00:00

Var: Added $request_uri (as in NGINX).

2022-05-31T10:40:02+00:00

This supports a new variable $request_uri that contains the path
and the query (See RFC 3986, section 3).  Its contents are percent
encoded.  This is useful for example to redirect HTTP to HTTPS:

{
    "return": "301",
    "location": "https://$host$request_uri"
}

When  is requested, the server
redirects to .

===

Testing:

//diff --git a/src/nxt_http_return.c b/src/nxt_http_return.c
//index 82c9156..adeb3a1 100644
//--- a/src/nxt_http_return.c
//+++ b/src/nxt_http_return.c
//@@ -196,6 +196,7 @@ nxt_http_return_send_ready(nxt_task_t *task,
    void *obj, void *data)
//         field->value = ctx->encoded.start;
//         field->value_length = ctx->encoded.length;
//     }
//+    fprintf(stderr, "ALX: target[%1$i]: <%2$.*1$s>\n",
    (int)r->target.length, r->target.start);
//
//     r->state = &nxt_http_return_send_state;
//

{
	"listeners": {
		"*:81": {
			"pass": "routes/ru"
		}
	},

	"routes": {
		"ru": [{
			"action": {
				"return": 301,
				"location": "$request_uri"
			}
		}]
	}
}

$ curl -i http://localhost:81/*foo%2Abar?baz#arg
HTTP/1.1 301 Moved Permanently
Location: /*foo%2Abar?baz
Server: Unit/1.27.0
Date: Mon, 30 May 2022 16:04:30 GMT
Content-Length: 0

$ sudo cat /usr/local/unit.log | grep ALX
ALX: target[15]: