[PR #165] [MERGED] fix(sse): auth header #3240

New Issue

GiteaMirror · 2026-06-17T01:19:03-05:00

GiteaMirror commented

2026-06-17 01:19:03 -05:00

📋 Pull Request Information

Original PR: https://github.com/reconurge/flowsint/pull/165
Author: @dextmorgn
Created: 6/3/2026
Status: ✅ Merged
Merged: 6/3/2026
Merged by: @dextmorgn

Base: main ← Head: fix/sse-auth-header

📝 Commits (6)

078ac77 build(app): add fetch-event-source + vitest test runner
6e5f26a feat(app): add connectSSE helper with header auth and backoff
d7f21e9 refactor(app): stream logs via connectSSE (token out of URL)
4a75d6a refactor(app): stream graph status via connectSSE (token out of URL)
9db1e67 test(api): SSE auth-gate and endpoint-removal tests (red)
b317e08 feat(api): authenticate SSE via header, drop query-param auth and dead route

📊 Changes

11 files changed (+555 additions, -178 deletions)

View changed files

📝 flowsint-api/app/api/deps.py (+1 -41)
📝 flowsint-api/app/api/routes/events.py (+4 -50)
➕ flowsint-api/tests/conftest.py (+38 -0)
➕ flowsint-api/tests/test_events_auth.py (+50 -0)
📝 flowsint-app/package.json (+4 -1)
➕ flowsint-app/src/api/sse.test.ts (+117 -0)
➕ flowsint-app/src/api/sse.ts (+72 -0)
📝 flowsint-app/src/hooks/use-events.ts (+13 -36)
📝 flowsint-app/src/hooks/use-graph-refresh.ts (+28 -47)
➕ flowsint-app/vitest.config.ts (+13 -0)
📝 yarn.lock (+215 -3)

📄 Description

fixes https://github.com/reconurge/flowsint/issues/145
closes https://github.com/reconurge/flowsint/pull/162

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/reconurge/flowsint/pull/165 **Author:** [@dextmorgn](https://github.com/dextmorgn) **Created:** 6/3/2026 **Status:** ✅ Merged **Merged:** 6/3/2026 **Merged by:** [@dextmorgn](https://github.com/dextmorgn) **Base:** `main` ← **Head:** `fix/sse-auth-header` --- ### 📝 Commits (6) - [`078ac77`](https://github.com/reconurge/flowsint/commit/078ac77eee0b4943b4acb5596049fa4e20b41264) build(app): add fetch-event-source + vitest test runner - [`6e5f26a`](https://github.com/reconurge/flowsint/commit/6e5f26aaf0d324499e827c9e0939955a640362f9) feat(app): add connectSSE helper with header auth and backoff - [`d7f21e9`](https://github.com/reconurge/flowsint/commit/d7f21e938421119645394aea58d8c884354950bb) refactor(app): stream logs via connectSSE (token out of URL) - [`4a75d6a`](https://github.com/reconurge/flowsint/commit/4a75d6a924129a65a6824658375f7101d6977fea) refactor(app): stream graph status via connectSSE (token out of URL) - [`9db1e67`](https://github.com/reconurge/flowsint/commit/9db1e675d67d28c8a8bbfa70e5b7a9aa0a5bc9de) test(api): SSE auth-gate and endpoint-removal tests (red) - [`b317e08`](https://github.com/reconurge/flowsint/commit/b317e08e5790b2aefef8796b5202fe98d20389b0) feat(api): authenticate SSE via header, drop query-param auth and dead route ### 📊 Changes **11 files changed** (+555 additions, -178 deletions) <details> <summary>View changed files</summary> 📝 `flowsint-api/app/api/deps.py` (+1 -41) 📝 `flowsint-api/app/api/routes/events.py` (+4 -50) ➕ `flowsint-api/tests/conftest.py` (+38 -0) ➕ `flowsint-api/tests/test_events_auth.py` (+50 -0) 📝 `flowsint-app/package.json` (+4 -1) ➕ `flowsint-app/src/api/sse.test.ts` (+117 -0) ➕ `flowsint-app/src/api/sse.ts` (+72 -0) 📝 `flowsint-app/src/hooks/use-events.ts` (+13 -36) 📝 `flowsint-app/src/hooks/use-graph-refresh.ts` (+28 -47) ➕ `flowsint-app/vitest.config.ts` (+13 -0) 📝 `yarn.lock` (+215 -3) </details> ### 📄 Description fixes https://github.com/reconurge/flowsint/issues/145 closes https://github.com/reconurge/flowsint/pull/162 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

GiteaMirror added the pull-request label 2026-06-17 01:19:03 -05:00

GiteaMirror closed this issue

2026-06-17 01:19:05 -05:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/flowsint#3240