Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

Not sanitizing user input? #7

Open
crcastle opened this issue Sep 1, 2020 · 2 comments
Open

Not sanitizing user input? #7

crcastle opened this issue Sep 1, 2020 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@crcastle
Copy link
Contributor

crcastle commented Sep 1, 2020 •
edited
Loading

I typed this as my caption and got the below GIF "chrome desktop (4:17pm build)"

1599002321743

Looks like the colon broke it.
@julianduque julianduque added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Sep 2, 2020
@julianduque
Copy link
Contributor

It would be also good to limit the size of the caption, when it is too long it's not possible to read.

@clif-os
Copy link
Contributor

clif-os commented Sep 3, 2020

Ok maybe we can change this ticket title to something like "Revisit Text Sanitization"

DKulan added a commit to DKulan/gif-booth that referenced this issue Apr 12, 2021
@DKulan
issue developerforce#7
11a72da
DKulan added a commit to DKulan/gif-booth that referenced this issue Apr 12, 2021
@DKulan
Revert "issue developerforce#7"
94400e6 
This reverts commit 11a72da.
DKulan added a commit to DKulan/gif-booth that referenced this issue Apr 12, 2021
@DKulan
Revert "Revert "issue developerforce#7""
257f261 
This reverts commit 94400e6.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@crcastle @julianduque @clif-os