[GH-ISSUE #283] add support for MD5-CRYPT and use it as default #267

Closed
opened 2026-02-27 11:10:53 +03:00 by kerem · 3 comments
Owner

Originally created by @tonioo on GitHub (Dec 4, 2013).
Original GitHub issue: https://github.com/modoboa/modoboa/issues/283

Originally assigned to: @tonioo on GitHub.

Originally created by Walter Doekes on 2012-07-01T07:45:53Z

Hi,

  • crypt truncates the password to 8 characters before hashing
  • all other implmented schemes don't have a salt

Solution: implemented MD5-CRYPT as default password scheme (default_pass_scheme = MD5-CRYPT in dovecot).

See attached patch.

Regards,
Walter Doekes
OSSO B.V.

Originally created by @tonioo on GitHub (Dec 4, 2013). Original GitHub issue: https://github.com/modoboa/modoboa/issues/283 Originally assigned to: @tonioo on GitHub. **Originally created by Walter Doekes on 2012-07-01T07:45:53Z** Hi, - crypt truncates the password to 8 characters before hashing - all other implmented schemes don't have a salt Solution: implemented MD5-CRYPT as default password scheme (default_pass_scheme = MD5-CRYPT in dovecot). See attached patch. Regards, Walter Doekes OSSO B.V.
kerem 2026-02-27 11:10:53 +03:00
Author
Owner

@tonioo commented on GitHub (Dec 4, 2013):

Posted by Antoine Nguyen on 2012-07-01T13:51:56Z

Hi Walter,

thanks for this patch, it is indeed an interesting feature. About the md5crypt implementation you propose, I think you should copy the original header inside it before Modoboa can use it.

I haven't seen anything wrong with your code anyway, I'll certainly add it to 0.9.1.

<!-- gh-comment-id:29815675 --> @tonioo commented on GitHub (Dec 4, 2013): **Posted by Antoine Nguyen on 2012-07-01T13:51:56Z** Hi Walter, thanks for this patch, it is indeed an interesting feature. About the md5crypt implementation you propose, I think you should copy the original header inside it before Modoboa can use it. I haven't seen anything wrong with your code anyway, I'll certainly add it to 0.9.1.
Author
Owner

@tonioo commented on GitHub (Dec 4, 2013):

Posted by Walter Doekes on 2012-07-01T22:43:17Z

Antoine Nguyen wrote:

About the md5crypt implementation you propose, I think you should copy the original header inside it before Modoboa can use it.

Do you need me to do anything? I'm not quite sure what you mean.

Regards,
Walter

<!-- gh-comment-id:29815678 --> @tonioo commented on GitHub (Dec 4, 2013): **Posted by Walter Doekes on 2012-07-01T22:43:17Z** Antoine Nguyen wrote: > About the md5crypt implementation you propose, I think you should copy the original header inside it before Modoboa can use it. Do you need me to do anything? I'm not quite sure what you mean. Regards, Walter
Author
Owner

@tonioo commented on GitHub (Dec 4, 2013):

Posted by Antoine Nguyen on 2012-07-02T08:38:57Z

Walter Doekes wrote:

Do you need me to do anything? I'm not quite sure what you mean.

No, sorry. I wasn't clear enough :-)

I've just copy/paste the license header to avoid any issue in the future.

<!-- gh-comment-id:29815680 --> @tonioo commented on GitHub (Dec 4, 2013): **Posted by Antoine Nguyen on 2012-07-02T08:38:57Z** Walter Doekes wrote: > Do you need me to do anything? I'm not quite sure what you mean. No, sorry. I wasn't clear enough :-) I've just copy/paste the license header to avoid any issue in the future.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/modoboa-modoboa#267
No description provided.