-
Notifications
You must be signed in to change notification settings - Fork 1
[Snyk] Fix for 4 vulnerabilities #9918
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
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 - https://snyk.io/vuln/SNYK-JS-UNDICI-14943963
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
participant Client
participant GraphQLBackend as GraphQL Backend Plugin
participant BackendCommon as @backstage/backend-common<br/>(v0.24.1)
participant CatalogGraphQL as @backstage/plugin-catalog-graphql<br/>(v0.1.1)
participant ApolloServer as Apollo Server
Note over GraphQLBackend: Plugin Initialization
GraphQLBackend->>BackendCommon: Import backend utilities
BackendCommon-->>GraphQLBackend: Provide common services
GraphQLBackend->>CatalogGraphQL: Import catalog schema & resolvers
CatalogGraphQL-->>GraphQLBackend: Return GraphQL type definitions
GraphQLBackend->>ApolloServer: Initialize with schema
activate ApolloServer
ApolloServer-->>GraphQLBackend: Server instance ready
Note over GraphQLBackend,ApolloServer: Runtime Query Execution
Client->>GraphQLBackend: GraphQL Query Request
GraphQLBackend->>ApolloServer: Forward query
ApolloServer->>CatalogGraphQL: Execute resolver
activate CatalogGraphQL
CatalogGraphQL->>BackendCommon: Use backend services
BackendCommon-->>CatalogGraphQL: Return data
CatalogGraphQL-->>ApolloServer: Resolved data
deactivate CatalogGraphQL
ApolloServer-->>GraphQLBackend: Query result
GraphQLBackend-->>Client: JSON response
deactivate ApolloServer
Note over GraphQLBackend: Dependencies now use<br/>fixed versions instead of<br/>workspace references
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 the GraphQL backend plugin’s Backstage dependencies to address Snyk-reported vulnerabilities. 🤖 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": { | ||
| "@apollo/server": "^4.0.0", | ||
| "@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.
In a Backstage monorepo, @backstage/backend-common is typically consumed via workspace:^; pinning 0.24.1 will pull from the registry and can cause duplicate/mismatched packages (also applies to @backstage/plugin-catalog-graphql). Can you confirm this is intentional?
🤖 Was this useful? React with 👍 or 👎
| "@backstage/backend-common": "0.24.1", | ||
| "@backstage/config": "workspace:^", | ||
| "@backstage/plugin-catalog-graphql": "workspace:^", | ||
| "@backstage/plugin-catalog-graphql": "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.
Snyk has created this PR to fix 4 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):
plugins/graphql-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-REMIXRUNROUTER-14908530
SNYK-JS-REACTROUTER-14908286
SNYK-JS-REMIXRUNROUTER-14908287
SNYK-JS-UNDICI-14943963
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)
🦉 Allocation of Resources Without Limits or Throttling