Page MenuHomeDevCentral

Degrade silently bit.ly API call on request failure
ClosedPublic

Authored by dereckson on Nov 30 2015, 04:45.
Tags
None
Referenced Files
F3313041: D95.diff
Fri, Jul 5, 16:44
Unknown Object (File)
Wed, Jul 3, 07:01
Unknown Object (File)
Mon, Jul 1, 01:44
Unknown Object (File)
Sat, Jun 29, 20:21
Unknown Object (File)
Fri, Jun 28, 18:12
Unknown Object (File)
Thu, Jun 27, 07:57
Unknown Object (File)
Wed, Jun 26, 11:12
Unknown Object (File)
Wed, Jun 26, 11:11
Subscribers
None

Details

Summary

If a too long URL is generated by the list replace tool, bit.ly API
throws a 500 error. To prevent this, we degrade silently to a regular
non-short URL mode when this occurs.

Fixes T601.

Test Plan

test with URL provided in bug case

Diff Detail

Repository
rTOOLS Nasqueron Tools
Lint
No Lint Coverage
Unit
No Test Coverage
Branch
T601
Build Status
Buildable 104
Build 104: arc lint + arc unit

Event Timeline

dereckson retitled this revision from to Degrade silently bit.ly API call on request failure.
dereckson updated this object.
dereckson edited the test plan for this revision. (Show Details)
dereckson added a reviewer: dereckson.
dereckson added inline comments.
lists/replace.php
121

It's not a time out error, but a 500.

dereckson marked an inline comment as done.
dereckson edited edge metadata.

More precise comment

dereckson edited edge metadata.

Cleaned ommits selection

dereckson edited edge metadata.
This revision is now accepted and ready to land.Nov 30 2015, 04:50
This revision was automatically updated to reflect the committed changes.