Hi Ellie,

--On 29. März 2016 um 12:30:34 +1100 ellie timoney <el...@fastmail.com> wrote:

Ellie, can you please fix the listen statement to accept correctly
bracketed ipv6 and backport to at least 2.5 and 2.4, shouldn't be many
changes in that code.

I don't think there's anything to fix here.  The code looks fine as is,
just docs missing.  Unless we want to explicitly *not* accepted
un-bracketed "ip6-address ':' port"?  But forbidding that seems silly,
considering it requires adding code to remove functionality.

I agree.

This code hasn't changed since 2012, when it was refactored by 306099b.
It's in the same state (modulo tabs/spaces change) on 2.5 and master
branches.  The 2.4 version is significantly different, as it didn't get
the refactor (but that doesn't appear to be a problem here).

I'm scouring the thread trying to figure out if there's even a problem
being reported (other than lack of docs) and I can't see it.  It seems
to have gone like:

    Sebastian: How do I do this?
    Various: It's not in docs, but try this...
    Sebastian: I tried this and it worked

Sebastian, is there anything you tried that *didn't* work, and if so,
what happened?

The only thing I tried that didn't work was to add a IPv6 listener and to HUP the master process. The manpage for master reads (in my version):

Cyrus-master rereads its configuration file when it receives a hangup signal, SIGHUP. Services and events may be added, deleted or modified when the configuration file is reread. Any active services removed from the configuration file will be allowed to run until completion.

From that it isn't obvious that some class of changes to cyrus.conf
apparently require a restart of the service. So I'm mainly asking for documentation fixes:

• clarify the allowed IPv6 address formats
• clarify that SIGHUP isn't enough for all (which?) config changes

Thanks, Sebastian´
--
   .:.Sebastian Hagedorn - Weyertal 121 (Gebäude 133), Zimmer 2.02.:.
                .:.Regionales Rechenzentrum (RRZK).:.
  .:.Universität zu Köln / Cologne University - ✆ +49-221-470-89578.:.

Attachment: pgpsgWv3PKZYS.pgp
Description: PGP signature

----
Cyrus Home Page: http://www.cyrusimap.org/
List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
To Unsubscribe:
https://lists.andrew.cmu.edu/mailman/listinfo/info-cyrus
  • IPv6 Sebastian Hagedorn via Info-cyrus
    • Re: IPv6 Eric Luyten via Info-cyrus
    • Re: IPv6 lst_hoe02--- via Info-cyrus
      • Re: IPv6 Sebastian Hagedorn via Info-cyrus
        • Re: IPv6 Hajimu UMEMOTO via Info-cyrus
          • Re: IPv6 Sebastian Hagedorn via Info-cyrus
            • Re: IPv6 Hajimu UMEMOTO via Info-cyrus
              • Re: IPv6 Bron Gondwana via Info-cyrus
                • Re: IPv6 ellie timoney via Info-cyrus
                • Re: IPv6 Sebastian Hagedorn via Info-cyrus
                • Re: IPv6 ellie timoney via Info-cyrus
                • Re: IPv6 Sebastian Hagedorn via Info-cyrus
                • Re: IPv6 ellie timoney via Info-cyrus
                • Re: IPv6 Sebastian Hagedorn via Info-cyrus

Reply via email to