Skip to content

fix(validation): add input validation for resource limit fields#9238

Merged
andrasbacsai merged 1 commit intocoollabsio:nextfrom
ShadowArcanist:jean/strict-input-validation-resource-limits
Mar 30, 2026
Merged

fix(validation): add input validation for resource limit fields#9238
andrasbacsai merged 1 commit intocoollabsio:nextfrom
ShadowArcanist:jean/strict-input-validation-resource-limits

Conversation

@ShadowArcanist
Copy link
Copy Markdown
Member

Changes

  • Added strict validation for all input fields on Resource Limits page
  • Added error message with custom text to help the user to understand which exact input field is rejecting the input and what input it expects

Issues

Category

  • Bug fix

Preview

Invalid value for CPU Weight Before:

image

Invalid value for CPU Weight After:

image

Invalid value for Memory limits Before:

image

Invalid value for Memory limits After:

image

AI Assistance

  • AI was used (please describe below)

If AI was used:

  • Tools used: Jean + Claude (Opus 4.6)
  • How extensively: every change on this PR (except the PR description)

Testing

Tried entering valid and invalid values on all input fields on Resource Limits page and Coolify rejects invalid inputs and shows error message with info about which exact input field have invalid data

Contributor Agreement

Important

  • I have read and understood the contributor guidelines. If I have failed to follow any guideline, I understand that this PR may be closed without review.
  • I have searched existing issues and pull requests (including closed ones) to ensure this isn't a duplicate.
  • I have tested all the changes thoroughly with a local development instance of Coolify and I am confident that they will work as expected when a maintainer tests them.

@andrasbacsai
Copy link
Copy Markdown
Member

Thank you for the PR! 💜

@andrasbacsai andrasbacsai merged commit 1efed14 into coollabsio:next Mar 30, 2026
2 checks passed
@andrasbacsai andrasbacsai mentioned this pull request Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants