Project

General

Profile

Bug #3900

DynamicDNS should allow "@" at hostname

Added by F. D.Castel about 5 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Very Low
Category:
Dynamic DNS
Target version:
Start date:
09/28/2014
Due date:
% Done:

100%

Estimated time:
Affected Version:
All
Affected Architecture:

Description

No-ip has the ability to update one of several IP addresses of a round-robin DNS A record. This is done adding a "@<label>" suffix at hostname. Eg.

  myhost@mydomain.no-ip.com@label1
  myhost@mydomain.no-ip.com@label2

However, trying to add such hostname in services_dyndns_edit.php results in the following error:

  The following input errors were detected:

  The Hostname contains invalid characters.

Confirmed on pfSense 2.1.5

History

#1 Updated by F. D.Castel about 5 years ago

BTW, where these configs are stored? Could I change them directly via Diagnostics / Edit File? It would be a good workaround, for now.

#2 Updated by F. D.Castel about 5 years ago

Sent a pull request to fix this annoyance. ( https://github.com/pfsense/pfsense/pull/1315 )

Forum discussion at: https://forum.pfsense.org/index.php?topic=83163.0

#3 Updated by Tiago Barrionuevo over 4 years ago

This won't be fixed?

There's a pull request / commit,

https://github.com/pfsense/pfsense/commit/6c163f6a881c96fc0cddb59cba6e13f5c619d3d8

but in pfSense 2.2.4 this is still not working.

Thanks in advance

#4 Updated by Phillip Davis over 4 years ago

The pull request had issues with branches and merging and...
https://github.com/pfsense/pfsense/pull/1315

It seems that the 1-line change was never resubmitted cleanly and so it never got committed. If someone looked at the current code in the repo, made the needed change to it, and submitted a pull request then it would be looked at.

#5 Updated by Kill Bill about 4 years ago

https://github.com/pfsense/pfsense/pull/1839
+ updated variant that just stops wasting time with unwanted sorts of hostname validations: https://github.com/pfsense/pfsense/pull/1914 (also backported as PR for the above 1839 RELENG_2_2 patch)

#6 Updated by Renato Botelho almost 4 years ago

  • Status changed from New to Feedback
  • Assignee set to Renato Botelho
  • Target version set to 2.3
  • % Done changed from 0 to 100

Pull request has been merged, thanks!

#7 Updated by Chris Buechler almost 4 years ago

  • Status changed from Feedback to Resolved
  • Affected Version changed from 2.1.x to All

Also available in: Atom PDF