-
Notifications
You must be signed in to change notification settings - Fork 1
[Snyk] Fix for 3 vulnerabilities #9908
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
….json to reduce vulnerabilities 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 passed. No 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 Dev as Developer
participant Snyk as Snyk Security Scanner
participant Repo as GitHub Repository
participant Pkg as package.json
participant Deps as Dependency Tree
participant Yarn as Yarn Package Manager
Dev->>Repo: Authorize Snyk integration
Snyk->>Repo: Scan dependencies
Repo->>Snyk: Return dependency manifest
Snyk->>Deps: Analyze dependency tree
Note over Snyk,Deps: Detects vulnerabilities:<br/>- react-router (Open Redirect)<br/>- @remix-run/router (XSS, Open Redirect)
Snyk->>Snyk: Calculate remediation strategy
Note over Snyk: Upgrade @backstage packages<br/>to pull in patched versions
Snyk->>Repo: Create PR #9908
Snyk->>Pkg: Update dependencies
Note over Pkg: @backstage/backend-common: 0.24.1<br/>@backstage/plugin-scaffolder-backend: 1.24.0<br/>@backstage/plugin-scaffolder-node: 0.2.0
Pkg-->>Deps: Trigger dependency resolution
alt Yarn lock update succeeds
Yarn->>Deps: Resolve new dependency tree
Deps-->>Yarn: Return resolved versions
Yarn->>Repo: Update yarn.lock
else Yarn lock update fails
Yarn-->>Snyk: Lock file update failed
Snyk->>Repo: Add warning to PR
Note over Repo: "Failed to update yarn.lock,<br/>please update manually"
end
Repo->>Dev: Notify of PR creation
Dev->>Repo: Review changes
opt Developer approves
Dev->>Repo: Merge PR
Repo->>Yarn: Install updated dependencies
Yarn->>Deps: Fetch patched packages
Note over Deps: Vulnerabilities resolved
end
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. |
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
🤖 Augment PR SummarySummary: Updates the Confluence-to-Markdown scaffolder backend module’s dependencies to newer released versions of 🤖 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.
| }, | ||
| "dependencies": { | ||
| "@backstage/backend-common": "workspace:^", | ||
| "@backstage/backend-common": "0.24.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.
Pinning @backstage/* deps here to 0.24.1/1.24.0/0.2.0 won’t match the in-repo workspace package versions (e.g. @backstage/plugin-scaffolder-backend is 1.15.2-next.0), so Yarn will likely pull published packages from the registry instead of using the local workspaces. That can lead to duplicate @backstage/* installs and runtime/type mismatches (also applies to the other pinned @backstage/* lines in this file).
🤖 Was this useful? React with 👍 or 👎
| "@backstage/integration": "workspace:^", | ||
| "@backstage/plugin-scaffolder-backend": "workspace:^", | ||
| "@backstage/plugin-scaffolder-node": "workspace:^", | ||
| "@backstage/plugin-scaffolder-backend": "1.24.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.
Snyk has created this PR to fix 3 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):
plugins/scaffolder-backend-module-confluence-to-markdown/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)