Stephan Bosch via Postfix-devel:
>
> Op 2-11-2023 om 15:22 schreef Wietse Venema:
> > Stephan Bosch via Postfix-devel:
> >> Looks like Postfix [...] somehow uses the data from the previous CONT auth
> >> service
> >> response as the reason.
> > Does this patch address the problem? It resets any previous Dovecot
> > auth service response before parsing the next Dovecot auth server
> > response.
> >
> > Wietse
>
> It does get rid of the base64 mess:
>
> 18:43:38.179584 send: 'AUTH OAUTHBEARER =\r\n' 18:43:42.184373 reply:
> b'334
> eyJzdGF0dXMiOiJpbnZhbGlkX3Rva2VuIiwib3BlbmlkLWNvbmZpZ3VyYXRpb24iOiJodHRwczovL2lkLm9wZW4teGNoYW5nZS5jb20vb2lkYy9jb25maWcifQ==\r\n'
>
> 18:43:42.184613 send: 'AQ==\r\n' 18:43:44.188326 reply: b'535 5.7.8
> Error: authentication failed: \r\n'
>
> Still, the error is now `Error: authentication failed: `. This also
> looks wrong, since it just ends in a colon and white space. The absence
> of a specific error message should be handled specially I guess.
The 'reason' value is now an empty string. What would you suggest instead?
Error: authentication failed\r\n
Error: authentication failed: some other text here\r\n
Wietse
_______________________________________________
Postfix-users mailing list -- postfix-users@postfix.org
To unsubscribe send an email to postfix-users-le...@postfix.org
