Closed MGibson1 closed 2 days ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 29.31%. Comparing base (
c35bbc5
) to head (aa614c3
). Report is 11 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One โ Scan Summary & Details โ dab37d9e-b265-400b-94bc-f395fc8bde22
Severity | Issue | Source File / Package | Checkmarx Insight |
---|---|---|---|
Unpinned Actions Full Length Commit SHA | /build-desktop.yml: 1222 | Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps... | |
Unpinned Actions Full Length Commit SHA | /build-desktop.yml: 1268 | Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps... | |
Client_DOM_Open_Redirect | /apps/browser/src/platform/popup/layout/popup-header.component.ts: 29 | Attack Vector | |
Client_Use_Of_Iframe_Without_Sandbox | /apps/web/src/connectors/duo.ts: 8 | Attack Vector |
Severity | Issue | Source File / Package |
---|---|---|
Unpinned Actions Full Length Commit SHA | /build-desktop.yml: 1256 | |
Unpinned Actions Full Length Commit SHA | /build-desktop.yml: 1210 |
Nice, I noticed the build failures and was about to open a PR when I saw you were quicker hah.
We're still missing setuptools
in the MacOS Package Dev Release Asset
job, but otherwise LGTM
๐ Objective
Fixes missed mac os build runners from #8073.
Mac runners have progressed above python 3.12, which removed
distutils
. We need to ensure it's installed for node-gyp compilations by installing the setuptools package.โฐ Reminders before review
๐ฆฎ Reviewer guidelines
:+1:
) or similar for great changes:memo:
) or โน๏ธ (:information_source:
) for notes or general info:question:
) for questions:thinking:
) or ๐ญ (:thought_balloon:
) for more open inquiry that's not quite a confirmed issue and could potentially benefit from discussion:art:
) for suggestions / improvements:x:
) or โ ๏ธ (:warning:
) for more significant problems or concerns needing attention:seedling:
) or โป๏ธ (:recycle:
) for future improvements or indications of technical debt:pick:
) for minor or nitpick changes