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

Firewall: Interface label not shown in WAN rules #6190

Closed
andre8244 opened this issue Jun 3, 2020 · 4 comments
Closed

Firewall: Interface label not shown in WAN rules #6190

andre8244 opened this issue Jun 3, 2020 · 4 comments
Labels
verified All test cases were verified successfully

Comments

@andre8244
Copy link
Contributor

In case of multiple red interfaces, it is possible to conditionally select which one to use.
At the bottom of Firewall -> WAN page the user can define this kind of rules.
Currently these rules don't show the user-defined label of the interface to use; they just display interface name, e.g. red2, red3...

Proposed solution

  • If the user has assigned a label for the interface of a rule, let's show it instead of interface name
  • Since a label can't be assigned to PPoE interfaces, let's show PPoE as label for this type of interfaces

Components

nethserver-firewall-base

@andre8244 andre8244 self-assigned this Jun 3, 2020
andre8244 added a commit to andre8244/nethserver-firewall-base that referenced this issue Jun 3, 2020
andre8244 added a commit to andre8244/nethserver-firewall-base that referenced this issue Jun 3, 2020
andre8244 added a commit to NethServer/nethserver-firewall-base that referenced this issue Jun 3, 2020
- Show interface label in WAN rules, if present
- Show PPPoE for PPPoE interfaces

NethServer/dev#6190
@andre8244 andre8244 removed their assignment Jun 3, 2020
@andre8244 andre8244 added the testing Packages are available from testing repositories label Jun 3, 2020
@nethbot
Copy link
Member

nethbot commented Jun 3, 2020

in 7.8.2003/testing:

@andre8244
Copy link
Contributor Author

Test case

  • Use a server where you can configure multiple red interfaces (preferably 3)
  • Access System -> Network on Cockpit
  • Configure an ethernet red interface and assign it a label
  • Configure a PPPoE red interface
  • Configure another ethernet red interface without assigning it a label
  • Go to Firewall -> WAN page
  • In Configuration section, verify that PPPoE is displayed as a label for PPPoE interface (at the right of ppp0 (redX) )
  • Click Create policy route button and create multiple rules, one for each red interface
  • In Rules section, verify the text label at the left of each draggable handle:
    • For ethernet interfaces with a label assigned, it should display the label
    • For ethernet interfaces without a label, it should display REDX (where X is a number)
    • For PPPoE interfaces, ith should display PPPoE

@cotosso
Copy link

cotosso commented Jun 3, 2020

Everything works as expected.
Firewall_fw1_test_dave

@cotosso cotosso added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Jun 3, 2020
@nethbot
Copy link
Member

nethbot commented Jun 4, 2020

in 7.8.2003/updates:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
verified All test cases were verified successfully
Projects
None yet
Development

No branches or pull requests

4 participants