[smtpd/qpsmtpd] 4f9af7: Make Redis optional

[email protected] (GitHub) Fri, 23 Jan 2015 14:22:22 -0800
Newsgroups perl.qpsmtpd.dev
Message-ID <54c2c99ea0774_239a3fed9709729c1160c0@hookshot-fe6-cp1-prd.iad.github.net.mail>
----==_mimepart_54c2c99ea0365_239a3fed9709729c11594a
Content-Type: text/plain;
 charset=UTF-8
Content-Transfer-Encoding: 7bit

  Branch: refs/heads/master
  Home:   https://github.com/smtpd/qpsmtpd
  Commit: 4f9af75e48d3a8ee3ece8f6e836858e31910ed1a
      https://github.com/smtpd/qpsmtpd/commit/4f9af75e48d3a8ee3ece8f6e836858e31910ed1a
  Author: Jared Johnson <[email protected]>
  Date:   2015-01-23 (Fri, 23 Jan 2015)

  Changed paths:
    M plugins/greylisting
    M t/plugin_tests/greylisting

  Log Message:
  -----------
  Make Redis optional

Previously the greylist plugin tried to use redis and fell back to DBM. This means that if a system already had an established DBM database, but happened to have Redis running, the existing DBM db would be abandoned for a new Redis DB. This would inevitably lead to more delays for legitimate mail, and possibly lost mail.

This adds a 'redis_server' argument which enables Redis and sets the location of the redis server; if it is not explicitly set, DBM is used instead.

If the redis server is unavailable, rather than failing to start QP, we instead fail to register the plugin.


  Commit: 462a2ae3672ba56d0c506f86cff2a2eb66e154a2
      https://github.com/smtpd/qpsmtpd/commit/462a2ae3672ba56d0c506f86cff2a2eb66e154a2
  Author: Jared Johnson <[email protected]>
  Date:   2015-01-23 (Fri, 23 Jan 2015)

  Changed paths:
    M plugins/greylisting
    M t/plugin_tests/greylisting

  Log Message:
  -----------
  Rename 'redis_server' arg to 'redis'


  Commit: 5a58e02e8025ecaf1a690bfed110d8c9a5a2436a
      https://github.com/smtpd/qpsmtpd/commit/5a58e02e8025ecaf1a690bfed110d8c9a5a2436a
  Author: Jared Johnson <[email protected]>
  Date:   2015-01-23 (Fri, 23 Jan 2015)

  Changed paths:
    M plugins/greylisting

  Log Message:
  -----------
  Scarier error message for redis failures


  Commit: 3ed568f9d482886e9c310c2e51c012193744b795
      https://github.com/smtpd/qpsmtpd/commit/3ed568f9d482886e9c310c2e51c012193744b795
  Author: Matt Simerson <[email protected]>
  Date:   2015-01-23 (Fri, 23 Jan 2015)

  Changed paths:
    M plugins/greylisting
    M t/plugin_tests/greylisting

  Log Message:
  -----------
  Merge pull request #202 from jaredj/greylist-storage-opts

Make Redis optional


Compare: https://github.com/smtpd/qpsmtpd/compare/012866ef973f...3ed568f9d482
----==_mimepart_54c2c99ea0365_239a3fed9709729c11594a--