diff --git a/coderd/httpapi/httpapi.go b/coderd/httpapi/httpapi.go
index c0dcc7cde985e..a83f4b6d19cfc 100644
--- a/coderd/httpapi/httpapi.go
+++ b/coderd/httpapi/httpapi.go
@@ -46,7 +46,7 @@ func init() {
valid := NameValid(str)
return valid == nil
}
- for _, tag := range []string{"username", "organization_name", "template_name", "group_name", "workspace_name", "oauth2_app_name"} {
+ for _, tag := range []string{"username", "organization_name", "template_name", "workspace_name", "oauth2_app_name"} {
err := Validate.RegisterValidation(tag, nameValidator)
if err != nil {
panic(err)
@@ -96,6 +96,20 @@ func init() {
if err != nil {
panic(err)
}
+
+ groupNameValidator := func(fl validator.FieldLevel) bool {
+ f := fl.Field().Interface()
+ str, ok := f.(string)
+ if !ok {
+ return false
+ }
+ valid := GroupNameValid(str)
+ return valid == nil
+ }
+ err = Validate.RegisterValidation("group_name", groupNameValidator)
+ if err != nil {
+ panic(err)
+ }
}
// Is404Error returns true if the given error should return a 404 status code.
diff --git a/coderd/httpapi/name.go b/coderd/httpapi/name.go
index c9f926d4b3b42..98bbf50f46861 100644
--- a/coderd/httpapi/name.go
+++ b/coderd/httpapi/name.go
@@ -96,6 +96,23 @@ func UserRealNameValid(str string) error {
return nil
}
+// GroupNameValid returns whether the input string is a valid group name.
+func GroupNameValid(str string) error {
+ // 36 is to support using UUIDs as the group name.
+ if len(str) > 36 {
+ return xerrors.New("must be <= 36 characters")
+ }
+ // Avoid conflicts with routes like /groups/new and /groups/create.
+ if str == "new" || str == "create" {
+ return xerrors.Errorf("cannot use %q as a name", str)
+ }
+ matched := UsernameValidRegex.MatchString(str)
+ if !matched {
+ return xerrors.New("must be alphanumeric with hyphens")
+ }
+ return nil
+}
+
// NormalizeUserRealName normalizes a user name such that it will pass
// validation by UserRealNameValid. This is done to avoid blocking
// little Bobby Whitespace from using Coder.
diff --git a/enterprise/coderd/groups_test.go b/enterprise/coderd/groups_test.go
index 986b308b86fef..dc4df7b8a866f 100644
--- a/enterprise/coderd/groups_test.go
+++ b/enterprise/coderd/groups_test.go
@@ -159,7 +159,7 @@ func TestPatchGroup(t *testing.T) {
const displayName = "foobar"
ctx := testutil.Context(t, testutil.WaitLong)
group, err := userAdminClient.CreateGroup(ctx, user.OrganizationID, codersdk.CreateGroupRequest{
- Name: "hi",
+ Name: "ff7dcee2-e7c4-4bc4-a9e4-84870770e4c5", // GUID should fit.
AvatarURL: "https://example.com",
QuotaAllowance: 10,
DisplayName: "",
@@ -168,14 +168,14 @@ func TestPatchGroup(t *testing.T) {
require.Equal(t, 10, group.QuotaAllowance)
group, err = userAdminClient.PatchGroup(ctx, group.ID, codersdk.PatchGroupRequest{
- Name: "bye",
+ Name: "ddd502d2-2984-4724-b5bf-1109a4d7462d", // GUID should fit.
AvatarURL: ptr.Ref("https://google.com"),
QuotaAllowance: ptr.Ref(20),
DisplayName: ptr.Ref(displayName),
})
require.NoError(t, err)
require.Equal(t, displayName, group.DisplayName)
- require.Equal(t, "bye", group.Name)
+ require.Equal(t, "ddd502d2-2984-4724-b5bf-1109a4d7462d", group.Name)
require.Equal(t, "https://google.com", group.AvatarURL)
require.Equal(t, 20, group.QuotaAllowance)
})
pFad - Phonifier reborn
Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.
Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.
Alternative Proxies:
Alternative Proxy
pFad Proxy
pFad v3 Proxy
pFad v4 Proxy