Open
Conversation
- Add __test__/harper-integration.spec.ts: starts a real Harper instance with TLS + UDS mirroring, reads metadata YAML, configures SymphonyProxy to terminate TLS and route through UDS sockets, then verifies HTTP responses. - Fix loadAddon() in ts/proxy.ts: try both one and two levels up from __dirname so the native addon is found in both production (dist/) and test (dist-test/ts/) build layouts. - Add test:integration script and harper/yaml devDependencies in package.json. - Add integration-test CI job that checks out and builds harper-pro, then runs the integration test with HARPER_INTEGRATION_TEST_INSTALL_SCRIPT. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Introduces a sourceAddressHeader route option with three modes:
- 'proxyProtocol' (default for UDS): send PROXY v1 header before data
- 'xForwardedFor': read the first HTTP chunk, insert X-Forwarded-For
after the request line, then copy the rest verbatim — no per-request
parsing overhead on keep-alive connections
- 'none' (default for TCP): no source address forwarding
This enables Bun backends (which lack PROXY protocol support) to receive
the real client IP via X-Forwarded-For over UDS upstreams.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Adds a row to the RouteConfig reference table and a new "Source address forwarding" section explaining all three modes, with examples for the PROXY protocol (HarperDB/nginx) and X-Forwarded-For (Bun) use cases. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…e, and configure TLS session resumption - **package.json**: Added `benchmark`, `benchmark:throughput`, and `diagnostic:keepalive` scripts for testing and diagnostics. - **proxy.rs**: Introduced multi-threaded runtime (`tokio::runtime::Runtime`) to distribute proxy tasks across CPU cores, improving connection handling. - **tls.rs**: Configured TLS session resumption with support for both TLS 1.2 and 1.3, reducing handshake latency for resumed sessions.
…e, and configure TLS session resumption
… to Release and CI workflows
…ifact publishing - **package.json**: Remove `build` step from `prepublishOnly` script. - **Cargo.toml**: Disable default `rustls` and `tokio-rustls` features; enable logging and TLS 1.2 support. - **release.yml**: Refactor artifact handling and npm publishing, publishing platform-specific packages separately via `napi`.
…/release workflows
Updates CI and release workflows to use `--ignore-scripts` flag instead of `--omit=optional` for npm ci, preventing execution of install scripts during dependency installation.
…workflows Updates CI and release workflows to upgrade npm globally before running `npm ci`, ensuring consistent dependency installation on macOS runners.
…e workflows Updates macOS and Linux workflow steps to use `npm install --ignore-scripts` instead of upgrading npm and running `npm ci`, simplifying dependency installation across all runners.
…optionalDependencies Updates the release workflow to manually populate `optionalDependencies` in package.json by reading platform package metadata from the `npm/` directory, replacing the previous `napi prepublish` command. Also upgrades npm to latest version before installing dependencies.
…e workflow Replaces `npm ci` with `npm install -g npm@latest && npm install --ignore-scripts` to ensure npm is upgraded before dependency installation, aligning with other workflow changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.