Project

General

Profile

Actions
Copy link

Bug #3900

closed
FD RB

DynamicDNS should allow "@" at hostname

Bug #3900: DynamicDNS should allow "@" at hostname

Added by F. D.Castel almost 12 years ago. Updated over 10 years ago.

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

100%

Estimated time:
Plus Target Version:
Release Notes:
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

FD Updated by F. D.Castel almost 12 years ago Actions
Copy link
 #1

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

FD Updated by F. D.Castel almost 12 years ago Actions
Copy link
 #2

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

TB Updated by Tiago Barrionuevo about 11 years ago Actions
Copy link
 #3

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

PD Updated by Phillip Davis about 11 years ago Actions
Copy link
 #4

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.

KB Updated by Kill Bill almost 11 years ago Actions
Copy link
 #5

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)

RB Updated by Renato Botelho over 10 years ago Actions
Copy link
 #6

  • 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!

CB Updated by Chris Buechler over 10 years ago Actions
Copy link
 #7

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

Also available in: Atom