OPNsense
  • Home
  • Help
  • Search
  • Login
  • Register

  • OPNsense Forum »
  • Archive »
  • 17.1 Legacy Series »
  • Trailing whitespace
« previous next »
  • Print
Pages: [1]

Author Topic: Trailing whitespace  (Read 2509 times)

speedy

  • Newbie
  • *
  • Posts: 3
  • Karma: 0
    • View Profile
Trailing whitespace
« on: July 16, 2017, 03:34:34 pm »
Hello everyone.

I was having a problem with one of my firewall aliases. The cause was accidental trailing whitespace. However, this was not obvious from the user interface, so it took me some time to figure out where I had made the error.

I don't think I'm the only newbie who might make a mistake like this, so maybe the user interface could somehow warn the user that a particular input field has trailing whitespace?
« Last Edit: July 16, 2017, 03:47:38 pm by speedy »
Logged

franco

  • Administrator
  • Hero Member
  • *****
  • Posts: 13936
  • Karma: 1208
    • View Profile
Re: Trailing whitespace
« Reply #1 on: July 16, 2017, 06:36:30 pm »
Hi speedy,

Which field? Some fields have validation for that reason, some fields can't have validation because they are being fed arbitrary strings including whitespace / UTF-8, etc.


Cheers,
Franco
Logged

speedy

  • Newbie
  • *
  • Posts: 3
  • Karma: 0
    • View Profile
Re: Trailing whitespace
« Reply #2 on: July 16, 2017, 08:36:59 pm »
Spamhaus drop URL table had accidentally two trailing spaces.
like this: https://www.spamhaus.org/drop/drop.txt 
correct value is: https://www.spamhaus.org/drop/drop.txt

It failed silently, and it was not instantly obvious that I had trailing white space.
Logged

speedy

  • Newbie
  • *
  • Posts: 3
  • Karma: 0
    • View Profile
Re: Trailing whitespace
« Reply #3 on: July 16, 2017, 08:46:33 pm »
Maybe whitespace could be somehow made visible? With color for example?
Logged

fabian

  • Hero Member
  • *****
  • Posts: 2768
  • Karma: 199
  • OPNsense Contributor (Language, VPN, Proxy, etc.)
    • View Profile
    • Personal Homepage
Re: Trailing whitespace
« Reply #4 on: July 16, 2017, 09:14:23 pm »
in this case a whitespace character is forbidden. It can be removed without issues because it would be converted to a "+" if it would be used.
Logged

  • Print
Pages: [1]
« previous next »
  • OPNsense Forum »
  • Archive »
  • 17.1 Legacy Series »
  • Trailing whitespace
 

OPNsense is an OSS project © Deciso B.V. 2015 - 2023 All rights reserved
  • SMF 2.0.19 | SMF © 2021, Simple Machines
    Privacy Policy
    | XHTML | RSS | WAP2