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

user_external should log error message if authentication over imap does not work #11

Closed
yennor opened this issue Feb 1, 2017 · 3 comments · Fixed by #34
Closed

user_external should log error message if authentication over imap does not work #11

yennor opened this issue Feb 1, 2017 · 3 comments · Fixed by #34
Labels
enhancement New feature or request
Milestone

Comments

@yennor
Copy link

yennor commented Feb 1, 2017

Affected apps

user_external

Expected behaviour

If user_external can not login over imap show a message what the error was!

Actual behaviour

It just shows a general purpose error message, that login was not successful.

It took me over an hour to find out, that the user_external app was properly enabled and even executed, just that the server certificate was wrong.
I managed to find that out by adding only adding a single line to imap.php

OCP\Util::writeLog('user_external', implode('|', imap_errors()), OCP\Util::ERROR);

Steps to reproduce

  1. activate user_external app
  2. change config.php to activate login over imap (but have an error in your configuration)
  3. try to login
@pmgouveia
Copy link

thanks! ^^

@MariusBluem MariusBluem transferred this issue from nextcloud/apps Dec 20, 2018
@violoncelloCH
Copy link
Member

@yennor could you create a PR implementing proper error handling?

@violoncelloCH violoncelloCH added the enhancement New feature or request label Dec 21, 2018
anojht added a commit to anojht/user_external that referenced this issue Jan 28, 2019
Prints IMAP errors and alerts to the nextcloud log.
anojht added a commit to anojht/user_external that referenced this issue Jan 28, 2019
Prints IMAP errors and alerts to the nextcloud log.

Signed-off-by: Anojh Thayaparan <anojh@hotmail.com>
@violoncelloCH violoncelloCH added this to the 0.5.1 milestone Jan 30, 2019
violoncelloCH added a commit that referenced this issue Jan 31, 2019
Implements error handling for imap errors #11
@violoncelloCH
Copy link
Member

should be fixed with #34
thanks @anojht !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants