#78 When deleting Email Account check out catchall

open
nobody
Broken (26)
7
2006-06-23
2006-06-23
Harm
No

Hi people of qmailadmin,

I discovered this bug with the following configuration:

qmailadmin 1.2.9
vpopmail 5.4.13

But I think this error is also in older releases of
qmailadmin. This is what needs to happen to repeat:

Administrator of domain 'example.com' makes an e-mail
account: 'info@example.com'. He sets this info address
to catchall.

Now the guy deletes the e-mail account
'info@example.com'. Here's the bug: the catchall still
remains. This causes vpopmail to react that there is a
system error, because the
~vpopmail/domains/xxxx/example.com/info directory
doesn't exist anymore.

Keep up the good work at Inter7!

Kind regards,
Harm van Tilborg
Tiscom Hosting B.V.

Discussion

  • Harm
    Harm
    2006-06-23

    • priority: 5 --> 7
     
  • Tom Collins
    Tom Collins
    2006-06-23

    Logged In: YES
    user_id=117662

    Any ideas on how to handle this on the UI side?

    On the "confirm delete" page, we could inform the user that they're about to
    delete a "catchall" mailbox, and present radio buttons with the following
    options: Delete user and change catchall to bounce, Delete user and change
    catchall to delete, Delete user and change catchall to (popup of accounts
    here).

    Of course, the popup will be huge for a domain with lots of email accounts.
    in that case, I image they'd want to change the catchall, and then delete the
    user.

     
  • Harm
    Harm
    2006-06-23

    Logged In: YES
    user_id=458751

    Hi Tom,

    That message on the 'confirm delete' page is fine. Make it
    'eyecatching' with a bold text or something. The radio
    button with these options looks good to me (catchall bounce,
    catchall delete, catchall to postmaster). Then also put some
    text there that they can change the catchall referer to
    another user after this screen. Well, that's not quite
    elegant, but the postmaster option is like the 'other user'
    option. However, people can change that when they have
    returned to the 'email accounts' menu.

    Or make it the hard way: you just can't delete an e-mail
    account when the catchall is turned to that one. Just
    display a text like that, that they first have to change
    their catchall account.

    That popup window is also a good idea, but as you said, with
    more then 20 users it's going to be a big window and not
    easy to choose.

    It's up to you ;].

    Btw, are you still using the CVS at sf.net?

     
  • Harm
    Harm
    2006-06-24

    Logged In: YES
    user_id=458751

    Hi Tom,

    A small addition :]. Maybe not the right place, but I would
    like to participate in redesiging the UI. Make it work with
    (X)HTML, CSS2 and some JavaScript addons (like disabling
    textfield etc.).

    Tell me if you're interested :].

     
  • Harm
    Harm
    2006-10-22

    Logged In: YES
    user_id=458751

    Maybe a status update on this. Is it easy to implement the
    hard way: just give a message that deleting a catch all
    account is impossible?

     
  • Rick Widmer
    Rick Widmer
    2007-04-18

    Logged In: YES
    user_id=939952
    Originator: NO

    Some thoughts...

    vpopmail should not allow delete of the catchall account. An attempt should return an error stating the problem.

    qmailadmin should not offer a delete icon for the catchall account. Maybe add a note in the catchall section telling why.

     
  • Rick Widmer
    Rick Widmer
    2007-05-21

    Logged In: YES
    user_id=939952
    Originator: NO

    vpopmail 5.4.19 no longer allows the catchall account to be deleted, as long as it is within the same account. It does not check other accounts, or other mail servers.

    Qmailadmin should not offer a delete option on the mail users page for the catchall user. Be sure to check for both ~vpopmail/domains/domain/user and user@domain methods of specifying the catchall account. The domain and user name both must match to hide the delete icon.