Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(131)

Unified Diff: remoting/signaling/xmpp_signal_strategy.cc

Issue 959033004: Remove ClientLogin support from remoting host (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/signaling/xmpp_signal_strategy.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/signaling/xmpp_signal_strategy.cc
diff --git a/remoting/signaling/xmpp_signal_strategy.cc b/remoting/signaling/xmpp_signal_strategy.cc
index 806852e99935f2c569a19a8c222efdd13637053d..5ad0c039abdfcb8e6577a937c4bf24bbccaa5a40 100644
--- a/remoting/signaling/xmpp_signal_strategy.cc
+++ b/remoting/signaling/xmpp_signal_strategy.cc
@@ -83,7 +83,7 @@ void XmppSignalStrategy::Connect() {
settings.set_user(login_jid.node());
settings.set_host(login_jid.domain());
settings.set_resource(resource_name_);
- settings.set_token_service(xmpp_server_config_.auth_service);
+ settings.set_token_service("oauth2");
settings.set_auth_token(buzz::AUTH_MECHANISM_GOOGLE_TOKEN,
xmpp_server_config_.auth_token);
@@ -203,12 +203,10 @@ bool XmppSignalStrategy::HandleStanza(const buzz::XmlElement* stanza) {
}
void XmppSignalStrategy::SetAuthInfo(const std::string& username,
- const std::string& auth_token,
- const std::string& auth_service) {
+ const std::string& auth_token) {
DCHECK(CalledOnValidThread());
xmpp_server_config_.username = username;
xmpp_server_config_.auth_token = auth_token;
- xmpp_server_config_.auth_service = auth_service;
}
void XmppSignalStrategy::SetResourceName(const std::string &resource_name) {
@@ -269,13 +267,8 @@ void XmppSignalStrategy::SendKeepAlive() {
buzz::PreXmppAuth* XmppSignalStrategy::CreatePreXmppAuth(
const buzz::XmppClientSettings& settings) {
buzz::Jid jid(settings.user(), settings.host(), buzz::STR_EMPTY);
- std::string mechanism = notifier::kDefaultGaiaAuthMechanism;
- if (settings.token_service() == "oauth2") {
- mechanism = "X-OAUTH2";
- }
-
return new notifier::GaiaTokenPreXmppAuth(
- jid.Str(), settings.auth_token(), settings.token_service(), mechanism);
+ jid.Str(), settings.auth_token(), settings.token_service(), "X-OAUTH2");
}
} // namespace remoting
« no previous file with comments | « remoting/signaling/xmpp_signal_strategy.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698