From 56038ded5192cfae5217ae6eae7480923b9298e5 Mon Sep 17 00:00:00 2001 From: Merov Linden Date: Thu, 25 Jul 2013 19:35:10 -0700 Subject: ACME-745 : Fix : Do not oversanitize the logged URLs... --- indra/newview/llviewermedia.cpp | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'indra') diff --git a/indra/newview/llviewermedia.cpp b/indra/newview/llviewermedia.cpp index 6453e2348f..13483790ed 100755 --- a/indra/newview/llviewermedia.cpp +++ b/indra/newview/llviewermedia.cpp @@ -2003,7 +2003,8 @@ void LLViewerMediaImpl::loadURI() { // Do not log the query parts LLURI u(uri); - llinfos << "Asking media source to load URI: " << u.scheme() << "://" << u.authority() << u.path() << llendl; + std::string sanitized_uri = (u.query().empty() ? uri : u.scheme() + "://" + u.authority() + u.path()); + llinfos << "Asking media source to load URI: " << sanitized_uri << llendl; } mMediaSource->loadURI( uri ); @@ -2574,7 +2575,8 @@ void LLViewerMediaImpl::navigateTo(const std::string& url, const std::string& mi { // Do not log the query parts LLURI u(url); - llinfos << "NOT LOADING media id= " << mTextureId << " url=" << u.scheme() << "://" << u.authority() << u.path() << " mime_type=" << mime_type << llendl; + std::string sanitized_url = (u.query().empty() ? url : u.scheme() + "://" + u.authority() + u.path()); + llinfos << "NOT LOADING media id= " << mTextureId << " url=" << sanitized_url << ", mime_type=" << mime_type << llendl; } // This impl should not be loaded at this time. @@ -2593,7 +2595,8 @@ void LLViewerMediaImpl::navigateInternal() { // Do not log the query parts LLURI u(mMediaURL); - llinfos << "media id= " << mTextureId << " url=" << u.scheme() << "://" << u.authority() << u.path() << " mime_type=" << mMimeType << llendl; + std::string sanitized_url = (u.query().empty() ? mMediaURL : u.scheme() + "://" + u.authority() + u.path()); + llinfos << "media id= " << mTextureId << " url=" << sanitized_url << ", mime_type=" << mMimeType << llendl; } if(mNavigateSuspended) -- cgit v1.2.3