Skip to content

fix(portmap): correct masquerading source address handling#1235

Merged
squeed merged 1 commit intocontainernetworking:mainfrom
l1b0k:fix/portmap
Apr 13, 2026
Merged

fix(portmap): correct masquerading source address handling#1235
squeed merged 1 commit intocontainernetworking:mainfrom
l1b0k:fix/portmap

Conversation

@l1b0k
Copy link
Copy Markdown
Contributor

@l1b0k l1b0k commented Feb 26, 2026

This commit fixes the portmap plugin's masquerading source address handling in the nftables backend. #1234

@l1b0k l1b0k force-pushed the fix/portmap branch 2 times, most recently from 050feaa to 0c169e8 Compare February 26, 2026 02:36
@squeed
Copy link
Copy Markdown
Member

squeed commented Apr 13, 2026

@danwinship would you mind giving this a sanity check?

Copy link
Copy Markdown
Member

@squeed squeed left a comment

Choose a reason for hiding this comment

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

Nice fix, thanks.

- Use 0.0.0.0/0 or ::/0 as source address when MasqAll is true for full traffic match

Signed-off-by: l1b0k <libokang.lbk@alibaba-inc.com>
@squeed squeed enabled auto-merge (rebase) April 13, 2026 14:43
@squeed squeed merged commit 3d27709 into containernetworking:main Apr 13, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants