Closed Bug 819550 Opened 10 years ago Closed 6 months ago

MozHttpd class should inherit from SimpleServer


(Testing :: Mozbase, defect)

Not set


(Not tracked)



(Reporter: k0scist, Unassigned)




(1 file)

Unless there is a good reason we are avoiding inheriting from python's
SimpleServer, which we utilize anyway.
I see that we use EasyServer which inherits from BaseHTTPServer. Do you want to remove EasyServer, make MozHttpd inherit from SimpleHTTPServer and just use that?
Yes, that sounds right.
Assignee: nobody → mihneadb
Attachment #711242 - Flags: review?(jhammel)
Comment on attachment 711242 [details] [diff] [review]
refactor as requested

This looks good to me.  I am not sure if this breaks existing stuff though so am giving the review over.  :jmaher, if you don't know or don't have time to look into this, could you pass the buck to :wlach?  And if he doesn't know he can pass the buck back to me for an r+ unless :ted wants to untangle this madness.
Attachment #711242 - Flags: review?(jhammel) → review?(jmaher)
For what it's worth, the mozbase tests pass.
(In reply to Mihnea Dobrescu-Balaur (:mihneadb) from comment #5)
> For what it's worth, the mozbase tests pass.

Sadly, that ain't worth much :(
Comment on attachment 711242 [details] [diff] [review]
refactor as requested

Review of attachment 711242 [details] [diff] [review]:

I remember originally wanting to go with BaseHttpServer, but the path of least resistance was SimpleHttpServer.  Please ensure that the performance of this is the same, I recall an attempt early on when loading about 1000 pages in a row took an order of magnitude longer.
Attachment #711242 - Flags: review?(jmaher) → review+

The bug assignee didn't login in Bugzilla in the last 7 months.
:gbrown, could you have a look please?
For more information, please visit auto_nag documentation.

Assignee: mihneadb → nobody
Flags: needinfo?(gbrown)
Closed: 6 months ago
Flags: needinfo?(gbrown)
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.