Open justindbaur opened 3 days ago
Attention: Patch coverage is 28.76712%
with 104 lines
in your changes missing coverage. Please review.
Project coverage is 29.39%. Comparing base (
d7bf0fe
) to head (50f4196
). Report is 8 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 โ 2b14214d-9b3f-4d73-88cb-1892264cd551
Severity | Issue | Source File / Package | Checkmarx Insight |
---|---|---|---|
![]() |
Client_DOM_Open_Redirect | /apps/browser/src/platform/popup/layout/popup-header.component.ts: 29 | Attack Vector |
๐๏ธ Tracking
๐ Objective
Refactor
NotificationsService
so that it is less messaging based and more reactive to the details it cares about. This removes itsupdateConnection
call that if called in the wrong place could cause us to open multiple connections. This is replaced with just astart
call (that is more resilient to when it is ran). It also creates a special foreground instance just for browser that throws an immediate error if any of it's methods are called.This also hopefully carves an extensibility point for dropping in a WebPush implementation a little bit easier.
๐ธ Screenshots
โฐ 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