Project

General

Profile

Bug #3213

Error creating more than 30 limiters

Added by Ivan Morgun about 5 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Ermal Luçi
Category:
Traffic Shaper
Target version:
Start date:
09/18/2013
Due date:
% Done:

100%

Estimated time:
Affected Version:
All
Affected Architecture:
i386

Description

When trying to create a 31st limiter I get an error message:
"The following input errors were detected: At least one bw specification is necessary"
Although I set the bandwidth specification in the form.

2013-09-18_110224.png (21.8 KB) 2013-09-18_110224.png Form wint input data Ivan Morgun, 09/18/2013 03:04 AM
2013-09-18_110252.png (9.87 KB) 2013-09-18_110252.png Error messege Ivan Morgun, 09/18/2013 03:04 AM
30-limiters-config.txt (10.6 KB) 30-limiters-config.txt Chris Buechler, 10/09/2014 12:24 AM

Associated revisions

Revision e02ea742 (diff)
Added by Ermal Luçi about 4 years ago

Fixes #3213. Allow up to 2900 limiters. This was set to 30 since limiters are to be controlled by mask and not created manually!

History

#1 Updated by Paul G over 4 years ago

I am seeing this exact same error and would love to see an update allowing more than 30 limiters.

#2 Updated by Chris Buechler about 4 years ago

  • Assignee set to Ermal Luçi
  • Target version set to 2.2

Thinking this is trivially easy to fix, just a matter of fixing the input validation it appears.

Ermal, any technical reason it can't go > 30, something more than an input validation bug here?

#3 Updated by Chris Buechler about 4 years ago

config snippet with 30 limiters attached if you want to just paste something in ready to test (add one after pasting this in and it'll fail as described).

#4 Updated by Ermal Luçi about 4 years ago

Bumped to 2900. The only reason left to 30 was for speed ones.
Normally having 30 limiters is a bit.... unusual since mask does all your settings for you!

#5 Updated by Ermal Luçi about 4 years ago

  • Status changed from New to Feedback

#6 Updated by Ermal Luçi about 4 years ago

  • % Done changed from 0 to 100

#7 Updated by Chris Buechler about 4 years ago

  • Status changed from Feedback to Resolved

that'll suffice

Also available in: Atom PDF