[zypp-commit] r8562 - in /branches/tmp/ma/jump_sat/libzypp/zypp: CMakeLists.txt media/MediaCurl.cc

Author: coolo Date: Fri Feb 8 10:11:29 2008 New Revision: 8562 URL: http://svn.opensuse.org/viewcvs/zypp?rev=8562&view=rev Log: r8368 | jkupec | 2008-01-24 18:26:36 +0100 (Do, 24 Jan 2008) | 2 lines - read .curlrc more robustly (#330351) Modified: branches/tmp/ma/jump_sat/libzypp/zypp/CMakeLists.txt branches/tmp/ma/jump_sat/libzypp/zypp/media/MediaCurl.cc Modified: branches/tmp/ma/jump_sat/libzypp/zypp/CMakeLists.txt URL: http://svn.opensuse.org/viewcvs/zypp/branches/tmp/ma/jump_sat/libzypp/zypp/CMakeLists.txt?rev=8562&r1=8561&r2=8562&view=diff ============================================================================== --- branches/tmp/ma/jump_sat/libzypp/zypp/CMakeLists.txt (original) +++ branches/tmp/ma/jump_sat/libzypp/zypp/CMakeLists.txt Fri Feb 8 10:11:29 2008 @@ -285,6 +285,7 @@ media/MediaISO.cc media/MediaManager.cc media/MediaUserAuth.cc + media/CurlConfig.cc ) SET( zypp_media_HEADERS @@ -304,6 +305,7 @@ media/MediaUserAuth.h media/Mount.h media/ProxyInfo.h + media/CurlConfig.h ) INSTALL( FILES Modified: branches/tmp/ma/jump_sat/libzypp/zypp/media/MediaCurl.cc URL: http://svn.opensuse.org/viewcvs/zypp/branches/tmp/ma/jump_sat/libzypp/zypp/media/MediaCurl.cc?rev=8562&r1=8561&r2=8562&view=diff ============================================================================== --- branches/tmp/ma/jump_sat/libzypp/zypp/media/MediaCurl.cc (original) +++ branches/tmp/ma/jump_sat/libzypp/zypp/media/MediaCurl.cc Fri Feb 8 10:11:29 2008 @@ -23,6 +23,7 @@ #include "zypp/media/proxyinfo/ProxyInfos.h" #include "zypp/media/ProxyInfo.h" #include "zypp/media/MediaUserAuth.h" +#include "zypp/media/CurlConfig.h" #include "zypp/thread/Once.h" #include <cstdlib> #include @@ -220,6 +221,9 @@ if ( !_url.isValid() ) ZYPP_THROW(MediaBadUrlException(_url)); + CurlConfig curlconf; + CurlConfig::parseConfig(curlconf); // parse ~/.curlrc + curl_version_info_data *curl_info = NULL; curl_info = curl_version_info(CURLVERSION_NOW); // curl_info does not need any free (is static) @@ -560,39 +564,17 @@ _proxyuserpwd = _url.getQueryParam( "proxyuser" ); if ( ! _proxyuserpwd.empty() ) { - string proxypassword( _url.getQueryParam( "proxypassword" ) ); if ( ! proxypassword.empty() ) { _proxyuserpwd += ":" + proxypassword; } - } else { - char *home = getenv("HOME"); - if( home && *home) + if (curlconf.proxyuserpwd.empty()) + DBG << "~/.curlrc does not contain the proxy-user option" << endl; + else { - Pathname curlrcFile = string( home ) + string( "/.curlrc" ); - - PathInfo h_info(string(home), PathInfo::LSTAT); - PathInfo c_info(curlrcFile, PathInfo::LSTAT); - - if( h_info.isDir() && h_info.owner() == getuid() && - c_info.isFile() && c_info.owner() == getuid()) - { - //! \todo FIXME adjust to .curlrc syntax: = and -- are optional!# - - map rc_data = base::sysconfig::read( curlrcFile ); - - map::const_iterator it = rc_data.find("--proxy-user"); - if (it != rc_data.end()) - _proxyuserpwd = it->second; - - DBG << "got proxy userpwd (--proxy-user) from ~/culrc" << endl; - } - else - { - WAR << "Not allowed to parse '" << curlrcFile - << "': bad file owner" << std::endl; - } + _proxyuserpwd = curlconf.proxyuserpwd; + DBG << "using proxy-user from ~/.curlrc" << endl; } } -- To unsubscribe, e-mail: zypp-commit+unsubscribe@opensuse.org For additional commands, e-mail: zypp-commit+help@opensuse.org