Open audreyality opened 4 days ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 29.31%. Comparing base (
1495b06
) to head (297e77b
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Checkmarx One โ Scan Summary & Details โ 8bf9690f-a545-4ad0-bfbf-82eb82a53a97
Severity | Issue | Source File / Package | Checkmarx Insight |
---|---|---|---|
![]() |
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... |
![]() |
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... |
Severity | Issue | Source File / Package |
---|---|---|
![]() |
Client_Privacy_Violation | /apps/browser/src/background/runtime.background.ts: 327 |
![]() |
Unpinned Actions Full Length Commit SHA | /build-desktop.yml: 1256 |
![]() |
Unpinned Actions Full Length Commit SHA | /build-desktop.yml: 1210 |
Latest commit ensures that there won't be a performance regression once #9812 merges.
๐ Merged main
to fix the macOS build errors.
๐๏ธ Tracking
https://bitwarden.atlassian.net/browse/PM-6394
๐ Objective
The cache causes a performance regression on the CLI, preventing
bw generate
from exiting immediately after generating a credential.Background: Policy evaluator construction was cached in the
DefaultGeneratorService
to avoid repeated allocations when bridging between observeables and promises. Now that the generator UI holds open a subscription, this code is unnecessary.๐ฆฎ 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