Accept truthy values for multiple claims.

Author: taeold

spec/v2/providers/https.spec.ts

@@ -577,14 +577,14 @@ describe("onCall", () => {
         const validResp = await runHandler(func, request({ auth: { pro: true, eap: true } }));
         expect(validResp.status).to.equal(200);
 
-        const missingResp = await runHandler(func, request({ auth: { pro: true } }));
-        expect(missingResp.status).to.equal(403);
-
-        const wrongTypeResp = await runHandler(
+        const truthyResp = await runHandler(
           func,
-          request({ auth: { pro: "true", eap: "true" } })
+          request({ auth: { pro: "true", eap: "abc" } })
         );
-        expect(wrongTypeResp.status).to.equal(403);
+        expect(truthyResp.status).to.equal(200);
+
+        const missingResp = await runHandler(func, request({ auth: { pro: true } }));
+        expect(missingResp.status).to.equal(403);
 
         const noClaimResp = await runHandler(func, request({ auth: {} }));
         expect(noClaimResp.status).to.equal(403);

src/v2/providers/https.ts

@@ -61,11 +61,11 @@ export interface HttpsOptions extends Omit<GlobalOptions, "region" | "enforceApp
 
   /** HTTP functions can override global options and can specify multiple regions to deploy to. */
   region?:
-    | SupportedRegion
-    | string
-    | Array<SupportedRegion | string>
-    | Expression<string>
-    | ResetValue;
+  | SupportedRegion
+  | string
+  | Array<SupportedRegion | string>
+  | Expression<string>
+  | ResetValue;
 
   /** If true, allows CORS on requests to this function.
    * If this is a `string` or `RegExp`, allows requests from domains that match the provided value.
@@ -251,7 +251,7 @@ export function hasClaim(
     claimsToCheck[claimOrClaims] = value;
   } else if (Array.isArray(claimOrClaims)) {
     for (const claim of claimOrClaims) {
-      claimsToCheck[claim] = true;
+      claimsToCheck[claim] = undefined;
     }
   } else {
     claimsToCheck = claimOrClaims;