Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added password driver for Mail-in-a-Box #7824

Merged
merged 2 commits into from
Jan 16, 2021

Conversation

ashtokalo
Copy link
Contributor

The driver allows to change password for email through API of Mail-in-a-Box service (https://mailinabox.email)

@alecpl
Copy link
Member

alecpl commented Jan 11, 2021

  1. All new code should use Guzzle HTTP Client instead of curl.
  2. Is there no way to change password using the current user credentials instead of the admin credentials?
  3. For drivers like this maybe a better option would be to use the universal driver as proposed in Add httpapi password change driver to connect to generic HTTP/HTTPS APIs #7439.

@ashtokalo
Copy link
Contributor Author

1. All new code should use Guzzle HTTP Client instead of curl.

Done

2. Is there no way to change password using the current user credentials instead of the admin credentials?

Unfortunately, no :(

3. For drivers like this maybe a better option would be to use the universal driver as proposed in #7439.

It's up to you. But I don't believe to universal solutions. Finally they become too complex and too buggy.

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