Closed Bug 1297561 Opened 4 years ago Closed 4 years ago

Batch IDs should be returned as an opaque string, not an integer

Categories

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

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rfkelly, Unassigned)

References

Details

(Whiteboard: [sync-atomic])

As noted in [1], the server is currently returning batch ids as integers in its JSON response:

  {
    "batch": 1234
  }

The protocol doc says they're opaque strings, so we should return them as a string type.  I suggest base64-encoding them to make them more obviously opaque:

  {
    "batch": "MTIzNA"
  }


[1] https://github.com/mozilla/firefox-ios/pull/2014#issuecomment-241757291
Summary: Batch ids should be returned as an opauqe string, not an integer → Batch IDs should be returned as an opaque string, not an integer
Priority: -- → P3
Whiteboard: [atomic-io]
Whiteboard: [atomic-io] → [sync-atomic]
This was fixed, closing.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.