politeia icon indicating copy to clipboard operation
politeia copied to clipboard

politeiad: Plugin metadata JSON bug.

Open lukebp opened this issue 3 years ago • 1 comments

The plugin hooks add plugin specific validation to record submissions and updates. A case that was not considered was if the client provides a valid base64 metadata payload, but that contains invalid JSON. This causes politeiad to fail with a 500 instead of handling the error gracefully with a UserErrorReply.

Attribution This bug was introduced by #1180. It was the result of not considering all input edge cases.

Prevention Test cases for invalid JSON should be added for all plugin metadata validation. A testing framework has not been setup yet for the politeiad plugins. This will be added by #1494.

lukebp avatar Jun 18 '21 17:06 lukebp

Blocked by #1406

lukebp avatar Jun 28 '21 22:06 lukebp