Project

General

Profile

Bug #9361

Cloudflare Not Allowing "*" Hostname Entry in Dynamic DNS

Added by Will Rutherford over 1 year ago. Updated 9 months ago.

Status:
Resolved
Priority:
Normal
Category:
Dynamic DNS
Target version:
Start date:
03/01/2019
Due date:
% Done:

100%

Estimated time:
Affected Version:
2.4.4_2
Affected Architecture:
amd64

Description

Cloudflare allows wildcard A records and the pfSense DDNS page has a wildcard checkbox (since 2.3?), but it will throw an "invalid hostname" error if you provide a "*". This can be circumvented with a direct edit on the config xml, but it's certainly not ideal.

History

#1 Updated by Tom Embt over 1 year ago

My comments about Route53 on #9053 likely also apply to Cloudflare here. Assuming so, the solution would differ slightly because Cloudflare is a "split domain" type.

#2 Updated by Tom Embt over 1 year ago

https://github.com/pfsense/pfsense/pull/4053

- hide wildcard and MX checkboxes since neither are used by the Cloudflare code
- update the call to is_domain() to have a truthy second argument, allowing wildcards for cloudflare and cloudflare-v6

Unlike the recent related Route53 and Linode edits, i have not tested this change.

#3 Updated by Renato Botelho over 1 year ago

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

PR has been merged. Thanks!

#4 Updated by Jim Pingle 10 months ago

  • Target version changed from 2.5.0 to 2.4.5

#5 Updated by Tom Embt 9 months ago

Testing in a current 2.4.5 snapshot. I don't have a Cloudflare account to see if the update happens, but the pfSense UI does now allow wildcard hostnames to be submitted and saved.

#6 Updated by Jim Pingle 9 months ago

That is likely a good enough test, but I'll hold this open for a bit yet in case someone can do a functional test against an active CloudFlare account.

Thanks for testing!

#7 Updated by Jim Pingle 9 months ago

  • Status changed from Feedback to Resolved

Also available in: Atom PDF