Skip to content

Increase job name max length from 12 to 30 characters#267

Merged
guillaume-chervet merged 2 commits intomainfrom
copilot/fix-196248643-610307777-eeb93a76-79dc-4f64-8d64-15c52ab588bd
Apr 15, 2026
Merged

Increase job name max length from 12 to 30 characters#267
guillaume-chervet merged 2 commits intomainfrom
copilot/fix-196248643-610307777-eeb93a76-79dc-4f64-8d64-15c52ab588bd

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 15, 2026

Job name validation was capped at 12 characters, which was too restrictive. This raises the upper bound to 30.

Changes

  • JobEndpoints.cs / JobScheduleEndpoints.cs: Updated IsValidFunctionName guard (> 12> 30) and all associated log/error messages.
// Before
if (functionName.Length < 3 || functionName.Length > 12 || !FunctionNamePattern().IsMatch(functionName))

// After
if (functionName.Length < 3 || functionName.Length > 30 || !FunctionNamePattern().IsMatch(functionName))
  • JobEndpointsTests.cs / JobScheduleEndpointsTests.cs:
    • Replaced 13-char invalid test cases with 31-char ones (13 chars is now valid).
    • Added 30-char boundary cases to the valid-name test sets.

Agent-Logs-Url: https://github.com/SlimPlanet/SlimFaas/sessions/93bb0e28-4c05-44a2-abc0-0e5df7ed0c15

Co-authored-by: guillaume-chervet <52236059+guillaume-chervet@users.noreply.github.com>
Copilot AI changed the title [WIP] Copilot Request Increase job name max length from 12 to 30 characters Apr 15, 2026
Copilot AI requested a review from guillaume-chervet April 15, 2026 13:58
@guillaume-chervet guillaume-chervet marked this pull request as ready for review April 15, 2026 14:00
@sonarqubecloud
Copy link
Copy Markdown

@guillaume-chervet guillaume-chervet merged commit 9780c1b into main Apr 15, 2026
27 checks passed
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.

[SlimFaas] update job name rules

3 participants