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

fix(chart): improve security on chart and rename charts' properties #1027

Merged
merged 1 commit into from
Oct 28, 2022

Conversation

VincentMolinie
Copy link
Member

Definition of Done

General

  • Write an explicit title for the Pull Request, following Conventional Commits specification
  • Test manually the implemented changes
  • Validate the code quality (indentation, syntax, style, simplicity, readability)
  • Ensure that Types have been updated according to your changes (if needed)

Security

  • Consider the security impact of the changes made

@VincentMolinie VincentMolinie force-pushed the fix/chart/improve-security-on-charts branch from c2efadc to e14c79a Compare October 28, 2022 08:49
@VincentMolinie VincentMolinie added the security Pull requests that address a security vulnerability label Oct 28, 2022
@VincentMolinie VincentMolinie merged commit 2e4c54e into beta Oct 28, 2022
@VincentMolinie VincentMolinie deleted the fix/chart/improve-security-on-charts branch October 28, 2022 09:11
forest-bot added a commit that referenced this pull request Oct 28, 2022
# [9.0.0-beta.2](v9.0.0-beta.1...v9.0.0-beta.2) (2022-10-28)

### Bug Fixes

* **chart:** improve security on chart and rename charts' properties ([#1027](#1027)) ([2e4c54e](2e4c54e))
@forest-bot
Copy link
Member

🎉 This PR is included in version 9.0.0-beta.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

forest-bot added a commit that referenced this pull request Nov 9, 2022
# [9.0.0](v8.5.14...v9.0.0) (2022-11-09)

### Bug Fixes

* **chart:** improve security on chart and rename charts' properties ([#1027](#1027)) ([2e4c54e](2e4c54e))
* **security:** validate that smart action approvals are launched with the same parameters than the initial trigger ([#1026](#1026)) ([928865e](928865e))

### Features

* **chart:** add support for context variables inside SQL query ([#1028](#1028)) ([6bf678e](6bf678e))
* **chart:** add support for context variables used by Workspaces ([#1029](#1029)) ([dc3009a](dc3009a))

### BREAKING CHANGES

* **security:** drop support of projects that are not using roles

Co-authored-by: Guillaume Gautreau <[email protected]>, Morgan Perre <[email protected]>
@forest-bot
Copy link
Member

🎉 This PR is included in version 9.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
released on @beta released security Pull requests that address a security vulnerability
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants