Skip to content

HTTP/SSE auth hardening and test file split#9

Closed
0xeb wants to merge 1 commit intomainfrom
hardening/server-auth
Closed

HTTP/SSE auth hardening and test file split#9
0xeb wants to merge 1 commit intomainfrom
hardening/server-auth

Conversation

@0xeb
Copy link
Copy Markdown
Owner

@0xeb 0xeb commented Nov 28, 2025

Summary

  • Split large interactions.cpp (5500+ lines) into 3 parts to fix MSVC heap exhaustion on Windows CI
  • Added shared test fixture header for cleaner code organization
  • Added HTTP/SSE server authentication hardening
  • Added auth-related tests

Changes

  • tests/server/interactions.cpp → split into interactions_p1.cpp, interactions_p2.cpp, interactions_p3.cpp
  • New tests/server/interactions_fixture.hpp for shared test infrastructure
  • New tests/server/http_auth.cpp for auth tests
  • HTTP/SSE server hardening in http_server.cpp and sse_server.cpp

Test plan

  • Windows CI passes (previously failing with heap exhaustion)
  • Linux CI passes
  • macOS CI passes
  • All 164 interaction tests still pass across the 3 split files

@0xeb
Copy link
Copy Markdown
Owner Author

0xeb commented Nov 28, 2025

Pulling offline to fix Windows heap issues - need finer split

@0xeb 0xeb closed this Nov 28, 2025
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