Bug #8531

URL Table aliases don't support FQDNs or names that return >1 IP

Added by → luckman212 about 2 years ago. Updated 6 months ago.

Aliases / Tables
Target version:
Start date:
Due date:
% Done:


Estimated time:
Affected Version:
Affected Architecture:


In my testing (pfSense 2.4.3-p1 as well as 'master') the only Alias type that supports FQDNs is "Host". This is limiting. One of the things I want to be able to do with aliases is to centrally distributing white- or black-lists containing a mix of IPs, subnets, and FQDNs. Some of these FQDNs may return multiple IPs, or a mix of IPv4 and IPv6 records, e.g.

$ dig +short A AAAA

I created a quick patch that enables URL Table aliases to return a mix of IPv4/IPv6 entries as well as accomodate multiple entries per FQDN. I did very limited testing with this but "it works for me". Before I submit a PR could someone please comment? Please go easy on me it's been a year since I've touched pfSense code...

Patch is below:


#1 Updated by → luckman212 about 2 years ago

I added timeout values to the dig command, but rather than 2 separate commits for this tiny patch, I made a new branch that has them squashed:

#2 Updated by Luiz Souza almost 2 years ago

  • Assignee set to Luiz Souza
  • Target version set to 48

#3 Updated by → luckman212 almost 2 years ago

Also, I should probably redo this patch in pure PHP and use dns_get_record() instead of shell functions to avoid the forking.

#4 Updated by Jim Pingle over 1 year ago

  • Target version changed from 48 to 2.5.0

#5 Updated by Jim Pingle 11 months ago

  • Category set to Rules / NAT

#6 Updated by Jim Pingle 11 months ago

  • Category changed from Rules / NAT to Aliases / Tables

#8 Updated by Jim Pingle 6 months ago

  • Status changed from New to Pull Request Review

#9 Updated by Renato Botelho 6 months ago

  • Status changed from Pull Request Review to Feedback
  • Target version changed from 2.5.0 to 2.4.5
  • % Done changed from 0 to 100

PR has been merged. Thanks!

#10 Updated by Viktor Gurov 6 months ago

  • Status changed from Feedback to Resolved

Renato Botelho wrote:

PR has been merged. Thanks!

tested on 2.4.5.a.20200108.1746

works as expected

Also available in: Atom PDF