Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(sdk/go): ensure empty string cannot be passed as path parameters #3722

Merged
merged 7 commits into from
Dec 16, 2024
Prev Previous commit
chore(build): remove now unnecessary setting of namespace default in …
…test
  • Loading branch information
GeorgeMac committed Dec 16, 2024
commit 69622724abe627df00e979065f1e0c0b8d488a62
3 changes: 0 additions & 3 deletions build/testing/integration/authz/auth.go
Original file line number Diff line number Diff line change
Expand Up @@ -281,9 +281,6 @@ func (s clientCallSet) assert(t *testing.T, ctx context.Context, client sdk.SDK)
type clientCall func(*testing.T, context.Context, sdk.SDK) error

func GetNamespace(in *flipt.GetNamespaceRequest) clientCall {
if in.GetNamespaceKey() == "" {
in.Key = "default"
}
return func(t *testing.T, ctx context.Context, s sdk.SDK) error {
_, err := s.Flipt().GetNamespace(ctx, in)
return fmt.Errorf("GetNamespace: %w", err)
Expand Down
Loading