HTTP resuming code should check for error status codes

RESOLVED FIXED in mozilla1.9alpha8

Status

()

RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: Biesinger, Assigned: Biesinger)

Tracking

unspecified
mozilla1.9alpha8
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(from bug 377243 comment 68)

An error status (4xx, etc) should cause the channel to fail when resuming
Created attachment 278137 [details] [diff] [review]
patch
Attachment #278137 - Flags: superreview?(bzbarsky)
Attachment #278137 - Flags: review?(bzbarsky)
Comment on attachment 278137 [details] [diff] [review]
patch

Looks reasonable
Attachment #278137 - Flags: superreview?(bzbarsky)
Attachment #278137 - Flags: superreview+
Attachment #278137 - Flags: review?(bzbarsky)
Attachment #278137 - Flags: review+
Target Milestone: --- → mozilla1.9 M8
Attachment #278137 - Flags: review+
Attachment #278137 - Flags: approval1.9?
Attachment #278137 - Flags: approval1.9+
Checking in protocol/http/src/nsHttpChannel.cpp;
/cvsroot/mozilla/netwerk/protocol/http/src/nsHttpChannel.cpp,v  <--  nsHttpChannel.cpp
new revision: 1.319; previous revision: 1.318
done
Checking in test/unit/head_channels.js;
/cvsroot/mozilla/netwerk/test/unit/head_channels.js,v  <--  head_channels.js
new revision: 1.2; previous revision: 1.1
done
RCS file: /cvsroot/mozilla/netwerk/test/unit/test_resumable_channel.js,v
done
Checking in test/unit/test_resumable_channel.js;
/cvsroot/mozilla/netwerk/test/unit/test_resumable_channel.js,v  <--  test_resumable_channel.js
initial revision: 1.1
done
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.