[Asterisk-code-review] http.c: Add an http test server (asterisk[master])

George Joseph asteriskteam at digium.com
Thu Nov 18 08:08:09 CST 2021


Attention is currently required from: N A, Kevin Harwell.
George Joseph has posted comments on this change. ( https://gerrit.asterisk.org/c/asterisk/+/17471 )

Change subject: http.c: Add an http test server
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

File main/http.c:

https://gerrit.asterisk.org/c/asterisk/+/17471/comment/bdcc3934_09cb1f17 
PS2, Line 2133: static struct ast_tcptls_session_args http_test_args = {
              : 	.accept_fd = -1,
              : 	.master = AST_PTHREADT_NULL,
              : 	.tls_cfg = NULL,
              : 	.poll_timeout = -1,
              : 	.name = "http test server",
              : 	.accept_fn = ast_tcptls_server_root,
              : 	.worker_fn = httpd_helper_thread,
              : };
              : 
If more than one test server is started is there going to be an issue with them sharing this static structure?



-- 
To view, visit https://gerrit.asterisk.org/c/asterisk/+/17471
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Change-Id: Ic5fb5f11e62c019a1c51310f4667b32a4dae52f5
Gerrit-Change-Number: 17471
Gerrit-PatchSet: 2
Gerrit-Owner: Kevin Harwell <kharwell at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: N A <mail at interlinked.x10host.com>
Gerrit-Attention: N A <mail at interlinked.x10host.com>
Gerrit-Attention: Kevin Harwell <kharwell at digium.com>
Gerrit-Comment-Date: Thu, 18 Nov 2021 14:08:09 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20211118/48e6c36d/attachment-0001.html>


More information about the asterisk-code-review mailing list