-
Notifications
You must be signed in to change notification settings - Fork 1
[Snyk] Security upgrade @backstage/catalog-model from 0.0.0-use.local to 0.1.1 #6354
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
base: master
Are you sure you want to change the base?
Conversation
…ties The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-ROLLUP-8073097
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
Reviewer's GuideReplaces local workspace reference of @backstage/catalog-model with an explicit 0.1.1 version to remediate a medium-severity XSS vulnerability and prompt a manual lockfile update. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
⛔ Snyk checks have failed. 1 issues have been found so far.
💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse. |
|
No dependency changes detected. Learn more about Socket for GitHub. 👍 No dependency changes detected in pull request |
| "@backstage/backend-tasks": "workspace:^", | ||
| "@backstage/catalog-client": "workspace:^", | ||
| "@backstage/catalog-model": "workspace:^", | ||
| "@backstage/catalog-model": "0.1.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Replacing a workspace reference with a fixed version (0.1.1) breaks the monorepo's dependency management pattern. This change could lead to version conflicts with other packages that expect to use the same workspace-managed version of @backstage/catalog-model.
Additionally, the PR description indicates that the yarn.lock file failed to update, which must be resolved before merging.
Consider either:
- Updating all instances of this package throughout the monorepo to the same version
- Maintaining the workspace reference and updating the package version in the root package.json instead
This approach would preserve compatibility while addressing the security vulnerability.
| "@backstage/catalog-model": "0.1.1", | |
| "@backstage/catalog-model": "workspace:^", |
Spotted by Diamond
Is this helpful? React 👍 or 👎 to let us know.
|
This PR has been automatically marked as stale because it has not had recent activity from the author. It will be closed if no further activity occurs. If the PR was closed and you want it re-opened, let us know and we'll re-open the PR so that you can continue the contribution! |
WalkthroughThis pull request updates the dependency management for the tech-insights-backend plugin by changing the Changes
Sequence DiagramThis diagram shows the interactions between components: sequenceDiagram
participant Dev as Developer
participant Package as tech-insights-backend
participant CatalogModel as @backstage/catalog-model
participant Registry as Package Registry
Note over Dev,Registry: Dependency Version Change
Dev->>Package: Update package.json
Note over Package: Change catalog-model<br/>from workspace:^ to 0.1.1
Dev->>Package: Run package install
Package->>Registry: Request @backstage/catalog-model@0.1.1
Registry-->>Package: Return specific version 0.1.1
Package->>CatalogModel: Install pinned version
Note over Package,CatalogModel: Now using fixed version 0.1.1<br/>instead of workspace reference
Note for WindsurfPlease change the default marketplace provider to the following in the windsurf settings:Marketplace Extension Gallery Service URL: https://marketplace.visualstudio.com/_apis/public/gallery Marketplace Gallery Item URL: https://marketplace.visualstudio.com/items Entelligence.ai can learn from your feedback. Simply add 👍 / 👎 emojis to teach it your preferences. More shortcuts belowEmoji Descriptions:
Interact with the Bot:
Also you can trigger various commands with the bot by doing The current supported commands are
More commands to be added soon. |
Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):
plugins/tech-insights-backend/package.jsonNote for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the
.yarn/cache/directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to runyarnto update the contents of the./yarn/cachedirectory.If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:
SNYK-JS-ROLLUP-8073097
Important
Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic
Learn how to fix vulnerabilities with free interactive lessons:
🦉 Cross-site Scripting (XSS)
Summary by Sourcery
Bug Fixes:
EntelligenceAI PR Summary
This PR locks the
@backstage/catalog-modeldependency to a specific version in the tech-insights-backend plugin.workspace:^to0.1.1inplugins/tech-insights-backend/package.json