Project

General

Profile

Bug #7601

Dynamic DNS - Hostname should not be required for DNS-O-Matic

Added by Scott Reynolds almost 3 years ago. Updated about 2 months ago.

Status:
Resolved
Priority:
Normal
Category:
Dynamic DNS
Target version:
Start date:
05/25/2017
Due date:
% Done:

100%

Estimated time:
Affected Version:
Affected Architecture:

Description

Background: DNS-O-Matic allows updates to be sent without a hostname or service specified, in which case all services are updated. However, the pfSense UI requires a valid hostname, and will not allow this case.

I hacked out all hostname validation to test the underlying update mechanism, forced an update, and verified on the DNS-O-Matic site that all services were updated. This suggests that only the UI validation needs to be updated.

services_dyndns_edit.php.diff (581 Bytes) services_dyndns_edit.php.diff Scott Reynolds, 06/05/2017 01:06 PM

Associated revisions

Revision 1ccc327f (diff)
Added by gizmotronic 3 months ago

Make hostname optional for for DNS-O-Matic.
This resolves ticket #7601.

Revision fcb61f94 (diff)
Added by gizmotronic 3 months ago

Make hostname optional for for DNS-O-Matic.
This resolves ticket #7601.

(cherry picked from commit 1ccc327f0014d74de501a066df556add28c38e78)

History

#1 Updated by Scott Reynolds over 2 years ago

Please see the attached trivial patch to make the hostname optional for DNS-O-Matic.

#2 Updated by Jim Pingle 7 months ago

  • Category set to Dynamic DNS
  • Status changed from New to Feedback

Is this still a problem? There have been lots of DynDNS changes since this report, but I don't see anything that looks like this patch in place.

If it is a problem, please make a pull request on github instead of attaching a patch.

#3 Updated by Jim Pingle 3 months ago

  • Status changed from Feedback to Pull Request Review
  • Target version set to 2.5.0

#4 Updated by Renato Botelho 3 months ago

  • Status changed from Pull Request Review to Feedback
  • Assignee set to Renato Botelho
  • % Done changed from 0 to 100

PR has been merged. Thanks!

#5 Updated by Jim Pingle 3 months ago

  • Target version changed from 2.5.0 to 2.4.5

#6 Updated by Jim Pingle about 2 months ago

  • Status changed from Feedback to Resolved

Though I cannot confirm the DNS-O-Matic service accepts the request, the GUI no longer requires the hostname.

Also available in: Atom PDF