Re: additional comments to my proxy changes

[Hakan Ardo <hakan@debian.org>]

-----BEGIN PGP SIGNED MESSAGE-----

On 10 Jun 1997, Ueli Rutishauser wrote:

> HTTPHandler.cc /h additional GetObject Method with a additional argument
> (type HTTPUrl) for proxy host/port later, i wan't make direct IP

A little typo:

When you search for already open connections:

  if( (cindex = FindAliveFD(url->host, url->port)) != -1) {

you search for connections to the host itselfe, not the proxy, this should
be:

  if( (cindex = FindAliveFD(proxy_url->host, proxy_url->port)) != -1) {

Or have I mussunderstood something?

> support, so that not allways the DNS must be asked. or better, i make
> this over the DNS Cache :) 

Ofcorse even the proxy should be included in the DNS cache, but  we
will keep an open connection to the proxy during the entire season, or
untill the proxy server times out and throuws us out...

> HTTPRequest10.cc
> default "GET http://host/location HTTP ...." instead of "GET location HTTP..."
> later http:// must be replaced with some other protocols (FTP)

I don't know what old/bad servers says about the http://host/location form
in stead of the usual location, and there is a slitely better (I think)
way to do this (using the old HTTPRequest10): 

In the HTTPHandler::GetObject method used by the proxy, you now construct
the Request like this:

 conns[cindex].req = new HTTPRequest10(mgroup,url->host,url->port,url->location);

If you instead do:

 conns[cindex].req = new HTTPRequest10(mgroup,url->host,url->port,s_url);

This way the location in HTTPRequest10 will be "http://host/location".
This will also make HTTPRequest10 and HTTPHandler completly protocoll
independent!

What do you say, is this a good way?

- ---------------------------------------------------------------
 Name:        Hakan Ardo
 E-Mail:      hakan@debian.org
 WWW:         http://www.ub2.lu.se/~hakan/sig.html
 Public Key:  Try "finger hakan@master.debian.org"
 Fingerprint: E9 81 FD 90 53 5C E9 3E  3D ED 57 15 1B 7E 29 F3
 Interests:   WWW, Programming, 3D graphics

 Thought for the day: As long as one understands, the
 spelling does not matter :-)
- ---------------------------------------------------------------

-----BEGIN PGP SIGNATURE-----
Version: 2.6.3i
Charset: noconv

iQCVAwUBM50Vet6dx9igIm71AQG/SwQAm3XUp9u+8/1T2QshmlC305l868KwwkXT
HZ1nku47hg/SwBf/q4QHpo6TT/v2MiiN2ciDumEgoZJjwXH1nmPA7Qt72gJ3iyr0
IXbHMHWMaZ6YzRqrG8IqyYPBP62KPwYkLJ7d418MUiKTboUbr+Ni4FPS10A/tgVm
+ZSKiniwX9M=
=dEd/
-----END PGP SIGNATURE-----