Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: ip rule is not added when only binding to wan #399

Merged
merged 2 commits into from
Jan 2, 2024
Merged

Conversation

mzz2017
Copy link
Contributor

@mzz2017 mzz2017 commented Jan 1, 2024

Background

Since #383 added the mark-based tproxy support on wan interfaces, we should also add ip rule when binding WAN interfaces.

Checklist

Full Changelogs

  • [Implement ...]

Issue Reference

Closes #[issue number]

Test Result

@mzz2017 mzz2017 requested review from a team as code owners January 1, 2024 17:24
@mzz2017 mzz2017 changed the title fix: ip rule is not added when binding wan fix: ip rule is not added when only binding wan Jan 1, 2024
@mzz2017 mzz2017 changed the title fix: ip rule is not added when only binding wan fix: ip rule is not added when only binding to wan Jan 1, 2024
Copy link
Member

@jschwinger233 jschwinger233 left a comment

Choose a reason for hiding this comment

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

Thanks!

Copy link
Contributor

@sumire88 sumire88 left a comment

Choose a reason for hiding this comment

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

Thanks for the prompt reaction to the bugs detected in the previous release!

@sumire88
Copy link
Contributor

sumire88 commented Jan 2, 2024

@mzz2017 Please proceed.

@mzz2017 mzz2017 merged commit d1d0f6d into main Jan 2, 2024
28 checks passed
@mzz2017 mzz2017 deleted the mzz/fix_wan branch January 2, 2024 12:02
Copy link
Contributor

@dae-prow dae-prow bot left a comment

Choose a reason for hiding this comment

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

🧪 Since the PR has been fully tested, please consider merging it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants