marlam / mpop

POP3 client
https://marlam.de/mpop
GNU General Public License v3.0
13 stars 1 forks source link

Microsoft broke my config #14

Closed cowingtonpost1 closed 3 months ago

cowingtonpost1 commented 3 months ago

My microsoft email setup using mutt_oauth2.py and the xoauth auth method in mpop worked up until July 6th, when it broke. mpop: POP3 server does not support authentication method XOAUTH2

I have tried letting mpop automatically select an auth method but it rejects my auth token. I don't know how to configure it to make microsoft auth work.

marlam commented 3 months ago

Try mpop --serverinfo to check which methods the POP3 server supports now. Maybe OAUTHBEARER is one of them, and maybe it works with the same token you used before.

cowingtonpost1 commented 3 months ago

The server only supports user and plain. I have tried both with my same auth token and username and it does not work.

michaelcadilhac commented 3 months ago

This is due to this commit: https://git.marlam.de/gitweb/?p=mpop.git;a=commitdiff;h=40feacd396e95e3e878ea423ac2305c98c95b5a2

The condition for "end of token" is not correct. It should read:

           if (string[i + token_len] == ' '
                    || string[i + token_len] == '\r'
                    || string[i + token_len] == '\0') /* valid end of token */
marlam commented 3 months ago

You are right, I messed up the "end of token" condition. Thank you very much for debugging this.

I wrote this code originally for msmtp, where the \r\n at the end of the server reply is always removed before token_in_string() is called, so that the function works correctly there.

I applied your change (commit 9063e68), and will release version 1.4.20 now to get this important bug fix out.

Sorry for the trouble this has caused.

michaelcadilhac commented 3 months ago

Thanks for the prompt response Martin! Cheers!

cowingtonpost1 commented 3 months ago

Thanks for fixing it