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

Unified Diff: remoting/host/host_event_logger_posix.cc

Issue 9727005: Log connection type to syslogs and to the server. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 8 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « remoting/host/client_session.cc ('k') | remoting/host/host_event_logger_win.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: remoting/host/host_event_logger_posix.cc
diff --git a/remoting/host/host_event_logger_posix.cc b/remoting/host/host_event_logger_posix.cc
index 56bc1749b60bab2fd817ab86a55a3d2e0e8763f3..33b4b311407d1f32268ba411256256d3a7e1196e 100644
--- a/remoting/host/host_event_logger_posix.cc
+++ b/remoting/host/host_event_logger_posix.cc
@@ -10,6 +10,7 @@
#include "net/base/ip_endpoint.h"
#include "remoting/host/chromoting_host.h"
#include "remoting/host/host_status_observer.h"
+#include "remoting/protocol/transport.h"
// Included here, since the #define for LOG_USER in syslog.h conflicts with the
// constants in base/logging.h, and this source file should use the version in
@@ -35,8 +36,7 @@ class HostEventLoggerPosix : public HostEventLogger, public HostStatusObserver {
virtual void OnClientRouteChange(
const std::string& jid,
const std::string& channel_name,
- const net::IPEndPoint& remote_end_point,
- const net::IPEndPoint& local_end_point) OVERRIDE;
+ const protocol::TransportRoute& route) OVERRIDE;
virtual void OnShutdown() OVERRIDE;
private:
@@ -78,12 +78,13 @@ void HostEventLoggerPosix::OnAccessDenied(const std::string& jid) {
void HostEventLoggerPosix::OnClientRouteChange(
const std::string& jid,
const std::string& channel_name,
- const net::IPEndPoint& remote_end_point,
- const net::IPEndPoint& local_end_point) {
+ const protocol::TransportRoute& route) {
Log(base::StringPrintf(
- "Channel IP for client: %s ip='%s' host_ip='%s' channel='%s'",
- jid.c_str(), remote_end_point.ToString().c_str(),
- local_end_point.ToString().c_str(), channel_name.c_str()));
+ "Channel IP for client: %s ip='%s' host_ip='%s' channel='%s' "
+ "connection='%s'",
+ jid.c_str(), route.remote_address.ToString().c_str(),
+ route.local_address.ToString().c_str(), channel_name.c_str(),
+ protocol::TransportRoute::GetTypeString(route.type).c_str()));
}
void HostEventLoggerPosix::OnShutdown() {
« no previous file with comments | « remoting/host/client_session.cc ('k') | remoting/host/host_event_logger_win.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698