Page MenuHomePhabricator

ConduitEpochParameterType validation fix to accept string values
AbandonedPublic

Authored by gd on Oct 12 2016, 11:37 AM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 16 2024, 6:49 AM
Unknown Object (File)
Feb 16 2024, 2:29 AM
Unknown Object (File)
Feb 3 2024, 12:26 PM
Unknown Object (File)
Jan 24 2024, 10:55 PM
Unknown Object (File)
Dec 24 2023, 11:48 AM
Unknown Object (File)
Nov 30 2023, 4:38 AM
Unknown Object (File)
Nov 17 2023, 12:58 PM
Unknown Object (File)
Oct 24 2023, 11:11 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

Allow epoch timestamp parameter to be used from calls where values are strings (e.g. curl).

Test Plan

Execute

$ curl http://$PHABRICATOR_HOST/api/maniphest.search -d api.token=$CONDUIT_TOKEN -d constraints[modifiedEnd]=$(date +%s) -d limit=1

Do not get error Error while reading "modifiedEnd": Expected epoch timestamp as integer, got something else..

Execute

$ curl http://$PHABRICATOR_HOST/api/maniphest.search -d api.token=$CONDUIT_TOKEN -d constraints[modifiedEnd]=99999999999999999999999 -d limit=1

Get error Error while reading "modifiedEnd": Epoch timestamp integer overflow.

Diff Detail

Repository
rP Phabricator
Branch
conduit-timestamp-validation-fix
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14079
Build 18266: arc lint + arc unit

Event Timeline

gd retitled this revision from to ConduitEpochParameterType validation fix to accept string values.
gd updated this object.
gd edited the test plan for this revision. (Show Details)
gd added a reviewer: epriestley.

Turns out this is a more general problem. I'm abandoning this in favour of D16694 attempt to fix this in a more general way.