-
Notifications
You must be signed in to change notification settings - Fork 1
[Snyk] Fix for 3 vulnerabilities #9910
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
…abilities The following vulnerabilities are fixed with an upgrade: - https://snyk.io/vuln/SNYK-JS-REMIXRUNROUTER-14908530 - https://snyk.io/vuln/SNYK-JS-REACTROUTER-14908286 - https://snyk.io/vuln/SNYK-JS-REMIXRUNROUTER-14908287
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
⛔ 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. |
WalkthroughChangesSequence DiagramThis diagram shows the interactions between components: sequenceDiagram
actor User
participant App as TechDocs CLI<br/>Embedded App
participant CoreAPI as @backstage/core-app-api<br/>(v1.1.0)
participant Router as react-router-dom<br/>(v6.30.3)
participant TechDocs as @backstage/plugin-techdocs<br/>(v0.1.1)
participant TechDocsReact as @backstage/plugin-techdocs-react<br/>(v0.1.0)
participant Integration as @backstage/integration-react<br/>(v0.1.1)
User->>App: Launch TechDocs CLI
activate App
App->>CoreAPI: Initialize app APIs
activate CoreAPI
CoreAPI-->>App: API context ready
deactivate CoreAPI
App->>Router: Setup routing (v6.30.3)
activate Router
Note over Router: Updated from v6.3.0<br/>to v6.30.3
Router-->>App: Routes configured
deactivate Router
User->>App: Navigate to documentation
App->>TechDocs: Load TechDocs plugin
activate TechDocs
TechDocs->>TechDocsReact: Render documentation UI
activate TechDocsReact
TechDocsReact-->>TechDocs: UI components
deactivate TechDocsReact
TechDocs->>Integration: Fetch documentation source
activate Integration
Integration-->>TechDocs: Documentation content
deactivate Integration
TechDocs-->>App: Rendered documentation
deactivate TechDocs
App-->>User: Display documentation
deactivate App
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. |
🤖 Augment PR SummarySummary: Updates dependencies for the TechDocs CLI embedded app to address Snyk-reported vulnerabilities. Changes:
Technical Notes: The PR text indicates 🤖 Was this summary useful? React with 👍 or 👎 |
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.
| "@backstage/cli": "workspace:^", | ||
| "@backstage/config": "workspace:^", | ||
| "@backstage/core-app-api": "workspace:^", | ||
| "@backstage/core-app-api": "1.1.0", |
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.
Pinning @backstage/* dependencies here to older published versions (e.g. @backstage/core-app-api 1.1.0) while the rest remain workspace:^ will likely make Yarn pull registry packages that don’t match the workspace versions, leading to duplicate Backstage deps and potential build/runtime breakage. Consider keeping these as workspace:^ or otherwise ensuring the specified versions satisfy the local workspace package versions (also applies to the other pinned @backstage/* entries in this file).
🤖 Was this useful? React with 👍 or 👎
| "react": "^17.0.2", | ||
| "react-dom": "^17.0.2", | ||
| "react-router-dom": "^6.3.0", | ||
| "react-router-dom": "^6.30.3", |
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.
The PR description notes yarn.lock wasn’t updated; without a lockfile refresh the repo may still resolve the vulnerable transitive versions and builds can be non-reproducible. Consider running yarn and committing the resulting lockfile update alongside this dependency bump.
🤖 Was this useful? React with 👍 or 👎
Snyk has created this PR to fix 3 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):
packages/techdocs-cli-embedded-app/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-REMIXRUNROUTER-14908530
SNYK-JS-REACTROUTER-14908286
SNYK-JS-REMIXRUNROUTER-14908287
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:
🦉 Open Redirect
🦉 Cross-site Scripting (XSS)