User field lengths are unbounded


(PJH) #1

Can we have some (configurable?) limits on the user text fields please?

http://what.thedailywtf.com/users/matches/activity -


(PJH) #2

Not quite sure what’s going on here either…

(User -> Admin)


(Jeff Atwood) #3

Hmm yeah we should have a max limit of 2,048 or something there. Perhaps you can add a sanity limit there @eviltrout these are not meant to be “about me” fields.


(Kane York) #4

User-configurable limits would probably be ideal, someone might want to put a public key as one of their user fields. Default it to 100 or so.


(Robin Ward) #5

I’ve limited it to 2048 in this commit. We might want to make that length configurable as @riking suggests in a follow up commit.

https://github.com/discourse/discourse/commit/8186d86f384130e4d7923a0e0301306a25561a32


To have user field max size configurable
(Jeff Atwood) #6