V2.1.5 Diagnostics > Test port bug

[A Former User]

As per the title,

Leaving the source port blank results in a warning that the source port should either be valid or left blank. A bug in the way that field is validated perhaps?

[Gertjan]

Try this https://github.com/pfsense/pfsense/pull/1418  ;)

[phil.davis]

Just another dumb bug. Fix here: https://github.com/pfsense/pfsense/pull/1418
I also made it not forget the user-entered data when there is an input validation error. It was wiping out everything I typed, which was annoying.
I tried the same code on a 2.1.5 system and it works, so it is possible to just copy the whole new diag_testport.php onto 2.1.5 without messing about trying to back out other changes in master/RELENG_2_2.

[A Former User]

I'm always thrilled to see quick turnarounds on bugs  ;D

Thanks, will test it soon(ish), as work permits.