[Freeipa-devel] [PATCH] 368 remove some duplicate code
Jason Gerard DeRose
jderose at redhat.com
Tue Feb 9 10:55:40 UTC 2010
On Mon, 2010-02-08 at 11:03 -0500, Rob Crittenden wrote:
> John Dennis wrote:
> > On 02/02/2010 10:56 PM, Rob Crittenden wrote:
> >> Move the dogtag HTTP and HTTPS request functions to a common library to
> >> remove duplication.
> >>
> >> I'm next planning on moving the XML parsing code into ipapython/dogtag
> >> because that is needed by the installer as well to identify errors. This
> >> is going to be quite a bit messier I think.
> >
> > There are a couple of minor things which need cleaning up.
> >
> > The new functions https_request and http_request are not general request
> > mechanisms because the request type is hardcoded to POST and the headers
> > are hardcoded as well. We need to either change the names to https_post
> > and http_post -or- add parameters specifying the request type and
> > headers. POST is specialized enough and called often enough I think the
> > first option is best, just rename the function to what it actually is, a
> > post request.
> >
> > The logging statements still refer to the old function names, they
> > should be updated to match the name of the function.
> >
> > If all you do is just edit the names I don't think you need to repost
> > the patch, I'll ACK it now.
> >
>
> Yeah, I had the same thoughts on the naming. I decided to go with them
> since the context is within dogtag and not a general HTTP/S client. I'm
> ok with renaming them again for clarity/maintainability.
>
> rob
ack. pushed to master.
We can rename these functions for greater clarity in a subsequent patch,
but in the mean time we should move forward with this code
consolidation.
More information about the Freeipa-devel
mailing list