Project

General

Profile

Actions

Bug #3117

closed

Services: Dynamic DNS client: leading and trailing spaces in hostname field breaks the DDNS functionality

Added by Willy Tenner over 8 years ago. Updated over 8 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
Dynamic DNS
Target version:
Start date:
07/30/2013
Due date:
% Done:

100%

Estimated time:
Plus Target Version:
Release Notes:
Affected Version:
2.0.x
Affected Architecture:

Description

I just added another DDNS provider and was wondering about the error message in the system log:

php: /services_dyndns_edit.php: phpDynDNS: (Error) Hostname supplied does not exist.

Again I looked into the "Dynamic DNS client" dialog and could not see any mistakes. So I clicked again into the hostname field and found, uups, a leading space. Bingo!

So here is my request: Please trim hostname fields so all leading and trailing whitespaces will be removed automatically.

Kind regards,
routerfreak

Actions #1

Updated by Phillip Davis over 8 years ago

There could be a general change to automatically trim all GUI input fields where it is invalid to have white space at the beginning or end of the string. That would be a bit of work to go through the whole system and decide which things to trim on input.

Actions #2

Updated by Renato Botelho over 8 years ago

  • Status changed from New to Feedback
  • % Done changed from 0 to 100
Actions #4

Updated by Renato Botelho over 8 years ago

Phillip Davis wrote:

There could be a general change to automatically trim all GUI input fields where it is invalid to have white space at the beginning or end of the string. That would be a bit of work to go through the whole system and decide which things to trim on input.

This can be useful for other places but it's not enough here, we need to make sure it's a valid domain name.

Actions #5

Updated by Renato Botelho over 8 years ago

  • Category set to Dynamic DNS
Actions #7

Updated by Chris Buechler over 8 years ago

  • Status changed from Feedback to Resolved
Actions

Also available in: Atom PDF