Commit 16e4f63
authored
fix: add Osano cookie consent CSP permissions (#463)
* fix: add Osano cookie consent CSP permissions
- Add cmp.osano.com to connect-src for configuration API calls
- Add cmp.osano.com to frame-src for consent UI iframe
- Add www.googletagmanager.com to script-src for GTM integration
Resolves CSP violations preventing Osano cookie consent from functioning properly.
Signed-off-by: ahmedomosanya <[email protected]>
* fix: Add Google Analytics & DoubleClick to connect-src CSP
Add Google Analytics and DoubleClick domains to connect-src directive
to allow Osano and GTM scripts to send analytics beacons:
- www.google-analytics.com - Google Analytics beacons
- analytics.google.com - Google Analytics 4
- www.googletagmanager.com - GTM fetch requests
- googleads.g.doubleclick.net - DoubleClick advertising
- stats.g.doubleclick.net - DoubleClick stats
Addresses CodeRabbit review feedback for complete Osano/GTM integration.
Signed-off-by: ahmedomosanya <[email protected]>
* fix: remove DoubleClick advertising domain from CSP connect-src
- Remove https://googleads.g.doubleclick.net from allowed connect sources
- Keep https://stats.g.doubleclick.net for analytics functionality
- Improves security by reducing CSP attack surface
---------
Signed-off-by: ahmedomosanya <[email protected]>1 parent a3713dd commit 16e4f63
1 file changed
+10
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
43 | 43 | | |
44 | 44 | | |
45 | 45 | | |
46 | | - | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
47 | 52 | | |
48 | 53 | | |
49 | 54 | | |
| |||
54 | 59 | | |
55 | 60 | | |
56 | 61 | | |
57 | | - | |
| 62 | + | |
| 63 | + | |
58 | 64 | | |
59 | 65 | | |
60 | 66 | | |
| |||
106 | 112 | | |
107 | 113 | | |
108 | 114 | | |
109 | | - | |
| 115 | + | |
| 116 | + | |
110 | 117 | | |
111 | 118 | | |
112 | 119 | | |
| |||
0 commit comments