Skip to content

fix: Force pod names to be lower case#2009

Merged
joachimvh merged 2 commits intomainfrom
fix/pod-lowercase
Mar 26, 2025
Merged

fix: Force pod names to be lower case#2009
joachimvh merged 2 commits intomainfrom
fix/pod-lowercase

Conversation

@joachimvh
Copy link
Member

📁 Related issues

Closes #1894

✍️ Description

Only relevant for subdomain pods as domains are case-insensitive. But for consistency, suffix pods are also changed.

Only relevant for subdomain pods as domains are case-insensitive.
But for consistency, suffix pods are also changed.
@TallTed
Copy link
Contributor

TallTed commented Mar 21, 2025

The user should be warned that their mixed-case entry is being changed to all-lowercase, as with any change of user input to whatever value is stored.

@joachimvh
Copy link
Member Author

The user should be warned that their mixed-case entry is being changed to all-lowercase, as with any change of user input to whatever value is stored.

That's fair. I'll add a line to the pod creation page.

@joachimvh joachimvh merged commit ca86b93 into main Mar 26, 2025
37 checks passed
@joachimvh joachimvh deleted the fix/pod-lowercase branch March 26, 2025 08:29
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.

Won't work with Upper Case Pod name

2 participants