Validate slug and block invalid slug

cofacts / rumors-api

GraphQL API server for clients like rumors-site and rumors-line-bot

https://api.cofacts.tw

MIT License

109 stars 26 forks source link

Validate slug and block invalid slug #252

Closed MrOrz closed 3 years ago

MrOrz commented 3 years ago

This PR fixes #242 .

The definition of "invalid slug": matches any of the following

empty, or contains just space
has trailing or leading space
contains URL components like : or /
is already taken by another user

This PR

adds a query API ValidateSlug for UI to check if a slug is valid
blocks invalid slug in UpdateUser

coveralls commented 3 years ago

Coverage increased (+0.06%) to 86.406% when pulling 81d5065a1755983413246effe9e3c9367696bf13 on validate-slug into 0c42992c82eb25851433f3f275b663eef84defab on master.