816799 - Up-front parsing and validation of "offset" token

Status: VERIFIED WONTFIX

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

Description

[Ryan Kelly [:rfkelly]]

After the cornice-validators refactor from Bug 812432, the way that the "offset" querystring arg gets handled is pretty yuck.  We basically extract it a an opaque string and pass it to the backend, which will raise an InvalidOffsetError if it's not valid.

It would likely be cleaner to parse and validate it up-front like all the other querystring args.  Maybe not, but worth exploring.

Idea:  add a parse_offset_token() method to the storage backend that can be used to do this upfront.

Comment 1

[Toby Elliott [:telliott]]

We added an offset token, so done!

Comment 2

[Ryan Kelly [:rfkelly]]

I agree there's no action we need to take here, but changing resolution to WONTFIX because technically, this thing this bug was about has not been done (and doesn't need to be)