Skip to content

Commit

Permalink
Finalize firewalld port forwarding support
Browse files Browse the repository at this point in the history
There are two major changes here.

Firstly, this adds proper support for port forwarding from
localhost via a new policy accepting traffic from HOST. This is
the last bit we were missing from the original port-forwarding
implementation.

Secondly, this fixes a bug where we generated incorrect rules
when port-forwarding from a single IP. Instead of doing standard
port-forwarding rules, those need rich rules. This was reported
as containers#881.

There are also some small code cleanups in how we handle setting
up and tearing down port forwarding. It's still rather ugly, but
at least a little better than it was before.

Fixes containers#881

Signed-off-by: Matthew Heon <[email protected]>
  • Loading branch information
mheon committed Jan 15, 2024
1 parent 5366e8f commit 0d30930
Show file tree
Hide file tree
Showing 2 changed files with 484 additions and 196 deletions.
Loading

0 comments on commit 0d30930

Please sign in to comment.