-
-
Notifications
You must be signed in to change notification settings - Fork 1
⬆️ Updates @actions/core to v1.9.1 [SECURITY] #553
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
Branch automerge failureThis PR was configured for branch automerge. However, this is not possible, so it has been raised as a PR instead.
|
|
Thanks for the PR! This section of the codebase is owner by https://github.com/AlexRogalskiy/ - if they write a comment saying "LGTM" then it will be merged. |
|
Thanks for opening an issue! Make sure you've followed CONTRIBUTING.md. |
Hello from PR HelperIs your PR ready for review and processing? Mark the PR ready by including If you still have work to do, even after marking this ready. Put the PR on hold by including |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub. |
b4fe220 to
a1e6b17
Compare
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
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.
Scan Summary
| Tool | Critical | High | Medium | Low | Status |
|---|---|---|---|---|---|
| Dependency Scan (universal) | 2 | 21 | 7 | 0 | ❌ |
| Security Audit for Infrastructure | 0 | 0 | 0 | 3 | ✅ |
Recommendation
Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍
a1e6b17 to
56448aa
Compare
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.
Scan Summary
| Tool | Critical | High | Medium | Low | Status |
|---|---|---|---|---|---|
| Dependency Scan (universal) | 2 | 22 | 7 | 0 | ❌ |
| Security Audit for Infrastructure | 0 | 0 | 0 | 3 | ✅ |
Recommendation
Please review the findings from Code scanning alerts before approving this pull request. You can also configure the build rules or add suppressions to customize this bot 👍
56448aa to
834ed44
Compare
Signed-off-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
834ed44 to
b94e7dc
Compare
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.
Scan Summary
| Tool | Critical | High | Medium | Low | Status |
|---|---|---|---|---|---|
| Security Audit for Infrastructure | 0 | 0 | 0 | 3 | ✅ |
Recommendation
Looks good ✔️
This PR contains the following updates:
1.2.7->1.9.1GitHub Vulnerability Alerts
CVE-2022-35954
Impact
The
core.exportVariablefunction uses a well known delimiter that attackers can use to break out of that specific variable and assign values to other arbitrary variables. Workflows that write untrusted values to theGITHUB_ENVfile may cause the path or other environment variables to be modified without the intention of the workflow or action author.Patches
Users should upgrade to
@actions/core v1.9.1.Workarounds
If you are unable to upgrade the
@actions/corepackage, you can modify your action to ensure that any user input does not contain the delimiter_GitHubActionsFileCommandDelimeter_before callingcore.exportVariable.References
More information about setting-an-environment-variable in workflows
If you have any questions or comments about this advisory:
actions/toolkitRelease Notes
actions/toolkit (@actions/core)
v1.9.1core.exportVariablev1.9.0toPosixPath,toWin32PathandtoPlatformPathutilities #1102v1.8.2@actions/http-client#1087v1.8.1@actions/http-clientv1.8.0markdownSummaryextension export in favor ofsummaryv1.7.0markdownSummaryextensionv1.6.0getIDTokenfileparameter toAnnotationPropertiesv1.5.0v1.4.0getMultilineInputfunctionv1.3.0Configuration
📅 Schedule: Branch creation - "" in timezone Europe/Moscow, Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.