net: lib: http_server: Remove POSIX_API dependency

There is no need to depend on CONFIG_POSIX_API because the HTTP server is an network internal library. Cleanup remaining POSIX configs left from earlier commits. Signed-off-by: Jukka Rissanen <[email protected]>
zephyrproject-rtos · Apr 17, 2024 · 74f9f1d · 74f9f1d
1 parent 839d8bb
commit 74f9f1d
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 4 deletions.
diff --git a/subsys/net/lib/http/Kconfig b/subsys/net/lib/http/Kconfig
@@ -40,7 +40,6 @@ config HTTP_SERVER
 	select HTTP_PARSER
 	select HTTP_PARSER_URL
 	select EXPERIMENTAL
-	select POSIX_API
 	help
 	  HTTP1 and HTTP2 server support.
 

diff --git a/subsys/net/lib/http/http_server_core.c b/subsys/net/lib/http/http_server_core.c
@@ -214,7 +214,7 @@ static int accept_new_client(int server_fd)
 
 static int close_all_sockets(struct http_server_ctx *ctx)
 {
-	close(ctx->fds[0].fd); /* close eventfd */
+	zsock_close(ctx->fds[0].fd); /* close eventfd */
 	ctx->fds[0].fd = -1;
 
 	for (int i = 1; i < ARRAY_SIZE(ctx->fds); i++) {

diff --git a/tests/net/lib/http_server/common/prj.conf b/tests/net/lib/http_server/common/prj.conf
@@ -10,4 +10,3 @@ CONFIG_HTTP_SERVER=y
 
 # Networking config
 CONFIG_NET_SOCKETS=y
-CONFIG_NET_SOCKETS_POSIX_NAMES=y
diff --git a/tests/net/lib/http_server/tls/src/main.c b/tests/net/lib/http_server/tls/src/main.c
@@ -217,7 +217,7 @@ static void test_tls(void)
 	ptr = (char *)zsock_inet_ntop(AF_INET, &sa.sin_addr, buf, sizeof(buf));
 	zassert_not_equal(ptr, NULL, "inet_ntop() failed (%d)", errno);
 
-	ret = connect(client_fd, (struct sockaddr *)&sa, sizeof(sa));
+	ret = zsock_connect(client_fd, (struct sockaddr *)&sa, sizeof(sa));
 	zassert_not_equal(ret, -1, "failed to connect (%d)", errno);
 
 	ret = zsock_send(client_fd, http1_request, sizeof(http1_request) - 1, 0);