staticman icon indicating copy to clipboard operation
staticman copied to clipboard

allowedFields isn't being respected.

Open dpnova opened this issue 7 years ago • 0 comments

Here's an example PR that shouldn't exist: https://github.com/enhancedsociety/website/pull/13

It looks like here, it's allowing the fields if they are empty. https://github.com/eduardoboucas/staticman/blob/44f95c6c0957099e3deb8f0ea7f51612be91b5d3/lib/Staticman.js#L391

We are trying to use them as a honeypot to filter spam, but allowing the empty string through breaks that idea.

dpnova avatar Jun 11 '18 11:06 dpnova