Project

General

Profile

Actions

Bug #6224

closed
PD RB

Firewall NAT Edit forgets dst type selection after reporting input errors

Bug #6224: Firewall NAT Edit forgets dst type selection after reporting input errors

Added by Phillip Davis over 10 years ago. Updated over 9 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.

PD Updated by Phillip Davis over 10 years ago Actions #1

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.

CB Updated by Chris Buechler over 10 years ago Actions #2

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

RB Updated by Renato Botelho almost 10 years ago Actions #4

  • 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!

JP Updated by Jim Pingle over 9 years ago Actions #5

  • Status changed from Feedback to Resolved

Works as expected now

JP Updated by Jim Pingle over 9 years ago Actions #6

  • Target version changed from 2.4.0 to 2.3.3
Actions

Also available in: Atom