fix(testutil/harness.go): set Content-Type header for POST requests

refactor(v1_identities_updateIdentity.ts): simplify ratelimits check logic
unkeyed · Sep 5, 2024 · edbf56e · edbf56e
1 parent 0d341be
commit edbf56e
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/apps/agent/pkg/api/testutil/harness.go b/apps/agent/pkg/api/testutil/harness.go
@@ -123,6 +123,12 @@ func CallRoute[Req any, Res any](t *testing.T, route *routes.Route, headers http
 
 	httpReq := httptest.NewRequest(route.Method(), route.Path(), body)
 	httpReq.Header = headers
+	if httpReq.Header == nil {
+		httpReq.Header = http.Header{}
+	}
+	if route.Method() == http.MethodPost {
+		httpReq.Header.Set("Content-Type", "application/json")
+	}
 
 	mux.ServeHTTP(rr, httpReq)
 	require.NoError(t, err)

diff --git a/apps/api/src/routes/v1_identities_updateIdentity.ts b/apps/api/src/routes/v1_identities_updateIdentity.ts
@@ -147,7 +147,7 @@ export const registerV1IdentitiesUpdateIdentity = (app: App) =>
       });
     }
 
-    if (req.ratelimits && req.ratelimits.length >= 2) {
+    if (req.ratelimits) {
       const uniqueNames = new Set<string>();
       for (const { name } of req.ratelimits) {
         if (uniqueNames.has(name)) {