opnsense / core

OPNsense GUI, API and systems backend
https://opnsense.org/
BSD 2-Clause "Simplified" License
3.35k stars 749 forks source link

Alias name check logic broken in 23.1.1 #6368

Closed andrewhotlab closed 1 year ago

andrewhotlab commented 1 year ago

Important notices

Before you add a new report, we ask you kindly to acknowledge the following:

Describe the bug

Editing or adding such a string, which is within the requirements, in the "name" field of an alias makes the GUI return this error in version 23.1.1:

2023-03-01_16-03

Previously versions (surely before 23.1) does not have this problem.

I suppose the problem lies in this line: https://github.com/opnsense/core/blob/bc1a8778ab2140e46175ecf2546b634f87087f07/src/opnsense/mvc/app/models/OPNsense/Firewall/FieldTypes/AliasNameField.php#L88

Thanks!

fichtner commented 1 year ago

https://twitter.com/opnsense/status/1628843345626779655

andrewhotlab commented 1 year ago

Thanks. In fact I realized a few minutes ago that the issue has been already solved with commit 05b48c151547 ! :)

fichtner commented 1 year ago

Sorry for the trouble. Will be in 23.1.2 next week :)