Skip to content

fix(api): add max len for BoardChanges #7519

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

fix(api): add max len for BoardChanges #7519

wants to merge 1 commit into from

Conversation

psychedelicious
Copy link
Collaborator

Summary

Add a max length to the BoardChanges.board_name and BoardChanges.cover_image_name to prevent massive payloads from causing slowdowns or crashes.

Related Issues / Discussions

n/a

QA Instructions

Try to set a long board name or cover image name. It should fail quickly and without impacting the application.

Merge Plan

n/a

Checklist

  • The PR has a short but descriptive title, suitable for a changelog

@github-actions github-actions bot added python PRs that change python files services PRs that change app services labels Jan 5, 2025
@psychedelicious
Copy link
Collaborator Author

Closing in favor of #7515

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hipsterusername hipsterusername hipsterusername approved these changes

@blessedcoolant blessedcoolant Awaiting requested review from blessedcoolant blessedcoolant is a code owner

@brandonrising brandonrising Awaiting requested review from brandonrising

Assignees
No one assigned
Labels
python PRs that change python files services PRs that change app services
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@psychedelicious @hipsterusername