Homec4science

Truncate Conduit 'clientDescription' so we don't overflow the column

Authored by epriestley <git@epriestley.com> on Feb 25 2014, 21:35.

Description

Truncate Conduit 'clientDescription' so we don't overflow the column

Summary: We store the arc commandline in this 255-character column, but it can be more than 255-characters long. If it's huge, truncate it.

Test Plan:
Executed:

arc list --conduit-uri=http://local.aphront.com:8080/ --conduit-version 6.aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa

Works fine after this patch.

Reviewers: btrahan

Reviewed By: btrahan

CC: aran

Differential Revision: https://secure.phabricator.com/D8327

Details

Committed
epriestley <git@epriestley.com>Feb 25 2014, 21:35
Pushed
aubortJan 31 2017, 17:16
Parents
rPHf0e6dc122109: Updated `.arcconfig` to match newer style configuration.
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH72a062d8027b: Truncate Conduit 'clientDescription' so we don't overflow the column (authored by epriestley <git@epriestley.com>).Feb 25 2014, 21:35