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

Multi-WAN limited to 9 connections #6752

Closed
filippocarletti opened this issue May 31, 2023 · 4 comments
Closed

Multi-WAN limited to 9 connections #6752

filippocarletti opened this issue May 31, 2023 · 4 comments
Labels
bug A defect of the software verified All test cases were verified successfully

Comments

@filippocarletti
Copy link
Member

Due to a missing decimal to hexadecimal conversion, the maximum number of wan connections is limited to 9 instead of 15 as documented.

Steps to reproduce

  • Create 10 wan connections

Expected behavior

No errors.

Actual behavior

Error:

Checking /etc/shorewall/providers...
   ERROR: Invalid Mark or Mask value (0x100000) /etc/shorewall/providers (line 31)
@filippocarletti filippocarletti added the bug A defect of the software label May 31, 2023
@filippocarletti filippocarletti self-assigned this May 31, 2023
filippocarletti added a commit to NethServer/nethserver-firewall-base that referenced this issue May 31, 2023
Shorewall provider marks are in hexadecimal format.

NethServer/dev#6752
gsanchietti added a commit to NethServer/nethserver-firewall-base that referenced this issue May 31, 2023
@nethbot
Copy link
Member

nethbot commented May 31, 2023

in 7.9.2009/testing:

@gsanchietti
Copy link
Member

Test case

Check the issue is not reproducible

@gsanchietti gsanchietti added the testing Packages are available from testing repositories label May 31, 2023
@francio87 francio87 self-assigned this Jun 5, 2023
@francio87
Copy link
Member

Confirm, fixed.

[root@ns79-test ~]# cat /etc/shorewall/providers 

...
...
#
# 80providers_output
#
red1    1       0x10000 -       eth0    10.0.120.1      balance=1,track,persistent
red10   2       0x20000 -       eth0.18 10.1.18.1       balance=1,track,persistent
red11   3       0x30000 -       eth0.19 10.1.19.1       balance=1,track,persistent
red12   4       0x40000 -       eth0.20 10.1.20.1       balance=1,track,persistent
red13   5       0x50000 -       eth0.21 10.1.21.1       balance=1,track,persistent
red14   6       0x60000 -       eth0.22 10.1.22.1       balance=1,track,persistent
red15   7       0x70000 -       eth0.23 10.1.23.1       balance=1,track,persistent
red2    8       0x80000 -       eth0.10 10.1.10.1       balance=1,track,persistent
red3    9       0x90000 -       eth0.17 10.1.17.1       balance=1,track,persistent
red4    10      0xA0000 -       eth0.11 10.1.11.1       balance=1,track,persistent
red5    11      0xB0000 -       eth0.12 10.1.12.1       balance=1,track,persistent
red6    12      0xC0000 -       eth0.13 10.1.13.1       balance=1,track,persistent
red7    13      0xD0000 -       eth0.14 10.1.14.1       balance=1,track,persistent
red8    14      0xE0000 -       eth0.15 10.1.15.1       balance=1,track,persistent
red9    15      0xF0000 -       eth0.16 10.1.16.1       balance=1,track,persistent

@francio87 francio87 added verified All test cases were verified successfully and removed testing Packages are available from testing repositories labels Jun 6, 2023
@francio87 francio87 removed their assignment Jun 6, 2023
@nethbot
Copy link
Member

nethbot commented Jun 6, 2023

in 7.9.2009/updates:

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

No branches or pull requests

4 participants