check-face / checkface

Putting a face to a hash
https://checkface.facemorph.me
Other
25 stars 13 forks source link

Whitespace is not handled correctly #14

Closed olivercoad closed 5 years ago

olivercoad commented 5 years ago

On the clients, the value is not trimmed, allowing for values such as "c " to be requested. However, on the website, it does not include trailing spaces in the url query parameter even though it makes the api request with the trailing spaces.

Current Behaviour

After typing in "c " in checkface.ml, the url is updated to https://checkface.ml/?value=c instead of https://checkface.ml/?value=c%20%20%20%20.

Expected behaviour

Either the worker trims the input so that both requests are the same, or the website should include trailing spaces in the query parameter.

olivercoad commented 5 years ago

Closed by 594152d45b3e400d14ef4111a8a81e8c544bcfe3