Closed Bug 968512 Opened 10 years ago Closed 10 years ago

application/newlines format requires a trailing newline

Categories

(Cloud Services Graveyard :: Server: Sync, defect)

Product:
Cloud Services Graveyard
Component:
Server: Sync
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: rfkelly, Assigned: rfkelly)

References

Details

(Whiteboard: [qa-])

Attachments

(2 files)

docs-trailing-newline.diff
989 bytes, patch
rnewman
: review+
Details | Diff | Splinter Review
sync15-trailing-newline.diff
1.11 KB, patch
telliott
: review+
Details | Diff | Splinter Review 
The sync client code expects a newline after every item in an application/newlines response body, including a trailing newline after the final item.  Let's clarify the spec on this topic and add the appropriate code to the server.
Whiteboard: [qa-]
Docs patch to clarify that every item is followed by a newline.
Attachment #8371105 - Flags: review?(rnewman)
OS: Windows 7 → All
Hardware: x86_64 → All
Code to actually send the trailing newline
Attachment #8371108 - Flags: review?(telliott)
Attachment #8371108 - Flags: review?(telliott) → review+
Attachment #8371105 - Flags: review?(rnewman) → review+
Blocks: 961806
Bob is spinning up a fresh deploy with this change now.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Blocks: 968341
Bug for deploying this to prod: Bug 968633
ok.
Status: RESOLVED → VERIFIED
Product: Cloud Services → Cloud Services Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: