Skip to content

Fix flow leak in NAT64 snat#702

Merged
guvenc merged 1 commit intomainfrom
fix/nat64_flow_leak
Aug 6, 2025
Merged

Fix flow leak in NAT64 snat#702
guvenc merged 1 commit intomainfrom
fix/nat64_flow_leak

Conversation

@PlagueCZ
Copy link
Copy Markdown
Contributor

I simply moved the (needed) code after the deletion of the original flow.

Fixes #701

@PlagueCZ PlagueCZ requested a review from a team as a code owner July 29, 2025 15:47
@github-actions github-actions bot added bug Something isn't working size/S labels Jul 29, 2025
@PlagueCZ PlagueCZ self-assigned this Jul 29, 2025
Copy link
Copy Markdown
Contributor

@guvenc guvenc left a comment

Choose a reason for hiding this comment

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

Thanks

@guvenc
Copy link
Copy Markdown
Contributor

guvenc commented Aug 6, 2025

@PlagueCZ
Could you please rebase to main ? I have accidentally merged a wrong PR which I reverted but this one now will need a rebase to main.

@PlagueCZ PlagueCZ force-pushed the fix/nat64_flow_leak branch from ff75fe0 to 6a46bae Compare August 6, 2025 10:32
@guvenc guvenc merged commit 6a46bae into main Aug 6, 2025
5 of 6 checks passed
@guvenc guvenc deleted the fix/nat64_flow_leak branch August 6, 2025 11:02
@github-project-automation github-project-automation bot moved this to Done in Roadmap Aug 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/networking bug Something isn't working size/S

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

NAT64 flow leak

3 participants