Skip to content

feat: support iptables firewall #10903

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.

Sign up for GitHub

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

Jump to bottom
Merged
ssongliu merged 16 commits into from
Nov 10, 2025
Merged

feat: support iptables firewall #10903

ssongliu merged 16 commits into from
Nov 10, 2025

Conversation

@ssongliu
Copy link
Member

@ssongliu ssongliu commented Nov 10, 2025 •
edited by wanghe-fit2cloud
Loading

@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Nov 10, 2025

Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@f2c-ci-robot
Copy link

f2c-ci-robot bot commented Nov 10, 2025

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign wanghe-fit2cloud for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ssongliu ssongliu force-pushed the pr@dev-v2@feat_new_iptables branch 2 times, most recently from 27af0db to b5d08eb Compare November 10, 2025 06:57
HynoR and others added 16 commits November 10, 2025 21:16
@HynoR @ssongliu
feat: support iptables control
4a93d93
@HynoR @ssongliu
chore: clean up
955123b
@HynoR @ssongliu
feat: improve iptables firewall
6177d40
@HynoR @ssongliu
fix
f2300ac
@HynoR @ssongliu
fix: change-mode
a8592da
@HynoR @ssongliu
fix: v1
5ae2135
@HynoR @ssongliu
fix: enhance iptables apply method
568c43b
@HynoR @ssongliu
fix: enhance iptables rule handle logic
04259e2
@HynoR @ssongliu
feat: merge iptables firewall
de6ba78
@HynoR @ssongliu
feat: add advanced controls for iptables management
e8c0dbd
@HynoR @ssongliu
fix: remove unnecessary condition for iptables advanced controls display
402672d
@HynoR @ssongliu
fix: update firewall status display logic and improve button states
a3e0c8e
@HynoR @ssongliu
refactor: simplify advanced operation handling and improve iptables c…
0575bcd 
…hain info loading
@ssongliu
feat: adjust iptables code
8dda3f0
@HynoR @ssongliu
merge: iptables (#10907)
5964299 
* fix: update iptables initialization logic to check for iptables command availability

* fix

* fix: improve iptables rule operation handling logic

* fix: add validation for unsafe DROP rules in iptables filter
@ssongliu
fix: adjust chain 1PANEL_INPUT position
b22b39d
@ssongliu ssongliu force-pushed the pr@dev-v2@feat_new_iptables branch from f60fd45 to b22b39d Compare November 10, 2025 13:16
@ssongliu ssongliu merged commit 6db79f4 into dev-v2 Nov 10, 2025
3 checks passed
@ssongliu ssongliu deleted the pr@dev-v2@feat_new_iptables branch November 10, 2025 13:17
@sonarqubecloud
Copy link

sonarqubecloud bot commented Nov 10, 2025

Quality Gate Passed Quality Gate passed

Issues
25 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
4.1% Duplication on New Code

See analysis details on SonarQube Cloud

ssongliu added a commit that referenced this pull request Nov 10, 2025
@ssongliu
Revert "feat: support iptables firewall control (#10903)"
fe0e95d 
This reverts commit 6db79f4.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wanghe-fit2cloud wanghe-fit2cloud Awaiting requested review from wanghe-fit2cloud

@zhengkunwang223 zhengkunwang223 Awaiting requested review from zhengkunwang223

Assignees

No one assigned

Labels

dev-v2 do-not-merge/release-note-label-needed

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@ssongliu @zhengkunwang223 @1Panel-bot @HynoR