Project

General

Profile

Actions

Bug #11566

closed

Firewall Maximum Table Entries "default size" is whatever is entered

Added by Steve Y about 3 years ago. Updated 4 months ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
Web Interface
Target version:
Start date:
02/27/2021
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
23.09.1
Release Notes:
Default
Affected Version:
Affected Architecture:
All

Description

On at least 2.4.5 and 2.5, Firewall Maximum Table Entries has text "On this system the default size is: 2000000." Whatever number is entered in the field becomes the default. To replicate:
  1. Change 2000000 to 2000001 or any other number
  2. Click Save
  3. Note the "default" number does not change
  4. Click "Firewall & NAT" tab to reload the page
  5. Note the default is the number you entered in step 1

(tried to search for this here but couldn't find it)


Files

clipboard-202311181832-dsopt.png (11.4 KB) clipboard-202311181832-dsopt.png Jordan G, 11/19/2023 12:32 AM
Actions #1

Updated by Steve Y 4 months ago

Since it came up in forum, just noting this is still an issue on 23.05.1. (haven't updated any to 23.09 yet since it just came out)

Actions #2

Updated by Marcos M 4 months ago

The code simply reports the current value rather than keeping track of the default value (which is accurate only when no value is provided). I'm not sure it's worth the extra complexity of keeping track of the default value separately.

Actions #3

Updated by Steve Y 4 months ago

It may not be. :) And actually per https://docs.netgate.com/pfsense/en/latest/config/advanced-firewall-nat.html#firewall-maximum-table-entries the default is a fixed number of 400,000, thus not dependent "on this system" hardware at all. (unless that was trying to say, "on this pfSense version"?) Seems like options are to remove the whole sentence and let the doc page indicate the number, or change the text to list the actual default.

Actions #4

Updated by Marcos M 4 months ago

  • Status changed from New to Pull Request Review
  • Assignee set to Marcos M
  • Target version set to 2.8.0
  • Plus Target Version set to 24.03
  • Release Notes set to Default
  • Affected Architecture All added
Actions #6

Updated by Marcos M 4 months ago

  • Status changed from Pull Request Review to Feedback
  • % Done changed from 0 to 100
Actions #7

Updated by Jordan G 4 months ago

applying this patch on 23.09 removes the "default" size listing below the entry field

clearing the field and saving populates it with the value from 'step 1' in op

Actions #8

Updated by Steve Y 4 months ago

A quick look at the code, I see a few of:

-['placeholder' => pfsense_default_table_entries_size()]
+['placeholder' => pfsense_current_table_entries_size()]

In lieu of rehashing the discussion of using the default, seems like the "leave this blank for the default" text should be removed, and/or not allow it to be blank?

I don't easily have a new/empty install to play with but is pfsense_current_table_entries_size() = 400000 if no value is set?

Actions #9

Updated by Marcos M 4 months ago

  • Status changed from Feedback to Resolved

We don't set a defined value by default - it's whatever the OS reports (which has its own defaults).

Actions #10

Updated by Jim Pingle 4 months ago

  • Target version changed from 2.8.0 to 2.7.2
  • Plus Target Version changed from 24.03 to 23.09.1
Actions

Also available in: Atom PDF