Closed Bug 1240905 Opened 4 years ago Closed 4 years ago

marionettejs http.Server should be close()-ed, not kill()-ed.


(Firefox OS Graveyard :: Gaia::Shared, defect)

Gonk (Firefox OS)
Not set


(Not tracked)

2.6 S6 - 1/29


(Reporter: sfoster, Assigned: sfoster)


(Whiteboard: [systemsfe])


(1 file)

There's a couple instances where we call .kill() on a http.Server instance. There is no kill method; we should call .close()
Comment on attachment 8709636 [details] [review]
[gaia] sfoster:close-server-bug-1240905 > mozilla-b2g:master

I just found these 2 instances of this problem. This is code in /shared/ but I'm not sure its the correct component? Either way you seemed like an appropriate reviewer, but please forward it along if not.
Attachment #8709636 - Flags: review?(aus)
Assignee: nobody → sfoster
Whiteboard: [systemsfe]
Target Milestone: --- → 2.6 S6 - 1/29
Comment on attachment 8709636 [details] [review]
[gaia] sfoster:close-server-bug-1240905 > mozilla-b2g:master

Looks good! I also checked and even in node 0.10 and 0.12, kill was removed. We must've been using a wrapper for a while and then replaced it with the straight up HTTP Server class from nodejs but failed to remove the use of 'kill' and instead use 'close'. I'm really surprised it didn't break things sooner! Sorry you had to step on that landmine but SUPER happy it's fixed!
Attachment #8709636 - Flags: review?(aus) → review+
There's a lot of blue in the test run here. I'm not clear if this is normal - is this good to land?
Flags: needinfo?(mhenretty)
Yup, those blues are pretty normal these days:
Flags: needinfo?(mhenretty)
Merged to master:
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.