diff options
| author | Christian Breunig <christian@breunig.cc> | 2023-11-20 10:13:21 +0100 |
|---|---|---|
| committer | Christian Breunig <christian@breunig.cc> | 2023-11-20 17:17:25 +0100 |
| commit | f5e43b1361fb59a9c260739bdb28729d5119507c (patch) | |
| tree | fcf6a04f62744cb34c7ce57b5b4c98c386f3c69e /interface-definitions/https.xml.in | |
| parent | e92667504e0c503b7c0d125d89d8795d6b6d5876 (diff) | |
| download | vyos-1x-f5e43b1361fb59a9c260739bdb28729d5119507c.tar.gz vyos-1x-f5e43b1361fb59a9c260739bdb28729d5119507c.zip | |
http: T5762: api: make API socket backend communication the one and only default
Why: Smoketests fail as they can not establish IPv6 connection to uvicorn
backend server.
https://github.com/vyos/vyos-1x/pull/2481 added a bunch of new smoketests.
While debugging those failing, it was uncovered, that uvicorn only listens on
IPv4 connections
vyos@vyos# netstat -tulnp | grep 8080
(Not all processes could be identified, non-owned process info
will not be shown, you would have to be root to see it all.)
tcp 0 0 127.0.0.1:8080 0.0.0.0:* LISTEN -
As the CLI already has an option to move the API communication from an IP to a
UNIX domain socket, the best idea is to make this the default way of
communication, as we never directly talk to the API server but rather use the
NGINX reverse proxy.
Diffstat (limited to 'interface-definitions/https.xml.in')
| -rw-r--r-- | interface-definitions/https.xml.in | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/interface-definitions/https.xml.in b/interface-definitions/https.xml.in index 5430193b5..448075b5b 100644 --- a/interface-definitions/https.xml.in +++ b/interface-definitions/https.xml.in @@ -68,7 +68,6 @@ <priority>1002</priority> </properties> <children> - #include <include/port-number.xml.i> <node name="keys"> <properties> <help>HTTP API keys</help> @@ -101,12 +100,6 @@ <hidden/> </properties> </leafNode> - <leafNode name="socket"> - <properties> - <help>Run server on Unix domain socket</help> - <valueless/> - </properties> - </leafNode> <node name="graphql"> <properties> <help>GraphQL support</help> |