Skip to content

Commit

Permalink
fix: Fixed the admin method set
Browse files Browse the repository at this point in the history
  • Loading branch information
JigarJoshi committed Sep 26, 2022
1 parent 4e4c4b6 commit 298bca8
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion server/request/request.go
Expand Up @@ -42,7 +42,7 @@ const (
)

var (
adminMethods = container.NewHashSet("/tigrisdata.management.v1.Management/createNamespace", "/tigrisdata.management.v1.Management/listNamespaces")
adminMethods = container.NewHashSet("/tigrisdata.management.v1.Management/CreateNamespace", "/tigrisdata.management.v1.Management/ListNamespaces")
)

type RequestMetadataCtxKey struct {
Expand Down
4 changes: 2 additions & 2 deletions server/request/request_test.go
Expand Up @@ -69,8 +69,8 @@ func TestRequestMetadata(t *testing.T) {
})

t.Run("isAdmin test", func(t *testing.T) {
require.True(t, IsAdminApi("/tigrisdata.management.v1.Management/createNamespace"))
require.True(t, IsAdminApi("/tigrisdata.management.v1.Management/listNamespaces"))
require.True(t, IsAdminApi("/tigrisdata.management.v1.Management/CreateNamespace"))
require.True(t, IsAdminApi("/tigrisdata.management.v1.Management/ListNamespaces"))
require.False(t, IsAdminApi("/.HealthAPI/Health"))
require.False(t, IsAdminApi("some-random"))
})
Expand Down

0 comments on commit 298bca8

Please sign in to comment.