Page MenuHomeElementl

[dagit] Handle WS issue in CSP for Safari
ClosedPublic

Authored by dish on Jul 22 2021, 5:40 PM.

Details

Summary

The connect-src: 'self' configuration in the CSP is failing in Safari, which is rejecting WebSocket connections. Add ws: and wss:, which is really unfortunate. I'd like to fix Safari temporarily with this, but would really prefer a better long-term solution than this.

Test Plan

Test dev and prod builds in Chrome, Firefox, Safari. Verify that Dagit loads and performs WS connections correctly.

Diff Detail

Repository
R1 dagster
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dish published this revision for review.Jul 22 2021, 5:46 PM
This revision is now accepted and ready to land.Jul 22 2021, 6:03 PM
This revision was landed with ongoing or failed builds.Jul 22 2021, 6:03 PM
This revision was automatically updated to reflect the committed changes.