From fd725bce37986e9d2d8b8324b47f5ed573f2d636 Mon Sep 17 00:00:00 2001 From: =?utf8?q?W=C5=82odzimierz=20Skiba?= Date: Fri, 17 Jun 2005 11:29:55 +0000 Subject: [PATCH] Patch #1222244: Fixes for bug #1212853 with unit test. git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@34689 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775 --- docs/changes.txt | 4 +- src/common/url.cpp | 306 ++++++++++++++++++++++---------------------- tests/uris/uris.cpp | 6 +- 3 files changed, 160 insertions(+), 156 deletions(-) diff --git a/docs/changes.txt b/docs/changes.txt index 268adf7aee..a521042ecd 100644 --- a/docs/changes.txt +++ b/docs/changes.txt @@ -7,7 +7,8 @@ wxWidgets Change Log - For more verbose changes, see the manual All: -- Fixed wxScopeGuard to work with VC++, documented it +- Fixed wxScopeGuard to work with VC++, documented it. +- Fixed proxy handling in wxURL. wxMSW: @@ -17,6 +18,7 @@ wxUniv: - Window creation now honours wxVSCROLL. + 2.6.1 ----- diff --git a/src/common/url.cpp b/src/common/url.cpp index dfd818b176..cf3272ef22 100644 --- a/src/common/url.cpp +++ b/src/common/url.cpp @@ -127,53 +127,53 @@ wxURL& wxURL::operator = (const wxString& url) bool wxURL::ParseURL() { - // If the URL was already parsed (m_protocol != NULL), pass this section. - if (!m_protocol) - { - // Clean up - CleanData(); - - // Make sure we have a protocol/scheme - if (!HasScheme()) + // If the URL was already parsed (m_protocol != NULL), pass this section. + if (!m_protocol) { - m_error = wxURL_SNTXERR; - return false; - } + // Clean up + CleanData(); - // Find and create the protocol object - if (!FetchProtocol()) - { - m_error = wxURL_NOPROTO; - return false; - } + // Make sure we have a protocol/scheme + if (!HasScheme()) + { + m_error = wxURL_SNTXERR; + return false; + } - // Do we need a host name ? - if (m_protoinfo->m_needhost) - { - // Make sure we have one, then - if (!HasServer()) - { - m_error = wxURL_SNTXERR; - return false; - } + // Find and create the protocol object + if (!FetchProtocol()) + { + m_error = wxURL_NOPROTO; + return false; + } + + // Do we need a host name ? + if (m_protoinfo->m_needhost) + { + // Make sure we have one, then + if (!HasServer()) + { + m_error = wxURL_SNTXERR; + return false; + } + } } - } #if wxUSE_PROTOCOL_HTTP - if (m_useProxy) - { - // Third, we rebuild the URL. - m_url = m_scheme + wxT(":"); - if (m_protoinfo->m_needhost) - m_url = m_url + wxT("//") + m_server; - - // We initialize specific variables. - m_protocol = m_proxy; // FIXME: we should clone the protocol - } + if (m_useProxy) + { + // Third, we rebuild the URL. + m_url = m_scheme + wxT(":"); + if (m_protoinfo->m_needhost) + m_url = m_url + wxT("//") + m_server; + + // We initialize specific variables. + m_protocol = m_proxy; // FIXME: we should clone the protocol + } #endif // wxUSE_PROTOCOL_HTTP - m_error = wxURL_NOERR; - return true; + m_error = wxURL_NOERR; + return true; } // -------------------------------------------------------------- @@ -183,9 +183,9 @@ bool wxURL::ParseURL() void wxURL::CleanData() { #if wxUSE_PROTOCOL_HTTP - if (!m_useProxy) + if (!m_useProxy) #endif // wxUSE_PROTOCOL_HTTP - delete m_protocol; + delete m_protocol; } wxURL::~wxURL() @@ -206,21 +206,21 @@ wxURL::~wxURL() bool wxURL::FetchProtocol() { - wxProtoInfo *info = ms_protocols; + wxProtoInfo *info = ms_protocols; - while (info) - { - if (m_scheme == info->m_protoname) + while (info) { - if (m_port.IsNull()) - m_port = info->m_servname; - m_protoinfo = info; - m_protocol = (wxProtocol *)m_protoinfo->m_cinfo->CreateObject(); - return true; + if (m_scheme == info->m_protoname) + { + if (m_port.IsNull()) + m_port = info->m_servname; + m_protoinfo = info; + m_protocol = (wxProtocol *)m_protoinfo->m_cinfo->CreateObject(); + return true; + } + info = info->next; } - info = info->next; - } - return false; + return false; } // -------------------------------------------------------------- @@ -229,125 +229,124 @@ bool wxURL::FetchProtocol() wxInputStream *wxURL::GetInputStream() { - if (!m_protocol) - { - m_error = wxURL_NOPROTO; - return NULL; - } - - m_error = wxURL_NOERR; - if (HasUserInfo()) - { - size_t dwPasswordPos = m_userinfo.find(':'); - - if (dwPasswordPos == wxString::npos) - m_protocol->SetUser(m_userinfo); - else - { - m_protocol->SetUser(m_userinfo(0, dwPasswordPos)); - m_protocol->SetPassword(m_userinfo(dwPasswordPos+1, m_userinfo.length() + 1)); - } - } + if (!m_protocol) + { + m_error = wxURL_NOPROTO; + return NULL; + } + + m_error = wxURL_NOERR; + if (HasUserInfo()) + { + size_t dwPasswordPos = m_userinfo.find(':'); + + if (dwPasswordPos == wxString::npos) + m_protocol->SetUser(m_userinfo); + else + { + m_protocol->SetUser(m_userinfo(0, dwPasswordPos)); + m_protocol->SetPassword(m_userinfo(dwPasswordPos+1, m_userinfo.length() + 1)); + } + } #if wxUSE_URL_NATIVE - // give the native implementation to return a better stream - // such as the native WinINet functionality under MS-Windows - if (m_nativeImp) - { - wxInputStream *rc; - rc = m_nativeImp->GetInputStream(this); - if (rc != 0) - return rc; - } - // else use the standard behaviour + // give the native implementation to return a better stream + // such as the native WinINet functionality under MS-Windows + if (m_nativeImp) + { + wxInputStream *rc; + rc = m_nativeImp->GetInputStream(this); + if (rc != 0) + return rc; + } + // else use the standard behaviour #endif // wxUSE_URL_NATIVE #if wxUSE_SOCKETS wxIPV4address addr; - // m_protoinfo is NULL when we use a proxy - if (!m_useProxy && m_protoinfo->m_needhost) - { - if (!addr.Hostname(m_server)) + // m_protoinfo is NULL when we use a proxy + if (!m_useProxy && m_protoinfo->m_needhost) { - m_error = wxURL_NOHOST; - return NULL; + if (!addr.Hostname(m_server)) + { + m_error = wxURL_NOHOST; + return NULL; + } + + addr.Service(m_port); + + if (!m_protocol->Connect(addr, true)) // Watcom needs the 2nd arg for some reason + { + m_error = wxURL_CONNERR; + return NULL; + } } +#endif + + wxString fullPath; - addr.Service(m_port); + // When we use a proxy, we have to pass the whole URL to it. + if (m_useProxy) + fullPath += m_url; - if (!m_protocol->Connect(addr, true)) // Watcom needs the 2nd arg for some reason + if(m_path.empty()) + fullPath += wxT("/"); + else + fullPath += m_path; + + if (HasQuery()) + fullPath += wxT("?") + m_query; + + if (HasFragment()) + fullPath += wxT("#") + m_fragment; + + wxInputStream *the_i_stream = m_protocol->GetInputStream(fullPath); + + if (!the_i_stream) { - m_error = wxURL_CONNERR; - return NULL; + m_error = wxURL_PROTOERR; + return NULL; } - } -#endif - // When we use a proxy, we have to pass the whole URL to it. - wxInputStream *the_i_stream; - - if (!m_useProxy) - { - the_i_stream = m_protocol->GetInputStream(m_url); - } - else - { - wxString fullPath = m_path; - - if (HasQuery()) - fullPath += wxT("?") + m_query; - - if (HasFragment()) - fullPath += wxT("#") + m_fragment; - - the_i_stream = m_protocol->GetInputStream(fullPath); - } - - if (!the_i_stream) - { - m_error = wxURL_PROTOERR; - return NULL; - } - - return the_i_stream; + return the_i_stream; } #if wxUSE_PROTOCOL_HTTP void wxURL::SetDefaultProxy(const wxString& url_proxy) { - if ( !url_proxy ) - { - if ( ms_proxyDefault ) - { - ms_proxyDefault->Close(); - delete ms_proxyDefault; - ms_proxyDefault = NULL; - } - } - else - { - wxString tmp_str = url_proxy; - int pos = tmp_str.Find(wxT(':')); - if (pos == wxNOT_FOUND) - return; - - wxString hostname = tmp_str(0, pos), - port = tmp_str(pos+1, tmp_str.Length()-pos); - wxIPV4address addr; - - if (!addr.Hostname(hostname)) - return; - if (!addr.Service(port)) - return; - - if (ms_proxyDefault) - // Finally, when all is right, we connect the new proxy. - ms_proxyDefault->Close(); - else - ms_proxyDefault = new wxHTTP(); - ms_proxyDefault->Connect(addr, true); // Watcom needs the 2nd arg for some reason - } + if ( !url_proxy ) + { + if ( ms_proxyDefault ) + { + ms_proxyDefault->Close(); + delete ms_proxyDefault; + ms_proxyDefault = NULL; + } + } + else + { + wxString tmp_str = url_proxy; + int pos = tmp_str.Find(wxT(':')); + if (pos == wxNOT_FOUND) + return; + + wxString hostname = tmp_str(0, pos), + port = tmp_str(pos+1, tmp_str.Length()-pos); + wxIPV4address addr; + + if (!addr.Hostname(hostname)) + return; + if (!addr.Service(port)) + return; + + if (ms_proxyDefault) + // Finally, when all is right, we connect the new proxy. + ms_proxyDefault->Close(); + else + ms_proxyDefault = new wxHTTP(); + ms_proxyDefault->Connect(addr, true); // Watcom needs the 2nd arg for some reason + } } void wxURL::SetProxy(const wxString& url_proxy) @@ -442,4 +441,3 @@ void wxURLModule::OnExit() #endif // wxUSE_SOCKETS #endif // wxUSE_URL - diff --git a/tests/uris/uris.cpp b/tests/uris/uris.cpp index c4aa89dc5c..cbf4aef571 100644 --- a/tests/uris/uris.cpp +++ b/tests/uris/uris.cpp @@ -377,8 +377,12 @@ void URITestCase::URLProxy() { wxURL url(wxT("http://www.asite.com/index.html")); url.SetProxy(wxT("pserv:3122")); + + wxURL::SetDefaultProxy(wxT("fol.singnet.com.sg:8080")); + wxURL url2(wxT("http://server-name/path/to/file?query_data=value")); + wxInputStream *data = url2.GetInputStream(); + CPPUNIT_ASSERT(data != NULL); } #endif // wxUSE_PROTOCOL_HTTP #endif // TEST_URL - -- 2.45.2