Skip to content

Conversation

wkaskie
Copy link

@wkaskie wkaskie commented Nov 26, 2024

PR Checklist

Please check to confirm your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • [ x ] Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Currently, enabling SRI will default to the sha384 algorithm. This is not always available on the server (s3 supports up to sha256, for example.)

Issue Number: 28946

What is the new behavior?

The modifications expose a new configuration property to match that of the webpack-subresource-integrity hashFuncNames which accepts an array of strings, leveraging the highest available. This allows users to set sha256 or sha512.

Does this PR introduce a breaking change?

  • Yes
  • [ x ] No

Other information

This comment was marked as resolved.

@angular-robot angular-robot bot added detected: feature PR contains a feature commit area: @angular/cli labels Nov 26, 2024
@wkaskie wkaskie force-pushed the wkaskie/CustomSriHash branch from b995055 to d9d6155 Compare November 26, 2024 15:07
@wkaskie wkaskie force-pushed the wkaskie/CustomSriHash branch from d9d6155 to abd249a Compare November 26, 2024 15:18
@alan-agius4 alan-agius4 linked an issue Nov 28, 2024 that may be closed by this pull request
@alan-agius4
Copy link
Collaborator

Thanks for your contribution.
However please see discussion in #28946

@wkaskie
Copy link
Author

wkaskie commented Dec 2, 2024

@alan-agius4 I was AFK for the holiday weekend. I responded to the issue with more details.

I've been asked to "just fork" the repository and use a custom build, but you know as well as I do what a horrible idea that is!

Thanks for the review!

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Jan 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area: @angular/cli detected: feature PR contains a feature commit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Allow for customized SRI Algorithm

2 participants