diff --git a/.changeset/proud-starfishes-walk.md b/.changeset/proud-starfishes-walk.md deleted file mode 100644 index fa13c964cd..0000000000 --- a/.changeset/proud-starfishes-walk.md +++ /dev/null @@ -1,6 +0,0 @@ ---- -'@backstage/plugin-permission-backend-module-policy-allow-all': patch -'@backstage/plugin-permission-backend': patch ---- - -Moved permissionModuleAllowAllPolicy to @backstage/plugin-permission-backend-module-policy-allow-all diff --git a/.changeset/tall-items-clap.md b/.changeset/tall-items-clap.md new file mode 100644 index 0000000000..78f9acd940 --- /dev/null +++ b/.changeset/tall-items-clap.md @@ -0,0 +1,6 @@ +--- +'@backstage/plugin-permission-backend-module-allow-all-policy': minor +'@backstage/plugin-permission-backend': patch +--- + +Moved `permissionModuleAllowAllPolicy` to `@backstage/plugin-permission-backend-module-allow-all-policy` diff --git a/packages/backend-next/package.json b/packages/backend-next/package.json index 4dc9056575..d8c90f6850 100644 --- a/packages/backend-next/package.json +++ b/packages/backend-next/package.json @@ -41,7 +41,7 @@ "@backstage/plugin-lighthouse-backend": "workspace:^", "@backstage/plugin-linguist-backend": "workspace:^", "@backstage/plugin-permission-backend": "workspace:^", - "@backstage/plugin-permission-backend-module-policy-allow-all": "workspace:^", + "@backstage/plugin-permission-backend-module-allow-all-policy": "workspace:^", "@backstage/plugin-permission-common": "workspace:^", "@backstage/plugin-permission-node": "workspace:^", "@backstage/plugin-proxy-backend": "workspace:^", diff --git a/packages/backend-next/src/index.ts b/packages/backend-next/src/index.ts index be9f1a1f53..6f736a2033 100644 --- a/packages/backend-next/src/index.ts +++ b/packages/backend-next/src/index.ts @@ -19,7 +19,7 @@ import { appPlugin } from '@backstage/plugin-app-backend/alpha'; import { catalogPlugin } from '@backstage/plugin-catalog-backend/alpha'; import { kubernetesPlugin } from '@backstage/plugin-kubernetes-backend/alpha'; import { permissionPlugin } from '@backstage/plugin-permission-backend/alpha'; -import { permissionModuleAllowAllPolicy } from '@backstage/plugin-permission-backend-module-policy-allow-all'; +import { permissionModuleAllowAllPolicy } from '@backstage/plugin-permission-backend-module-allow-all-policy'; import { scaffolderPlugin } from '@backstage/plugin-scaffolder-backend/alpha'; import { catalogModuleTemplateKind } from '@backstage/plugin-scaffolder-backend/alpha'; import { searchModuleCatalogCollator } from '@backstage/plugin-search-backend-module-catalog/alpha'; diff --git a/plugins/permission-backend-module-policy-allow-all/README.md b/plugins/permission-backend-module-policy-allow-all/README.md index 5caae189b4..1cdeadcf95 100644 --- a/plugins/permission-backend-module-policy-allow-all/README.md +++ b/plugins/permission-backend-module-policy-allow-all/README.md @@ -1,4 +1,4 @@ -# @backstage/plugin-permission-backend-module-policy-allow-all +# @backstage/plugin-permission-backend-module-allow-all-policy The allow all policy backend module for the permission plugin. diff --git a/plugins/permission-backend-module-policy-allow-all/api-report.md b/plugins/permission-backend-module-policy-allow-all/api-report.md index 4adfd37219..a965e9fa54 100644 --- a/plugins/permission-backend-module-policy-allow-all/api-report.md +++ b/plugins/permission-backend-module-policy-allow-all/api-report.md @@ -1,4 +1,4 @@ -## API Report File for "@backstage/plugin-permission-backend-module-policy-allow-all" +## API Report File for "@backstage/plugin-permission-backend-module-allow-all-policy" > Do not edit this file. It is a report generated by [API Extractor](https://api-extractor.com/). diff --git a/plugins/permission-backend-module-policy-allow-all/package.json b/plugins/permission-backend-module-policy-allow-all/package.json index decb1573e7..71a02539af 100644 --- a/plugins/permission-backend-module-policy-allow-all/package.json +++ b/plugins/permission-backend-module-policy-allow-all/package.json @@ -1,5 +1,5 @@ { - "name": "@backstage/plugin-permission-backend-module-policy-allow-all", + "name": "@backstage/plugin-permission-backend-module-allow-all-policy", "description": "Allow all policy backend module for the permission plugin.", "version": "0.0.0", "main": "src/index.ts", diff --git a/plugins/permission-backend-module-policy-allow-all/src/policy.ts b/plugins/permission-backend-module-policy-allow-all/src/policy.ts index fae693faba..da853e47db 100644 --- a/plugins/permission-backend-module-policy-allow-all/src/policy.ts +++ b/plugins/permission-backend-module-policy-allow-all/src/policy.ts @@ -1,13 +1,3 @@ -import { BackstageIdentityResponse } from '@backstage/plugin-auth-node'; -import { - AuthorizeResult, - PolicyDecision, -} from '@backstage/plugin-permission-common'; -import { - PermissionPolicy, - PolicyQuery, -} from '@backstage/plugin-permission-node'; - /* * Copyright 2023 The Backstage Authors * @@ -23,6 +13,17 @@ import { * See the License for the specific language governing permissions and * limitations under the License. */ + +import { BackstageIdentityResponse } from '@backstage/plugin-auth-node'; +import { + AuthorizeResult, + PolicyDecision, +} from '@backstage/plugin-permission-common'; +import { + PermissionPolicy, + PolicyQuery, +} from '@backstage/plugin-permission-node'; + export class AllowAllPermissionPolicy implements PermissionPolicy { async handle( _request: PolicyQuery, diff --git a/plugins/permission-backend/src/plugin.ts b/plugins/permission-backend/src/plugin.ts index cb13204477..9cf0024e47 100644 --- a/plugins/permission-backend/src/plugin.ts +++ b/plugins/permission-backend/src/plugin.ts @@ -61,7 +61,7 @@ export const permissionPlugin = createBackendPlugin({ const winstonLogger = loggerToWinstonLogger(logger); if (!policies.policy) { throw new Error( - 'No policy module installed! Please install a policy module. If you want to allow all requests, use @backstage/plugin-permission-backend-module-policy-allow-all permissionModuleAllowAllPolicy', + 'No policy module installed! Please install a policy module. If you want to allow all requests, use @backstage/plugin-permission-backend-module-allow-all-policy permissionModuleAllowAllPolicy', ); } diff --git a/yarn.lock b/yarn.lock index 2556f758b5..965ff642e0 100644 --- a/yarn.lock +++ b/yarn.lock @@ -8223,9 +8223,9 @@ __metadata: languageName: unknown linkType: soft -"@backstage/plugin-permission-backend-module-policy-allow-all@workspace:^, @backstage/plugin-permission-backend-module-policy-allow-all@workspace:plugins/permission-backend-module-policy-allow-all": +"@backstage/plugin-permission-backend-module-allow-all-policy@workspace:^, @backstage/plugin-permission-backend-module-allow-all-policy@workspace:plugins/permission-backend-module-policy-allow-all": version: 0.0.0-use.local - resolution: "@backstage/plugin-permission-backend-module-policy-allow-all@workspace:plugins/permission-backend-module-policy-allow-all" + resolution: "@backstage/plugin-permission-backend-module-allow-all-policy@workspace:plugins/permission-backend-module-policy-allow-all" dependencies: "@backstage/backend-common": "workspace:^" "@backstage/backend-plugin-api": "workspace:^" @@ -25833,7 +25833,7 @@ __metadata: "@backstage/plugin-lighthouse-backend": "workspace:^" "@backstage/plugin-linguist-backend": "workspace:^" "@backstage/plugin-permission-backend": "workspace:^" - "@backstage/plugin-permission-backend-module-policy-allow-all": "workspace:^" + "@backstage/plugin-permission-backend-module-allow-all-policy": "workspace:^" "@backstage/plugin-permission-common": "workspace:^" "@backstage/plugin-permission-node": "workspace:^" "@backstage/plugin-proxy-backend": "workspace:^"