Skip to content

Revert \"Merge pull request #2578 from keboola/pepa/PAT-1538_e2bStreamlit\"#2592

Merged
Matovidlo merged 1 commit intomainfrom
devin/1777464672-revert-pr-2578
Apr 29, 2026
Merged

Revert \"Merge pull request #2578 from keboola/pepa/PAT-1538_e2bStreamlit\"#2592
Matovidlo merged 1 commit intomainfrom
devin/1777464672-revert-pr-2578

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Release Notes

Reverts #2578 — restores the Streamlit WebSocket Origin rewriting workaround in apps-proxy.

Plans for customer communication

None.

Impact analysis

PR #2578 removed a temporary workaround that rewrote the WebSocket Origin header to match the upstream hostname for Streamlit (Tornado) apps. The commit message claimed Streamlit apps were now configured with STREAMLIT_BROWSER_SERVER_ADDRESS / STREAMLIT_BROWSER_SERVER_PORT env vars, but these env vars were never actually configured anywhere.

This caused Tornado to reject WebSocket connections with 403 (Origin mismatch), breaking the /_stcore/stream endpoint across all stacks:

  • /_stcore/stream 403s jumped from 0 → 31,803 in a 4-hour window
  • Streamlit clients retry failed stream connections, amplifying total traffic ~3.5x
  • /_stcore/health and /_stcore/host-config requests jumped from ~16k → ~56k per 4h
  • Most affected environments: kbc-eu-central-1 (22,975 spans), cloud-keboola-groupon (17,243 spans)

This revert restores the Origin rewrite workaround until STREAMLIT_BROWSER_SERVER_ADDRESS / STREAMLIT_BROWSER_SERVER_PORT are properly configured on all Streamlit Data Apps.

Change type

Revert / bugfix

Justification

Production incident — Streamlit Data Apps broken across all stacks after v1.10.1 deployment.

Deployment

Merge & automatic deploy.

Rollback plan

Revert of this PR.

Post release support plan

None.

Link to Devin session: https://app.devin.ai/sessions/a88bf1485da340738418021851aadc44

…lit"

This reverts commit ee88991, reversing
changes made to a40cb25.
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@linear
Copy link
Copy Markdown

linear Bot commented Apr 29, 2026

@Matovidlo Matovidlo merged commit 6463bd7 into main Apr 29, 2026
41 of 43 checks passed
@Matovidlo Matovidlo deleted the devin/1777464672-revert-pr-2578 branch April 29, 2026 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant