search

mguessan / davmail

DavMail POP/IMAP/SMTP/Caldav/Carddav/LDAP Exchange and Office 365 Gateway - Synced with main subversion repository at
http://davmail.sourceforge.net
GNU General Public License v2.0
580 stars 86 forks source link

Changed authentication link log level #236

Open CameronDevine opened 2 years ago

CameronDevine commented 2 years ago

Changed authentication link logging statements to warn from debug so they will not be missed if the log level has not been increased.

When I was trying to get the O365Modern mode working I was confused because I couldn't find the link that the document specifies should exist in the logs to complete the login process. I eventually figured out that I needed to increase the log level, but that took a while. To help others in the future I believe the URL should be visible at a lower log level.

jdchristensen commented 10 months ago

For future readers of this PR, note that the level was changed from debug to warn, not the other way around. Maybe @CameronDevine can edit the original message to correct this.

CameronDevine commented 10 months ago

@jdchristensen rereading my original pull request message I believe it is correct.

jdchristensen commented 10 months ago

@CameronDevine Sorry, you're right, I just misread it.

CameronDevine commented 10 months ago

@jdchristensen no problem

esabol commented 10 months ago

I think @mguessan committed this change, so it should be in the most recently released version (6.2.1). Please confirm and then close this issue if so, @CameronDevine.