Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #2900 - Checks slur regex to see if it is too permissive #3146

Merged
merged 16 commits into from
Jun 27, 2023
Merged

Fixes #2900 - Checks slur regex to see if it is too permissive #3146

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
a51ffe6
Fixes #2900 - Checks slur regex to see if it is too permissive along …
ninanator Jun 16, 2023
ca6c519
Clean up variable names, add handler for valid empty string usecase
ninanator Jun 16, 2023
69b9b79
Merge branch 'main' into lemmy-2900-check-slur-regex
ninanator Jun 16, 2023
6836080
Update tests
ninanator Jun 16, 2023
c01e944
Create validation function and add tests
ninanator Jun 17, 2023
3569b0f
Test clean up
ninanator Jun 17, 2023
3526feb
Use payload value vs local site value to prevent stunlocking
ninanator Jun 17, 2023
aba61cf
Remove println added while testing
ninanator Jun 17, 2023
753d50c
Fall back to local site regex if not provided from request
ninanator Jun 17, 2023
33eff7e
Attempt clean up of flaky comment_view tests
ninanator Jun 18, 2023
cf1f0e4
Pull in latest submodule
ninanator Jun 19, 2023
2613892
Merge in main
ninanator Jun 27, 2023
9b883f2
Merge main, resolve conflicts
ninanator Jun 27, 2023
597d8f1
Move application, post check into functions, add more tests and impro…
ninanator Jun 27, 2023
cfb1f0f
Merge branch 'main' into lemmy-2900-check-slur-regex
Nutomic Jun 27, 2023
eea4b20
Merge branch 'main' into lemmy-2900-check-slur-regex
dessalines Jun 27, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 0 additions & 9 deletions crates/api_common/src/utils.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -299,15 +299,6 @@ pub fn password_length_check(pass: &str) -> Result<(), LemmyError> {
}
}

/// Checks the site description length
pub fn site_description_length_check(description: &str) -> Result<(), LemmyError> {
if description.len() > 150 {
Err(LemmyError::from_message("site_description_length_overflow"))
} else {
Ok(())
}
}

/// Checks for a honeypot. If this field is filled, fail the rest of the function
pub fn honeypot_check(honeypot: &Option<String>) -> Result<(), LemmyError> {
if honeypot.is_some() && honeypot != &Some(String::new()) {
Expand Down
Loading