fix: connection stability improvements for production issues #1648
Conversation
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
- test_gateway_to_gateway_connection: tests gateway-to-gateway connection setup - test_gateway_packet_size_change_after_60s: monitors connection for 75 seconds with GET requests every 5 seconds - test_production_decryption_error_scenario: attempts to reproduce exact production scenario These tests are designed to reproduce the decryption errors seen in production where: - Errors start appearing ~60 seconds after connection establishment - Packet sizes change from 48 bytes to 256 bytes when errors begin - Same inbound_key is used throughout (no key rotation issue) 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
- Add transport_secret_key field to RemoteConnection for RSA decryption - Implement intro packet detection for 256-byte packets on established connections - Send ACK response when intro packets are successfully decrypted - This addresses the secondary bug where established peers don't handle intro packets gracefully - Related to production decryption errors occurring after ~60 seconds
…uilds - Set KEEP_ALIVE_INTERVAL to 10 seconds for all builds - Set KILL_CONNECTION_AFTER to 30 seconds for all builds - Prevents incompatibility between debug and release peers - Debug peers were timing out connections to release peers after 6 seconds - This was a critical design flaw causing connection instability
This directory will contain: - Release automation scripts - Deployment summaries - Local test files - Other files that shouldn't be committed to git 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
Fixes
Root Cause
Production decryption errors after ~60 seconds were caused by:
Test plan
🤖 Generated with Claude Code