Skip to content
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

chore(dependencies): bump angular from 1.6.10 to 1.8.0 #9836

Merged
merged 1 commit into from
Apr 9, 2022

Conversation

bliu2368
Copy link
Contributor

@bliu2368 bliu2368 commented Apr 4, 2022

Updating Angular from 1.6.10 to 1.8.0.

Dependencies changed:
Angular 1.6.10 -> 1.8.0
Angular-sanitize 1.6.10 -> 1.8.0
Angular-mocks 1.6.10 -> 1.8.0
Angular-messages 1.6.10 -> 1.8.0

Other changes applied according to Angular migration documentation. See https://docs.angularjs.org/guide/migration

  • PreAssignBindingsEnabled Removed
  • Binding default values wrapped in $onInit
  • Checkbox click event fixed for tests

Angular dependencies are bumped to address security vulnerabilities.
These Vulns include:

Prototype Pollution - fixed in Angular 1.7.9 (https://security.snyk.io/vuln/SNYK-JS-ANGULAR-534884)
Cross-site Scripting - Fixed in Angular 1.8.0 (https://security.snyk.io/vuln/SNYK-JS-ANGULAR-572020)

@mattgogerly mattgogerly added the ready to merge Reviewed and ready for merge label Apr 9, 2022
@mergify mergify bot added the auto merged Merged automatically by a bot label Apr 9, 2022
@mergify mergify bot merged commit 911d5ae into spinnaker:master Apr 9, 2022
jervi added a commit to jervi/deck that referenced this pull request Jan 30, 2023
After the upgrade to Angular 1.8 in spinnaker#9836, the account/regions/cluster picker was missing (e.g. in the Scale Down Cluster stage and the Resize Server Group stage). Quick fix was to wrap the existing controller code in a `$onInit` function.
mergify bot pushed a commit that referenced this pull request Feb 1, 2023
After the upgrade to Angular 1.8 in #9836, the account/regions/cluster picker was missing (e.g. in the Scale Down Cluster stage and the Resize Server Group stage). Quick fix was to wrap the existing controller code in a `$onInit` function.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Reviewed and ready for merge target-release/1.29
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants