Re: apt https method v2
On Sat, 6 Apr 2002, Tomas Pospisek wrote:
> Ummm.. so which direction do you want me to try? Should I try moving the
> Connect/Read/Write/Close/Open stuff into Connection or not (which will
> make an even bigger diff!)
I think it will be smaller actually, compared with my code you need to add
two new classes (Connection and ConnectionSSL) and shuffle some routines
around slightly. The code you already wrote is still correct, just needs
some renamed function calls.
> And if - how do you think Connection should look like? Should I do a http
> version and subclass it into ConnectionSSL?
Well, one would have a standard POSIX version and an OpenSSL version that
creates streams with the various special semantics.
> > Comment at the top of https.cc should describe the interface to openssl.
>
> Something like:
No.. right now there is not much to say, just that it uses openssl to
provide encryption/authentication for https - but eventually you will have
lots to say about how the authentication mechanisms work. What you have
now is very much wrong for the file it describes.
> So what I need to know to proceed without reimplementing everything
> the next time over again in a different way:
Well, IHMO there are two options
- Suck everything into CircleBuf, rename it to 'ConnectionWithBuffering'
or some such and have the necessary magic for that - this was the
orignal suggestion
- Suck all the connection management stuff into a Connection class and
teach CircleBuf how to do IO on a Connection class - this is the new
suggestion
The latter seems to be the nicer of the two, more accurately models
things.
Jason
--
To UNSUBSCRIBE, email to deity-request@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmaster@lists.debian.org
Reply to: