<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Thu, Jun 28, 2018 at 9:15 PM Kevin J. McCarthy <<a href="mailto:kevin@8t8.us">kevin@8t8.us</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Jun 26, 2018 at 05:02:11PM -0700, Brandon Long wrote:<br>
> Ok, this patch updates the current head to have<br>
> (imap|pop|smtp)_oauth_refresh_command, consolidates the code for executing<br>
> the commands and generating the oauthbearer sasl token, and adds support or<br>
> oauthbearer for pop.<br>
<br>
Hi Brandon, I have two comments:<br>
<br>
1) In mutt_account_getoauthbearer(), mutt_read_line() can return NULL,<br>
so check for that too.<br></blockquote><div><br></div><div>done </div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
2) In the second patch:<br>
<br>
--- a/imap/auth_oauth.c<br>
+++ b/imap/auth_oauth.c<br>
@@ -50,7 +50,7 @@ imap_auth_res_t imap_auth_oauth (IMAP_DATA* idata, const char* method)<br>
<br>
   ilen = strlen (oauthbearer) + 30;<br>
   ibuf = safe_malloc (ilen);<br>
-  snprintf (ibuf, ilen, "AUTHENTICATE OAUTHBEARER %s", oauthbearer);<br>
+  snprintf (ibuf, ilen, "AUTHENTICATE OAUTHBEARER f%s", oauthbearer);</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Was the added "f" prefix intentional?<br></blockquote><div> </div><div><span style="background-color:rgb(255,255,255);text-decoration-style:initial;text-decoration-color:initial;float:none;display:inline">No, that was when I was testing my error handling, I thought I'd removed it.</span> </div><div><br></div><div>Brandon</div></div></div>