Skip to content

[OGUI-1936] Allow extra connect-src values - #3700

Open
isaachilly wants to merge 1 commit into
feature/OGUI-1940/release-build-stepfrom
feature/FRM/OGUI-1936/csp-extra-connect-src
Open

isaachilly wants to merge 1 commit into
feature/OGUI-1940/release-build-stepfrom
feature/FRM/OGUI-1936/csp-extra-connect-src

Conversation

@isaachilly

@isaachilly isaachilly commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

I have JIRA issue created

  • branch and/or PR name(s) includes JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected
  • FLP integration tests were ran successful

Ticket:

Need to allow extra sources in our CSP otherwise Vite's HMR dev server functionality will not work when integrated into ILG.

Code:

HTTP config now takes extraConnectSrc list and appends it to the end of the default CSP policy in Helmet.

Reviewer:

@isaachilly
isaachilly added this pull request to stack #3702 October 23, 2026 14:51
@isaachilly isaachilly self-assigned this Oct 23, 2026
@isaachilly
isaachilly force-pushed the feature/FRM/OGUI-1936/csp-extra-connect-src branch from d9df7d8 to 910cff7 Compare November 10, 2026 15:45
@isaachilly
isaachilly force-pushed the feature/FRM/OGUI-1936/csp-extra-connect-src branch from 910cff7 to ba1d8fa Compare November 10, 2026 15:59
@isaachilly
isaachilly marked this pull request as ready for review November 11, 2026 07:42
@isaachilly
isaachilly requested a review from graduta as a code owner November 11, 2026 07:42
@isaachilly
isaachilly removed this pull request from stack #3702 November 11, 2026 16:55
@isaachilly
isaachilly added this pull request to stack #3734 November 11, 2026 16:56
@isaachilly
isaachilly force-pushed the feature/FRM/OGUI-1936/csp-extra-connect-src branch from ba1d8fa to cbcfc13 Compare November 11, 2026 18:58

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

1 participant