[dovecot/core] 3ee009: lib-storage: Fix mailbox list index notifications ...

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/dovecot/core
  Commit: 3ee009bb150bd1db1c5064c3285afc7573fb4572
      https://github.com/dovecot/core/commit/3ee009bb150bd1db1c5064c3285afc7573fb4572
  Author: Timo Sirainen <timo.sirainen at dovecot.fi>
  Date:   2017-05-19 (Fri, 19 May 2017)

  Changed paths:
    M src/lib-storage/list/mailbox-list-index-notify.c

  Log Message:
  -----------
  lib-storage: Fix mailbox list index notifications when modseqs aren't enabled.

No notifications were sent if highest_modseqs weren't updated in mailbox
list index.


  Commit: 3b86acdd4194c5bb44d73244435d19d8b5665c8d
      https://github.com/dovecot/core/commit/3b86acdd4194c5bb44d73244435d19d8b5665c8d
  Author: Timo Sirainen <timo.sirainen at dovecot.fi>
  Date:   2017-05-19 (Fri, 19 May 2017)

  Changed paths:
    M src/lib-storage/list/mailbox-list-index-status.c

  Log Message:
  -----------
  lib-storage: Always update highestmodseq in mailbox list index

This is required for mailbox-list-notify API to work correctly. For
example IMAP NOTIFY session with CONDSTORE enabled could be listening for
FlagChanges, but another session without CONDSTORE enabled could be updating
flags in a mailbox that has never been opened with CONDSTORE enabled.
NOTIFY wouldn't see this change, unless it was always written to
dovecot.list.index.


  Commit: a384b1fccb51884e8afe0f571420961851a2149d
      https://github.com/dovecot/core/commit/a384b1fccb51884e8afe0f571420961851a2149d
  Author: Timo Sirainen <timo.sirainen at dovecot.fi>
  Date:   2017-05-19 (Fri, 19 May 2017)

  Changed paths:
    M src/lib-storage/index/imapc/imapc-list.c
    M src/lib-storage/index/shared/shared-list.c
    M src/lib-storage/list/mailbox-list-fs.c
    M src/lib-storage/list/mailbox-list-index-backend.c
    M src/lib-storage/list/mailbox-list-maildir.c
    M src/lib-storage/list/mailbox-list-none.c

  Log Message:
  -----------
  lib-storage: Explicitly name function pointers in mailbox lists

This allows adding more fields to struct mailbox_list_vfuncs without
compiler warnings.


  Commit: 58a89627905e3590381cdd5eb931b9537c4b4ea6
      https://github.com/dovecot/core/commit/58a89627905e3590381cdd5eb931b9537c4b4ea6
  Author: Timo Sirainen <timo.sirainen at dovecot.fi>
  Date:   2017-05-19 (Fri, 19 May 2017)

  Changed paths:
    M src/lib-storage/list/mailbox-list-index-notify.c
    M src/lib-storage/list/mailbox-list-index.c
    M src/lib-storage/list/mailbox-list-index.h
    M src/lib-storage/mailbox-list-notify.c
    M src/lib-storage/mailbox-list-notify.h
    M src/lib-storage/mailbox-list-private.h

  Log Message:
  -----------
  lib-storage: Implement mailbox_list_notify_flush()

This flushes any pending notifications. For now it only checks if there's
a pending notification waiting for a timeout. It could also stat() the
files to make sure, but that's probably unnecessary.


  Commit: 369847496787678ee18608754a97f0c2df80a36f
      https://github.com/dovecot/core/commit/369847496787678ee18608754a97f0c2df80a36f
  Author: Timo Sirainen <timo.sirainen at dovecot.fi>
  Date:   2017-05-19 (Fri, 19 May 2017)

  Changed paths:
    M src/imap/cmd-noop.c
    M src/imap/imap-notify.c
    M src/imap/imap-notify.h

  Log Message:
  -----------
  imap NOTIFY: Flush any pending notifications on NOOP

This is mainly to allow testing NOTIFY easily with imaptest scripts.
This is cheap anyway, because mailbox_list_notify_flush() doesn't do any
syscalls when there's no work.


Compare: https://github.com/dovecot/core/compare/e031d9aaae59...369847496787