Skip to content

build: update dependency sass_api to v17.6.0#3697

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
angular-robot:ng-renovate/sass-api-17-x
May 22, 2026
Merged

build: update dependency sass_api to v17.6.0#3697
alan-agius4 merged 1 commit into
angular:mainfrom
angular-robot:ng-renovate/sass-api-17-x

Conversation

@angular-robot
Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Type Update Change
sass_api dependencies minor 17.5.017.6.0

  • If you want to rebase/retry this PR, check this box

See associated pull request for more information.
@angular-robot angular-robot added action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only] labels May 22, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the versions of the sass and sass_api packages in pubspec.yaml and pubspec.lock. The reviewer identified a high-risk security vulnerability related to potential dependency confusion attacks, noting that the specified versions for both packages are not available on the official pub.dev registry.

# Add regular dependencies here.
dependencies:
sass_api: 17.5.0
sass_api: 17.6.0
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

security-high high

The version "17.6.0" for "sass_api" does not exist on the official pub.dev registry (the latest version is "1.0.0"). This discrepancy is a significant security concern as it may indicate a dependency confusion attack where a malicious package is published with a higher version number to a public registry. Additionally, the dependency is pinned to a specific version, which is inconsistent with the semantic version ranges (e.g., "^7.0.0") used for other dependencies in this file.

url: "https://pub.dev"
source: hosted
version: "1.99.0"
version: "1.100.0"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

security-high high

The version "1.100.0" for the "sass" package is inconsistent with the official release versions on pub.dev (currently at "1.83.x"). This unusual version jump suggests a potential security risk related to dependency confusion.

@alan-agius4 alan-agius4 merged commit ef4f555 into angular:main May 22, 2026
17 of 18 checks passed
@alan-agius4
Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

@angular-robot angular-robot deleted the ng-renovate/sass-api-17-x branch May 22, 2026 10:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants