[Freeipa-devel] [PATCH] 166+167 Zonemgr improvements

[Martin Kosek]

On Fri, 2011-11-11 at 10:22 -0500, Rob Crittenden wrote:
> Martin Kosek wrote:
> > There are 2 patches improving our zone zonemgr:
> >
> > 1) ipa-server-install --zonemgr option validation and normalization +
> > the same thing in ipa dnszone-add/mod --admin-email. They now allow and
> > correctly process '.' in a local-part of the zonemgr e-mail (it is
> > encoded as '\.'.
> >
> > How to test:
> >
> > ipa-server-install -p secret123 -a secret123 --setup-dns --zonemgr=foo.bar at example.com
> > OR if e-mail is passed in SOA format:
> > ipa-server-install -p secret123 -a secret123 --setup-dns --zonemgr='foo\.bar.example.com'
> >
> > In both cases, the zonemgr e-mail will be set to correct format: 'foo
> > \.bar.example.com'.
> >
> > 2) Our default zonemgr is changed to follow RFC 2142 recommendation -
> > hostmaster@<domain>
> >
> > hostmaster is an alias to root anyway (see /etc/aliases).
> >
> > Martin
> 
> NACK 166, this illegal address is allowed: foo\.bar\.baz\.com
> 
> ACK 167 when 166 is ready.
> 
> rob

Are you sure that you quoted the string in shell properly? It likes to
eat backslashes when one is not cautious. The zonemgr value, including
backslashes, should then show up in `ipa dnszone-show ZONE'.

This is my output:

# ipa dnszone-mod example.com --admin-email='foo\.bar\.baz\.com'
ipa: ERROR: invalid 'admin_email': address domain is not fully qualified
("example.com" instead of just "example")

Anyway, attaching a rebased patch (it collided with my patch 120).

Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mkosek-166-2-improve-zonemgr-validator-and-normalizer.patch
Type: text/x-patch
Size: 12637 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20111114/ecd49dc9/attachment.bin>