marrow / mailer

A light-weight, modular, message representation and mail delivery framework for Python.
MIT License
276 stars 62 forks source link

Traceback from configuration handler #45

Closed plaes closed 8 years ago

plaes commented 12 years ago

Trying to use following configuration:

from marrow.mailer import Mailer
mailer = Mailer({'transport': {'use': 'mbox', 'file': '/tmp/komasu.mbox'}})

Gives me following traceback:

Traceback (most recent call last):
  File "x.py", line 2, in <module>
    mailer = Mailer({'transport': {'use': 'mbox', 'file': '/tmp/komasu.mbox'}})
  File "/home/plaes/.virtualenvs/komasu/lib/python2.7/site-packages/marrow/mailer/__init__.py", line 50, in __init__
    self.manager_config = manager_config = Bunch.partial('manager', config)
  File "/home/plaes/.virtualenvs/komasu/lib/python2.7/site-packages/marrow/util/bunch.py", line 48, in partial
    raise ValueError()
ValueError

When I add the missing manager: {} to the configuration, it complains about missing message

plaes commented 12 years ago

I also have the changes from #35 present, so no idea what's going wrong.

plaes commented 12 years ago

Following commit fixes it https://github.com/plaes/marrow.mailer/commit/56990184f525e9df9658574120466a8e6d89cbd5

domenkozar commented 12 years ago

I'm getting the same error, could this be fixed? :) Happens when you do:

Mailer({}, prefix="mail.")
domenkozar commented 12 years ago

Basically, just needs a release :)

amcgregor commented 8 years ago

Should be fixed.