No longer necessary to treat 304 responses specially. #322

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@smh

The end event should also be emitted for 304 (Not modified) responses.

Starting with commit 182dcd3, there is no need to special handle these response statuses.

Ensures that end is emitted also for 304 responses.

@smh smh No longer necessary to test for status code 304.
The end event should also be emitted for 304 (Not modified) responses.
Starting with commit 182dcd3, there is
no need to special handle these response statuses.

Ensures that end is emitted also for 304 responses.
c3a2949
@smh
smh commented Nov 4, 2012

@mmalecki any hope of getting this in? As it stands, a 304 response will not emit an end event, unlike all other responses.

@indexzero indexzero closed this in 28a3b0a Mar 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment