@backstage/plugin-auth-backend 0.10.2 → 0.11.0

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # @backstage/plugin-auth-backend
 
+## 0.11.0
+
+### Minor Changes
+
+- 3884bf0348: **BREAKING**: The default sign-in resolvers for all providers, if you choose to
+  use them, now emit the token `sub` and `ent` claims on the standard,
+  all-lowercase form, instead of the mixed-case form. The mixed-case form causes
+  problems for implementations that naively do string comparisons on refs. The end
+  result is that you may for example see your Backstage token `sub` claim now
+  become `'user:default/my-id'` instead of `'user:default/My-ID'`.
+
+  On a related note, specifically the SAML provider now correctly issues both
+  `sub` and `ent` claims, and on the full entity ref form instead of the short
+  form with only the ID.
+
+  **NOTE**: For a long time, it has been strongly recommended that you provide
+  your own sign-in resolver instead of using the builtin ones, and that will
+  become mandatory in the future.
+
+### Patch Changes
+
+- d64b8d3678: chore(deps): bump `minimatch` from 3.0.4 to 5.0.0
+- 6e1cbc12a6: Updated according to the new `getEntityFacets` catalog API method
+- 919cf2f836: Minor updates to match the new `targetRef` field of relations, and to stop consuming the `target` field
+- Updated dependencies
+  - @backstage/backend-common@0.11.0
+  - @backstage/catalog-model@0.11.0
+  - @backstage/catalog-client@0.7.2
+  - @backstage/plugin-auth-node@0.1.3
+
 ## 0.10.2
 
 ### Patch Changes

package/dist/index.cjs.js

@@ -607,10 +607,10 @@ class CatalogIdentityClient {
     }
     const memberOf = entities.flatMap((e) => {
       var _a, _b;
-      return (_b = (_a = e.relations) == null ? void 0 : _a.filter((r) => r.type === catalogModel.RELATION_MEMBER_OF).map((r) => r.target)) != null ? _b : [];
+      return (_b = (_a = e.relations) == null ? void 0 : _a.filter((r) => r.type === catalogModel.RELATION_MEMBER_OF).map((r) => r.targetRef)) != null ? _b : [];
     });
     const newEntityRefs = [
-      ...new Set(resolvedEntityRefs.concat(memberOf).map(catalogModel.stringifyEntityRef))
+      ...new Set(resolvedEntityRefs.map(catalogModel.stringifyEntityRef).concat(memberOf))
     ];
     logger == null ? void 0 : logger.debug(`Found catalog membership: ${newEntityRefs.join()}`);
     return newEntityRefs;
@@ -620,7 +620,7 @@ class CatalogIdentityClient {
 function getEntityClaims(entity) {
   var _a, _b;
   const userRef = catalogModel.stringifyEntityRef(entity);
-  const membershipRefs = (_b = (_a = entity.relations) == null ? void 0 : _a.filter((r) => r.type === catalogModel.RELATION_MEMBER_OF && r.target.kind.toLocaleLowerCase("en-US") === "group").map((r) => catalogModel.stringifyEntityRef(r.target))) != null ? _b : [];
+  const membershipRefs = (_b = (_a = entity.relations) == null ? void 0 : _a.filter((r) => r.type === catalogModel.RELATION_MEMBER_OF && r.targetRef.startsWith("group:")).map((r) => r.targetRef)) != null ? _b : [];
   return {
     sub: userRef,
     ent: [userRef, ...membershipRefs]
@@ -1261,10 +1261,15 @@ class GithubAuthProvider {
 const githubDefaultSignInResolver = async (info, ctx) => {
   const { fullProfile } = info.result;
   const userId = fullProfile.username || fullProfile.id;
+  const entityRef = catalogModel.stringifyEntityRef({
+    kind: "User",
+    namespace: catalogModel.DEFAULT_NAMESPACE,
+    name: userId
+  });
   const token = await ctx.tokenIssuer.issueToken({
     claims: {
-      sub: `user:default/${userId}`,
-      ent: [`user:default/${userId}`]
+      sub: entityRef,
+      ent: [entityRef]
     }
   });
   return { id: userId, token };
@@ -1333,8 +1338,16 @@ const gitlabDefaultSignInResolver = async (info, ctx) => {
   if (profile.email) {
     id = profile.email.split("@")[0];
   }
+  const entityRef = catalogModel.stringifyEntityRef({
+    kind: "User",
+    namespace: catalogModel.DEFAULT_NAMESPACE,
+    name: id
+  });
   const token = await ctx.tokenIssuer.issueToken({
-    claims: { sub: `user:default/${id}`, ent: [`user:default/${id}`] }
+    claims: {
+      sub: entityRef,
+      ent: [entityRef]
+    }
   });
   return { id, token };
 };
@@ -1566,8 +1579,16 @@ const googleDefaultSignInResolver = async (info, ctx) => {
     ctx.logger.warn(`Failed to look up user, ${error}, falling back to allowing login based on email pattern, this will probably break in the future`);
     userId = profile.email.split("@")[0];
   }
+  const entityRef = catalogModel.stringifyEntityRef({
+    kind: "User",
+    namespace: catalogModel.DEFAULT_NAMESPACE,
+    name: userId
+  });
   const token = await ctx.tokenIssuer.issueToken({
-    claims: { sub: `user:default/${userId}`, ent: [`user:default/${userId}`] }
+    claims: {
+      sub: entityRef,
+      ent: [entityRef]
+    }
   });
   return { id: userId, token };
 };
@@ -1723,10 +1744,15 @@ const microsoftDefaultSignInResolver = async (info, ctx) => {
     throw new Error("Profile contained no email");
   }
   const userId = profile.email.split("@")[0];
+  const entityRef = catalogModel.stringifyEntityRef({
+    kind: "User",
+    namespace: catalogModel.DEFAULT_NAMESPACE,
+    name: userId
+  });
   const token = await ctx.tokenIssuer.issueToken({
     claims: {
-      sub: `user:default/${userId}`,
-      ent: [`user:default/${userId}`]
+      sub: entityRef,
+      ent: [entityRef]
     }
   });
   return { id: userId, token };
@@ -1868,14 +1894,22 @@ class OAuth2AuthProvider {
     return Buffer.from(`${clientID}:${clientSecret}`).toString("base64");
   }
 }
-const oAuth2DefaultSignInResolver$1 = async (info, ctx) => {
+const oAuth2DefaultSignInResolver = async (info, ctx) => {
   const { profile } = info;
   if (!profile.email) {
     throw new Error("Profile contained no email");
   }
   const userId = profile.email.split("@")[0];
+  const entityRef = catalogModel.stringifyEntityRef({
+    kind: "User",
+    namespace: catalogModel.DEFAULT_NAMESPACE,
+    name: userId
+  });
   const token = await ctx.tokenIssuer.issueToken({
-    claims: { sub: `user:default/${userId}`, ent: [`user:default/${userId}`] }
+    claims: {
+      sub: entityRef,
+      ent: [entityRef]
+    }
   });
   return { id: userId, token };
 };
@@ -1906,7 +1940,7 @@ const createOAuth2Provider = (options) => {
     const authHandler = (options == null ? void 0 : options.authHandler) ? options.authHandler : async ({ fullProfile, params }) => ({
       profile: makeProfileInfo(fullProfile, params.id_token)
     });
-    const signInResolverFn = (_c = (_b = options == null ? void 0 : options.signIn) == null ? void 0 : _b.resolver) != null ? _c : oAuth2DefaultSignInResolver$1;
+    const signInResolverFn = (_c = (_b = options == null ? void 0 : options.signIn) == null ? void 0 : _b.resolver) != null ? _c : oAuth2DefaultSignInResolver;
     const signInResolver = (info) => signInResolverFn(info, {
       catalogIdentityClient,
       tokenIssuer,
@@ -2102,16 +2136,21 @@ class OidcAuthProvider {
     return response;
   }
 }
-const oAuth2DefaultSignInResolver = async (info, ctx) => {
+const oidcDefaultSignInResolver = async (info, ctx) => {
   const { profile } = info;
   if (!profile.email) {
     throw new Error("Profile contained no email");
   }
   const userId = profile.email.split("@")[0];
+  const entityRef = catalogModel.stringifyEntityRef({
+    kind: "User",
+    namespace: catalogModel.DEFAULT_NAMESPACE,
+    name: userId
+  });
   const token = await ctx.tokenIssuer.issueToken({
     claims: {
-      sub: `user:default/${userId}`,
-      ent: [`user:default/${userId}`]
+      sub: entityRef,
+      ent: [entityRef]
     }
   });
   return { id: userId, token };
@@ -2146,7 +2185,7 @@ const createOidcProvider = (options) => {
         picture: userinfo.picture
       }
     });
-    const signInResolverFn = (_b = (_a = options == null ? void 0 : options.signIn) == null ? void 0 : _a.resolver) != null ? _b : oAuth2DefaultSignInResolver;
+    const signInResolverFn = (_b = (_a = options == null ? void 0 : options.signIn) == null ? void 0 : _a.resolver) != null ? _b : oidcDefaultSignInResolver;
     const signInResolver = (info) => signInResolverFn(info, {
       catalogIdentityClient,
       tokenIssuer,
@@ -2281,8 +2320,16 @@ const oktaDefaultSignInResolver = async (info, ctx) => {
     throw new Error("Okta profile contained no email");
   }
   const userId = profile.email.split("@")[0];
+  const entityRef = catalogModel.stringifyEntityRef({
+    kind: "User",
+    namespace: catalogModel.DEFAULT_NAMESPACE,
+    name: userId
+  });
   const token = await ctx.tokenIssuer.issueToken({
-    claims: { sub: `user:default/${userId}`, ent: [`user:default/${userId}`] }
+    claims: {
+      sub: entityRef,
+      ent: [entityRef]
+    }
   });
   return { id: userId, token };
 };
@@ -2520,8 +2567,16 @@ class SamlAuthProvider {
 }
 const samlDefaultSignInResolver = async (info, ctx) => {
   const id = info.result.fullProfile.nameID;
+  const entityRef = catalogModel.stringifyEntityRef({
+    kind: "User",
+    namespace: catalogModel.DEFAULT_NAMESPACE,
+    name: id
+  });
   const token = await ctx.tokenIssuer.issueToken({
-    claims: { sub: id }
+    claims: {
+      sub: entityRef,
+      ent: [entityRef]
+    }
   });
   return { id, token };
 };