Skip to content

libnetwork: expose proxy information in FirewallInfo - #52321

Merged
thaJeztah merged 1 commit into
moby:masterfrom
AkihiroSuda:api-userland-proxy
Apr 25, 2026
Merged

thaJeztah merged 1 commit into
moby:masterfrom
AkihiroSuda:api-userland-proxy

Conversation

@AkihiroSuda

@AkihiroSuda AkihiroSuda commented Apr 7, 2026 •

Copy link
Copy Markdown
Member

- What I did
Exposed userland proxy info in/info` API.

This will help users analyze whether they are using the proxy.

Preparation for:

- How I did it
Updated the API and the implementation.

- How to verify it

$ docker info
[...]
Firewall Backend: iptables
  EnableUserlandProxy: true
  UserlandProxyPath: /usr/local/bin/docker-proxy

- Human readable description for the release notes

Expose diagnostic data about userland proxy in `docker info`.

- A picture of a cute animal (not mandatory but encouraged)

@AkihiroSuda
AkihiroSuda force-pushed the api-userland-proxy branch 2 times, most recently from b7714df to 8fa4370 Compare April 7, 2026 04:34
@AkihiroSuda AkihiroSuda added the kind/enhancement Enhancements are not bugs or new features but can improve usability or performance. label Apr 7, 2026
@AkihiroSuda
AkihiroSuda force-pushed the api-userland-proxy branch 4 times, most recently from 4f3fef8 to 705a1ef Compare April 7, 2026 13:14
@AkihiroSuda
AkihiroSuda requested review from robmry and thaJeztah April 10, 2026 09:44
Comment thread api/types/system/info.go Outdated
@AkihiroSuda AkihiroSuda changed the title api: bump to 1.55; expose UserlandProxy and UserlandProxyPath in /info API ibnetwork: expose proxy information in FirewallInfo Apr 20, 2026
@vvoland vvoland modified the milestone: 29.5.0 Apr 20, 2026
@vvoland vvoland changed the title ibnetwork: expose proxy information in FirewallInfo libnetwork: expose proxy information in FirewallInfo Apr 20, 2026

@vvoland vvoland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@thaJeztah thaJeztah left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AkihiroSuda

Copy link
Copy Markdown
Member Author

Rebased

Signed-off-by: Akihiro Suda <akihiro.suda.cz@hco.ntt.co.jp>
@thaJeztah
thaJeztah merged commit de6c4d6 into moby:master Apr 25, 2026
289 of 291 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/daemon Core Engine impact/api impact/changelog kind/enhancement Enhancements are not bugs or new features but can improve usability or performance.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants