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

Side by Side Diff: remoting/protocol/it2me_host_authenticator_factory.cc

Issue 1755273003: Simplify AuthenticationMethod type and PIN hash handling. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 9 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "remoting/protocol/it2me_host_authenticator_factory.h" 5 #include "remoting/protocol/it2me_host_authenticator_factory.h"
6 6
7 #include "base/logging.h" 7 #include "base/logging.h"
8 #include "base/strings/string_util.h" 8 #include "base/strings/string_util.h"
9 #include "remoting/base/rsa_key_pair.h" 9 #include "remoting/base/rsa_key_pair.h"
10 #include "remoting/protocol/negotiating_host_authenticator.h" 10 #include "remoting/protocol/negotiating_host_authenticator.h"
11 #include "remoting/protocol/rejecting_authenticator.h" 11 #include "remoting/protocol/rejecting_authenticator.h"
12 12
13 namespace remoting { 13 namespace remoting {
14 namespace protocol { 14 namespace protocol {
15 15
16 It2MeHostAuthenticatorFactory::It2MeHostAuthenticatorFactory( 16 It2MeHostAuthenticatorFactory::It2MeHostAuthenticatorFactory(
17 const std::string& local_cert, 17 const std::string& local_cert,
18 scoped_refptr<RsaKeyPair> key_pair, 18 scoped_refptr<RsaKeyPair> key_pair,
19 const std::string& shared_secret, 19 const std::string& access_code,
20 const std::string& required_client_domain) 20 const std::string& required_client_domain)
21 : local_cert_(local_cert), 21 : local_cert_(local_cert),
22 key_pair_(key_pair), 22 key_pair_(key_pair),
23 shared_secret_(shared_secret), 23 access_code_(access_code),
24 required_client_domain_(required_client_domain) { 24 required_client_domain_(required_client_domain) {}
25 }
26 25
27 It2MeHostAuthenticatorFactory::~It2MeHostAuthenticatorFactory() { 26 It2MeHostAuthenticatorFactory::~It2MeHostAuthenticatorFactory() {}
28 }
29 27
30 scoped_ptr<Authenticator> It2MeHostAuthenticatorFactory::CreateAuthenticator( 28 scoped_ptr<Authenticator> It2MeHostAuthenticatorFactory::CreateAuthenticator(
31 const std::string& local_jid, 29 const std::string& local_jid,
32 const std::string& remote_jid, 30 const std::string& remote_jid,
33 const buzz::XmlElement* first_message) { 31 const buzz::XmlElement* first_message) {
34 // Check the client domain policy. 32 // Check the client domain policy.
35 if (!required_client_domain_.empty()) { 33 if (!required_client_domain_.empty()) {
36 std::string client_username = remote_jid; 34 std::string client_username = remote_jid;
37 size_t pos = client_username.find('/'); 35 size_t pos = client_username.find('/');
38 if (pos != std::string::npos) { 36 if (pos != std::string::npos) {
39 client_username.replace(pos, std::string::npos, ""); 37 client_username.replace(pos, std::string::npos, "");
40 } 38 }
41 if (!base::EndsWith(client_username, 39 if (!base::EndsWith(client_username,
42 std::string("@") + required_client_domain_, 40 std::string("@") + required_client_domain_,
43 base::CompareCase::INSENSITIVE_ASCII)) { 41 base::CompareCase::INSENSITIVE_ASCII)) {
44 LOG(ERROR) << "Rejecting incoming connection from " << remote_jid 42 LOG(ERROR) << "Rejecting incoming connection from " << remote_jid
45 << ": Domain mismatch."; 43 << ": Domain mismatch.";
46 return make_scoped_ptr( 44 return make_scoped_ptr(
47 new RejectingAuthenticator(Authenticator::INVALID_CREDENTIALS)); 45 new RejectingAuthenticator(Authenticator::INVALID_CREDENTIALS));
48 } 46 }
49 } 47 }
50 48
51 return NegotiatingHostAuthenticator::CreateWithSharedSecret( 49 return NegotiatingHostAuthenticator::CreateForIt2Me(local_cert_, key_pair_,
52 local_cert_, key_pair_, shared_secret_, AuthenticationMethod::NONE, 50 access_code_);
53 nullptr);
54 } 51 }
55 52
56 } // namespace protocol 53 } // namespace protocol
57 } // namespace remoting 54 } // namespace remoting
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698