Proof of concept ready - #8
Merged
Merged
Conversation
…nd optimizations - Add comprehensive documentation (architecture, API, deployment, SFU setup, user guide) - Implement benchmarks for SFU server, client, and WASM - Enhance telemetry with metrics collection and error tracking - Add WASM bundle size optimization (opt-level=z, LTO) - Create Cloudflare Tunnel configuration - Add recording capability stub - Integrate all SFU modules (recorder, room_manager, router) - Create CHANGELOG.md with version history - Update README with complete documentation links
- Created tests/signaling_integration_tests.rs with full handshake tests - Created tests/wasm_integration_tests.rs with WASM module tests - Verified all existing tests pass (35 tests total) - Confirmed media crate has real WebRTC implementation - Updated task.md to reflect actual implementation status
- Added 2 platform-agnostic tests for SFU client reconnection manager - Added 6 comprehensive tests for SFU server room manager - Fixed test compilation errors (field names, imports) - All 45 tests now passing across workspace - Updated task.md to reflect test completion status
…ability, and code quality fixes
- Add unique participant ID generation for each client (user-{timestamp})
- Implement proper chat message attribution (You vs Participant)
- Add TURN server configuration for NAT traversal
- Implement WebSocket keepalive with ping/pong (30s intervals, 60s timeout)
- Add Pong message type to signaling protocol
- Fix unused imports and variables (cargo fix)
- Add wasm-bindgen-test dependency for test compilation
- Remove trailing whitespace from all files
Fixes participant identity issues, improves connection stability, and ensures code quality standards.
- Update WASM layer to send messages with sender labels (You/Participant) - Parse sender_id and compare with client's participant_id - Update JavaScript to parse sender:text format from WASM events - Messages now display correctly as 'You' for sender, 'Participant' for others - Fixes issue where both users saw messages as 'Participant' - Fixes issue where chat messages weren't appearing in UI Note: TURN server credentials issue remains - may need alternative TURN server
- Add TURN server credentials to .env (Metered: 1 STUN + 4 TURN servers) - Update .example.env with ICE server configuration template - Modify negotiation.rs to load ICE servers from environment variables - Support dynamic TURN server configuration (TURN_URL_1 through TURN_URL_4) - Add logging for TURN server registration - Fallback to default STUN server if env vars not set Fixes: 'Failed to handle offer: invalid turn server credentials' error
- Rebuild WASM for web and extension with latest chat attribution fixes - Fix clippy redundant field name warning in sfu-client - Prefix unused data_channel field with underscore - WASM now includes participant ID comparison logic - Chat messages will display correct sender labels All quality checks passing: - Clippy (no warnings) - Format check - WASM build (web + extension)
…t fixes) note: still having blank video issue
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.