[PATCH] services: dnsmasq: Support the --address flag.

DoneSubmitted by Pierre Langlois.
Details
2 participants
  • 宋文武
  • Pierre Langlois
Owner
unassigned
Severity
normal
P
P
Pierre Langlois wrote on 10 Apr 2020 19:59
Download
(address . guix-patches@gnu.org)
87wo6np6hh.fsf@gmx.com
Download
Hello Guix!
Here's a patch that allows passing the --address flag multiple times todnsmasq. WDYT?
Thanks,Pierre
Download
From 407265e5d9b0f863a8b14d74f6240ed102f9a907 Mon Sep 17 00:00:00 2001From: Pierre Langlois <pierre.langlois@gmx.com>Date: Fri, 10 Apr 2020 17:00:47 +0100Subject: [PATCH] services: dnsmasq: Support the --address flag.
Introduce a new `addresses' field that translates to passing `--address='multiple times to dnsmasq.
* gnu/services/dns.scm (<dnsmasq-configuration>): Add an addresses field.(dnsmasq-shepherd-service): Match the addresses field and translate it tomultiple '--address=' flags.* doc/guix.texi (DNS Services): Document it.--- doc/guix.texi | 20 ++++++++++++++++++++ gnu/services/dns.scm | 7 ++++++- 2 files changed, 26 insertions(+), 1 deletion(-)
Toggle diff (79 lines)diff --git a/doc/guix.texi b/doc/guix.texiindex 891e2693f6..7ca4bd6ef3 100644--- a/doc/guix.texi+++ b/doc/guix.texi@@ -77,6 +77,7 @@ Copyright @copyright{} 2020 Jakub Kądziołka@* Copyright @copyright{} 2020 Jack Hill@* Copyright @copyright{} 2020 Naga Malleswari@* Copyright @copyright{} 2020 Brice Waegeneire@*+Copyright @copyright{} 2020 Pierre Langlois@* Permission is granted to copy, distribute and/or modify this document under the terms of the GNU Free Documentation License, Version 1.3 or@@ -21772,6 +21773,25 @@ When true, don't read @var{resolv-file}. @item @code{servers} (default: @code{'()}) Specify IP address of upstream servers directly. +@item @code{addresses} (default: @code{'()})+For each entry, specify an IP address to return for any host in the+given domains. Queries in the domains are never forwarded and always+replied to with the specified IP address.++This is useful for redirecting hosts locally, for example:++@lisp+(service dnsmasq-service-type+ (dnsmasq-configuration+ (addresses+ '(; Redirect to a local web-server.+ "/example.org/127.0.0.1"+ ; Redirect subdomain to a specific IP.+ "/subdomain.example.org/192.168.1.42"))))+@end lisp++Note that rules in @file{/etc/hosts} take precedence over this.+ @item @code{cache-size} (default: @code{150}) Set the size of dnsmasq's cache. Setting the cache size to zero disables caching.diff --git a/gnu/services/dns.scm b/gnu/services/dns.scmindex a07946d085..478c837d13 100644--- a/gnu/services/dns.scm+++ b/gnu/services/dns.scm@@ -1,6 +1,7 @@ ;;; GNU Guix --- Functional package management for GNU ;;; Copyright © 2017 Julien Lepiller <julien@lepiller.eu> ;;; Copyright © 2018 Oleg Pykhalov <go.wigust@gmail.com>+;;; Copyright © 2020 Pierre Langlois <pierre.langlois@gmx.com> ;;; ;;; This file is part of GNU Guix. ;;;@@ -748,6 +749,8 @@ cache.size = 100 * MB (default #f)) ;boolean (servers dnsmasq-configuration-servers (default '())) ;list of string+ (addresses dnsmasq-configuration-addresses+ (default '())) ;list of string (cache-size dnsmasq-configuration-cache-size (default 150)) ;integer (negative-cache? dnsmasq-configuration-negative-cache?@@ -759,7 +762,7 @@ cache.size = 100 * MB no-hosts? port local-service? listen-addresses resolv-file no-resolv? servers- cache-size negative-cache?)+ addresses cache-size negative-cache?) (shepherd-service (provision '(dnsmasq)) (requirement '(networking))@@ -783,6 +786,8 @@ cache.size = 100 * MB '()) #$@(map (cut format #f "--server=~a" <>) servers)+ #$@(map (cut format #f "--address=~a" <>)+ addresses) #$(format #f "--cache-size=~a" cache-size) #$@(if negative-cache? '()-- 2.26.0
P
P
Pierre Langlois wrote on 22 Apr 2020 20:16
Download
(address . 40545@debbugs.gnu.org)
87zhb3per7.fsf@gmx.com
Download
Hi all,
Pierre Langlois writes:
Toggle quote (5 lines)> Hello Guix!>> Here's a patch that allows passing the --address flag multiple times to> dnsmasq. WDYT?
Any comment on this patch? Let me know if I should attempt at addingsome service tests for dnsmasq, that might be useful.
Thanks,Pierre
宋
宋
宋文武 wrote on 2 May 2020 07:04
Download
(name . Pierre Langlois)(address . pierre.langlois@gmx.com)(address . 40545-done@debbugs.gnu.org)
87bln76i79.fsf@member.fsf.org
Download
Pierre Langlois <pierre.langlois@gmx.com> writes:
Toggle quote (9 lines)> Hi all,>> Pierre Langlois writes:>>> Hello Guix!>>>> Here's a patch that allows passing the --address flag multiple times to>> dnsmasq. WDYT?
Hello Pierre, this patch looks good to me, and I pushed it to masternow, thank you!
Toggle quote (4 lines)>> Any comment on this patch? Let me know if I should attempt at adding> some service tests for dnsmasq, that might be useful.
Yes, I think adding a system test for dnsmasq is welcomed too :-)
Closed
?
Your comment

Commenting via the web interface is currently disabled.

To comment on this conversation send email to 40545@debbugs.gnu.org