search

roundcube / roundcubemail

The Roundcube Webmail suite
https://roundcube.net
GNU General Public License v3.0
5.82k stars 1.63k forks source link

v1.5-beta - XOAUTH - aud param is an array #8123

Closed quenenni closed 3 years ago

quenenni commented 3 years ago

Hello,

While trying to connect my Roundcube to my LemonLdap, I found that the value of $body['aud'] is an array, not a string.

In the file program/include/rcmail_oauth.php (line 150), I had to do this change in order to make it works:

else if (isset($body['aud']) && $body['aud'] !== $this->options['client_id']) {

by

else if (isset($body['aud']) && $body['aud'][0] !== $this->options['client_id']) {
alecpl commented 3 years ago

This is fixed in 1.5-rc.

thomascube commented 3 years ago

See #8088 and #8096