Graph server sends an extra newline before message body, invalidating Content-Length

RESOLVED FIXED

Status

RESOLVED FIXED
9 years ago
2 years ago

People

(Reporter: catlee, Assigned: catlee)

Tracking

Trunk
x86
Linux

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

9 years ago
Created attachment 442709 [details] [diff] [review]
Change all newlines to CRLF, and remove extra newline after headers.
Attachment #442709 - Flags: review?(rdoherty)
(Assignee)

Updated

9 years ago
Attachment #442709 - Flags: review?(rdoherty) → review?(ianb)
Attachment #442709 - Flags: review?(ianb) → review+
Looks fine, though CRLF isn't really necessary: this is CGI, not HTTP, so everything gets processed before being sent, and the \r will be added then.
(Assignee)

Comment 2

9 years ago
(In reply to comment #1)
> Looks fine, though CRLF isn't really necessary: this is CGI, not HTTP, so
> everything gets processed before being sent, and the \r will be added then.

Huh, good to know!  Is that a property of apache's CGI handler, or the CGI spec?
(Assignee)

Comment 3

9 years ago
Comment on attachment 442709 [details] [diff] [review]
Change all newlines to CRLF, and remove extra newline after headers.

Checked in as changeset:   299:e1f168c91a17
(Assignee)

Updated

9 years ago
Depends on: 563799
(Assignee)

Updated

9 years ago
Assignee: nobody → catlee
(Assignee)

Updated

9 years ago
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
Product: Webtools → Webtools Graveyard
You need to log in before you can comment on or make changes to this bug.