----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/102696/#review6938 -----------------------------------------------------------
This review has been submitted with commit 4a1e32abdd591d63cd1da7830a71b025fb69dc3f by Dawit Alemayehu to branch KDE/4.7. - Commit Hook On Sept. 27, 2011, 9:12 p.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/102696/ > ----------------------------------------------------------- > > (Updated Sept. 27, 2011, 9:12 p.m.) > > > Review request for kdelibs. > > > Description > ------- > > This 5th patch in a serious of patches meant to improve proxy support in KDE > deals with support at the kioslave level. More specifically the http ioslave. > The patch is necessary to provide proper support for PAC script based proxy > configuration. Namely allowing aleternate proxy servers to be specified and > used as necessary. Here are the change this patch makes: > > * Add a new function in TCPSlaveBase to connect to the a remote server > without > automatically sending error notitification to the client. [NEW API] > > * Move the proxy related code from 'resetSessionSettings' to > 'httpOpenConnection'. > Proxy information will now be only set from 'setHost' and reset from > 'reparseConfiguration' as it should have been from the beginning. No > need to > reparse proxy related information on every request. > > * Added a new variable, proxyUrls, to HTTPRequest to store the multiple > proxy URLs > obtained from the "ProxyUrls" meta-data. > > * Modified 'httpShouldCloseConnection' to account for multiple proxy > addresses. > > * Modified 'sendQuery' to connect to the remote server before formatting > the HTTP > headers so that the headers can properly reflect the correct proxy > settings. > > > Diffs > ----- > > kio/kio/tcpslavebase.h 3f87ea8 > kio/kio/tcpslavebase.cpp ec70559 > kioslave/http/http.h d8c47c7 > kioslave/http/http.cpp 6d41a13 > > Diff: http://git.reviewboard.kde.org/r/102696/diff/diff > > > Testing > ------- > > - Tested changes using a real proxy server, privoxy. > > - Tested changes using a poor man's SOCKS proxy setup: > ssh -D <port#> <ssh-server-address> > and the following PAC script: > > function FindProxyForURL( url, host ) > { > var resolved_ip = dnsResolve(host); > > if (isInNet(resolved_ip, "127.0.0.1", "255.255.255.0")) > return "DIRECT"; > > return "SOCKS 127.0.0.1:9999; DIRECT"; > } > > > Thanks, > > Dawit Alemayehu > >