commit c2e6e962b582ed3c22c9d6511f96d651411ad53f
Author: Oswald Buddenhagen <[email protected]>
Date: Tue Jan 5 19:44:13 2021 +0100
tune SASL-related comments
- add explanations to the callbacks
- remove bogus comment - EXTERNAL can be in fact missing (when no
authentication id is set)
src/drv_imap.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/src/drv_imap.c b/src/drv_imap.c
index 8c93bda..1f5a7e5 100644
--- a/src/drv_imap.c
+++ b/src/drv_imap.c
@@ -2115,8 +2115,8 @@ process_sasl_interact( sasl_interact_t *interact,
imap_server_conf_t *srvc )
switch (interact->id) {
case SASL_CB_LIST_END:
return 0;
- case SASL_CB_USER:
- case SASL_CB_AUTHNAME:
+ case SASL_CB_USER: // aka authorization id - who to act as
+ case SASL_CB_AUTHNAME: // who is really logging in
val = ensure_user( srvc );
break;
case SASL_CB_PASS:
@@ -2366,7 +2366,7 @@ imap_open_store_authenticate2( imap_store_t *ctx )
return;
notsasl:
if (!ctx->sasl || sasl_listmech( ctx->sasl, NULL, "", " ", "",
&saslavail, NULL, NULL ) != SASL_OK)
- saslavail = "(none)"; /* EXTERNAL is always there
anyway. */
+ saslavail = "(none)";
if (!auth_login) {
error( "IMAP error: selected SASL mechanism(s) not
available;\n"
" selected:%s\n available: %s\n", saslmechs,
saslavail );
_______________________________________________
isync-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/isync-devel
