Project

General

Profile

Actions

Bug #6224

closed

Firewall NAT Edit forgets dst type selection after reporting input errors

Added by Phillip Davis almost 8 years ago. Updated about 7 years ago.

Status:
Resolved
Priority:
Low
Category:
Web Interface
Target version:
Start date:
04/21/2016
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.3.x
Affected Architecture:

Description

Example:
1) Start with a NAT port forward that has some special source type and destination type - e.g. just for fun source type LAN net, destination type This Firewall.
2) Edit the NAT port forward - change source type to Network, change destination type to Network. Do not actually input an network address.
3) Press Save
4) Input error messages are displayed (good):
The field Source address is required.
The field Destination address is required.
5) The source type remains as Network, awaiting entry of a network and mask - good.
6) The destination type becomes WAN address - problem!
If the user is not watching, they might end up leaving WAN address there because they do not realize what just happened.

I was expecting that it would either:
a) Redisplay the form with destination type Network, awaiting entry of the network and mask; or
b) Redisplay the original setting (e.g. This Firewall)

It will be something in the JavaScript which is doing different stuff for the dsttype field than for the srctype field.

Actions #1

Updated by Phillip Davis almost 8 years ago

Note: This behavior is seen with the 2.3-RELEASE code, and with subsequent changes related to https://redmine.pfsense.org/issues/6173 - so this is not directly related to the issues for 6173, it just happens that I noticed it while testing stuff related to 6173.

Actions #2

Updated by Chris Buechler over 7 years ago

  • Status changed from New to Confirmed
  • Affected Version changed from 2.3 to 2.3.x
Actions #4

Updated by Renato Botelho about 7 years ago

  • Status changed from Confirmed to Feedback
  • Assignee set to Renato Botelho
  • Target version set to 2.4.0
  • % Done changed from 0 to 100

Merged, thanks!

Actions #5

Updated by Jim Pingle about 7 years ago

  • Status changed from Feedback to Resolved

Works as expected now

Actions #6

Updated by Jim Pingle about 7 years ago

  • Target version changed from 2.4.0 to 2.3.3
Actions

Also available in: Atom PDF