Re: [Bugzproxy-general] bugzproxy
Status: Alpha
Brought to you by:
freefrag
|
From: JP R. <jp...@no...> - 2007-06-29 13:50:50
|
On Fri, 2007-06-29 at 11:10 +0200, Mads Bondo Dydensborg wrote: > torsdag 28 juni 2007 23:23 skrev JP Rosevear: > > On Thu, 2007-06-28 at 21:44 +0200, Mads Bondo Dydensborg wrote: > > > torsdag 28 juni 2007 21:07 skrev JP Rosevear: > > > > I've been playing with this for a couple of days and have worked around > > > > a couple of issues: > > > > > > > > 1) No https handling > > > > > > > > http:// is hardcoded > > > > > > > > 2) No http authentication support > > > > > > > > xml-rpc.net supports this > > > > > > My bad - I have absolutely no experience with using bugzilla with https. > > > > Patch for both attached. > > Thanks. Seems good. I am a bit confused about the PreAuthenticate and > Credentials code - what are they for? (Just so I know why I am patching, and > I would like to add some docs for it. :-) They are for http realm auth, see: https://msdn2.microsoft.com/en-us/library/system.net.webrequest.preauthenticate.aspx > Also, should they not be made accessible from the IServer interface, or is > that not needed? Actually, given your layout it probably is a better idea to move them there. > In any case, I have already applied the patch, and committed to cvs. The patch > did not apply cleanly, because there has been some changes to support > bugzilla installations that are not at the root of the server, but I think > everything is as it should be now, but if you check out from cvs, you may > find that the server constructors have an argument more than they used too - > I hope not, though. Ok, I diffed against 0.1.0 not the trunk so thats probably the issue. -JP -- JP Rosevear <jp...@no...> Novell, Inc. |