[evolution-data-server/gnome-3-16] [CalDAV] Stop re-trying authentication with bearer authenticator
- From: Milan Crha <mcrha src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [evolution-data-server/gnome-3-16] [CalDAV] Stop re-trying authentication with bearer authenticator
- Date: Mon, 23 Mar 2015 16:56:55 +0000 (UTC)
commit 38c4ca537a09a8eadcef8103f2a8a6d81b6a175f
Author: Milan Crha <mcrha redhat com>
Date: Mon Mar 23 17:41:10 2015 +0100
[CalDAV] Stop re-trying authentication with bearer authenticator
The bearer authenticator is used for OAuth2 authentication, which
cannot ask for passwords, thus do not let it behave like that and
simply return the credentials were rejected, instead of reusing
those stored again and again. This could eventually lock Google
accounts too.
calendar/backends/caldav/e-cal-backend-caldav.c | 8 ++++++--
1 files changed, 6 insertions(+), 2 deletions(-)
---
diff --git a/calendar/backends/caldav/e-cal-backend-caldav.c b/calendar/backends/caldav/e-cal-backend-caldav.c
index fa90ce1..914649f 100644
--- a/calendar/backends/caldav/e-cal-backend-caldav.c
+++ b/calendar/backends/caldav/e-cal-backend-caldav.c
@@ -127,6 +127,7 @@ struct _ECalBackendCalDAVPrivate {
* message than a generic SOUP_STATUS_UNAUTHORIZED description. */
GError *bearer_auth_error;
GMutex bearer_auth_error_lock;
+ gboolean using_bearer_auth;
};
/* Forward Declarations */
@@ -1047,10 +1048,12 @@ soup_authenticate (SoupSession *session,
extension_name = E_SOURCE_EXTENSION_AUTHENTICATION;
auth_extension = e_source_get_extension (source, extension_name);
+ cbdav->priv->using_bearer_auth = E_IS_SOUP_AUTH_BEARER (auth);
+
if (retrying)
return;
- if (E_IS_SOUP_AUTH_BEARER (auth)) {
+ if (cbdav->priv->using_bearer_auth) {
soup_authenticate_bearer (session, msg, auth, cbdav);
/* do not send same password twice, but keep it for later use */
@@ -5431,7 +5434,8 @@ caldav_authenticate_sync (EBackend *backend,
}
if (username && *username) {
- if (!e_named_parameters_get (credentials, E_SOURCE_CREDENTIAL_PASSWORD))
+ if (!cbdav->priv->using_bearer_auth &&
+ !e_named_parameters_get (credentials, E_SOURCE_CREDENTIAL_PASSWORD))
result = E_SOURCE_AUTHENTICATION_REQUIRED;
else
result = E_SOURCE_AUTHENTICATION_REJECTED;
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]