Project

General

Profile

Bug #5099

firewall_rules_edit.php - Destination Port options are hidden on page load with TCP or UDP

Added by Jim Pingle almost 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Target version:
Start date:
09/03/2015
Due date:
% Done:

100%

Estimated time:
Spent time:

Description

When creating a new rule, or editing an existing TCP or UDP rule, the destination ports are hidden even though the rule defaults to TCP.

Changing the rule to some other value and then back to TCP or UDP shows the destination port controls as expected.

Associated revisions

Revision 580f2151 (diff)
Added by Steve Beaver almost 4 years ago

Fixed #5099
Corrected destination port group class

Revision b2236853 (diff)
Added by Steve Beaver almost 4 years ago

Fixed #5099
Javascript logic changged for source port display
"Advanced" button now has toggle acton and its state survives a protocol change.

Revision d0613e4f (diff)
Added by Steve Beaver almost 4 years ago

Fixed #5099
Destination port fields now behave as required.

History

#1 Updated by Steve Beaver almost 4 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Steve Beaver to Jim Pingle

Corrected destination port group class

#2 Updated by Steve Beaver almost 4 years ago

  • % Done changed from 0 to 100

#3 Updated by Jim Pingle almost 4 years ago

  • Status changed from Feedback to Confirmed
  • Assignee changed from Jim Pingle to Steve Beaver

The destination ports are appearing now, but so are the source ports, which should remain hidden.

  • New rule, defaults to TCP - OK, source ports hidden, destination ports shown
  • New rule, change to UDP or TCP/UDP - Advanced button still shows but source ports appear (Bad), destination ports are shown (Good)
  • Edit existing rules TCP, UDP, or TCP/UDP - OK on initial page load, source ports hidden, destination ports shown. Switch rule type to any other of TCP, UDP, or TCP/UDP and the same bad behavior appears: Advanced button still shows but source ports appear (Bad), destination ports are shown (Good)
Source ports should only appear in two conditions:
  • User manually clicks advanced
  • Source port already contains a value.

#4 Updated by Steve Beaver almost 4 years ago

  • Status changed from Confirmed to Feedback
  • Assignee changed from Steve Beaver to Chris Buechler

#6 Updated by Jim Pingle almost 4 years ago

  • Status changed from Feedback to Assigned
  • Assignee changed from Chris Buechler to Steve Beaver

No change in behavior from my previous test with the last commit in place.

#7 Updated by Steve Beaver almost 4 years ago

  • Status changed from Assigned to Feedback
  • Assignee changed from Steve Beaver to Jim Pingle

Javascript logic changged for source port display "Advanced" button now has toggle acton and its state survives a protocol change.

#9 Updated by Jim Pingle almost 4 years ago

  • Status changed from Feedback to Assigned
  • Assignee changed from Jim Pingle to Steve Beaver

The source port behavior is better, but the destination ports have gone missing.

So two problems now:
  • If Source ports already contain a value, they should be shown without having to click Advanced
  • Destination ports need to be restored, they are not displayed at all on new rules or rules with existing destination ports. They should always be shown for TCP, UDP, and TCP/UDP.

#10 Updated by Steve Beaver almost 4 years ago

First item is covered in bug #5045 so I'd like to continue to keep it separate and not just for this page.

Glad the source ports are now good. I'll do something similar to the dest ports.

#11 Updated by Steve Beaver almost 4 years ago

  • Status changed from Assigned to Feedback
  • Assignee changed from Steve Beaver to Jim Pingle

Destination ports now behave as required.

If Source ports already contain a value, they should be shown without having to click Advanced will be dealt with via Bug #5045

#13 Updated by Jim Pingle almost 4 years ago

  • Status changed from Feedback to Resolved

Looks good now, thanks!

Also available in: Atom PDF