Skip to content

Add a password_scheme field in mailbox table.#43

Closed
madmatah wants to merge 1 commit intoopensolutions:masterfrom
madmatah:store-password-scheme
Closed

Add a password_scheme field in mailbox table.#43
madmatah wants to merge 1 commit intoopensolutions:masterfrom
madmatah:store-password-scheme

Conversation

@madmatah
Copy link
Copy Markdown

@madmatah madmatah commented Jun 9, 2013

Add a password_scheme field in mailbox table.

This is useful if you want to deal with multiple password scheme. The
one you use today might be too weak in the future and you should have
the possibility to change it.

The password scheme names stored in this new field are the one
described here : http://wiki2.dovecot.org/Authentication/PasswordSchemes

To use it with Dovecot, you just have to do a query like :

SELECT CONCAT('{', password_scheme, '}', password) FROM `mailbox`
WHERE [...]

The password_scheme field will contain an empty string for password
which have been set before this change.

( this is also related to issue #32 )

This is useful if you want  to deal with multiple password scheme. The
one you use today might be too  weak in the future and you should have
the possibility to change it.

The password scheme names stored in this new field are the one
described here :
http://wiki2.dovecot.org/Authentication/PasswordSchemes

To use it with Dovecot, you just have to do a query like :
SELECT CONCAT('{', password_scheme, '}', password) FROM `mailbox`
WHERE [...]

The password_scheme  field will contain  an empty string  for password
which have been set before this change.
@barryo
Copy link
Copy Markdown
Member

barryo commented Jun 10, 2013

Thanks for the pull request.

I cannot merge it in right now as GitHub is substantially behind pur internal development of V3 which is near ready for release. We've implemented this another way in V3.

@barryo barryo closed this Jun 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants