Use random tie-breaker value for ICE connectivity compatibility#258
Open
dsugisawa-mixi wants to merge 63 commits intosepfy:mainfrom
Open
Use random tie-breaker value for ICE connectivity compatibility#258dsugisawa-mixi wants to merge 63 commits intosepfy:mainfrom
dsugisawa-mixi wants to merge 63 commits intosepfy:mainfrom
Conversation
dsugisawa-mixi
commented
Jan 28, 2026
dsugisawa-mixi
commented
- Generate random 64-bit tie-breaker for ICE binding requests instead of fixed 0
- Improves connectivity with libjuice (libdatachannel) which does not accept tie-breaker value of 0
- RFC 8445, uniformly distributed between 0 and (2^64) − 1 (that is, a 64-bit positive integer)
- Add fat_library target to combine all static libs into libpeer_fat.a - Add tcp_transport for non-TLS signaling connections (http/mqtt schemes) - Simplify generic example to DataChannel-only demo
Previously, binding_request_time was only updated when receiving STUN Binding Requests from the remote peer. This caused connections to timeout after CONFIG_KEEPALIVE_TIMEOUT even when other data (RTP/RTCP/DTLS) was actively flowing. Now update the keepalive timestamp whenever any valid data is received in PEER_CONNECTION_COMPLETED state, preventing spurious disconnections.
- Add cmake/unity/CMakeLists.txt for cross-platform Unity builds - Add build scripts for each platform: - build_ios.sh: Device (arm64) + Simulator (arm64) xcframework - build_android.sh: arm64-v8a, armeabi-v7a, x86_64 shared libs - build_macos.sh: Universal binary bundle with codesign/notarization - Add UPM package (jp.co.mixi.libpeer) with: - Pre-built native plugins for all platforms - C# P/Invoke bindings (LibPeer.cs) - Unity meta files for proper asset importing Install via: https://github.com/dsugisawa-mixi/libpeer.git?path=upm/jp.co.mixi.libpeer
- Add new rp2040-baremetal example project with CMake build system - Include POSIX compatibility shims for socket, pthread, and inet APIs - Configure lwIP and mbedTLS for embedded environment - Update socket layer to support lwIP backend - Update Unity sample scene
- Implement getaddrinfo/freeaddrinfo using lwIP dns_gethostbyname - Fix -include flags to apply only to C/C++ (not assembler) - Reduce lwIP memory usage (PBUF_POOL_SIZE, TCP_WND, TCP_SND_BUF)
Reduce RAM usage for TLS/DTLS on RP2040's 264KB RAM: - Reduce SSL buffers from 16KB to 4KB each (~24KB saved) - Limit ECC curves to P-256, P-384, Curve25519 (from 12 curves) - Use ECDHE key exchange only (no RSA key exchange) - Disable debug and unused features (MD5, DHM, PKCS5) - Enable memory-saving options (AES_ROM_TABLES, SHA256_SMALLER) Maintains Cloudflare TLS 1.2 and WebRTC DTLS compatibility.
- Change PICO_BOARD from pico_w to pico2_w - Update project name to rp2350bm - Increase SSL buffers to 16KB (RP2350 has 520KB RAM) - Let pico-sdk auto-detect TinyUSB MCU configuration
Initialize UART0 with 115200 baud on GP0/GP1 before stdio_init_all() to ensure serial output works on Pico 2 W (RP2350).
- Add explicit UART pin/baud definitions in CMakeLists.txt - Add LED blink after cyw43 init (3 blinks) for diagnostics - Add LED blink after WiFi connect (10 blinks) for diagnostics
- Add RP2040-specific recv_timeout using cyw43_arch_poll - Fix operator precedence bug in tcp_socket_connect return check - Add error handling for host resolution failure - Update .envrc to use pico2_w board and new signaling URL
- Replace single UDP receive buffer with 8-entry FIFO to handle multiple incoming packets without dropping - Update select() stub to poll cyw43_arch_poll() during timeout period, giving lwIP time to process incoming packets and fire callbacks
Document each patch file with specific changes and reasons: - libsrtp: uint32_t type fix for SSRC on 32-bit ARM - mbedtls: DTLS-SRTP extension enablement - usrsctp: lwIP headers, sockaddr_conn layout, hardware RNG, debug logs
…ty layer - Add non-blocking LED blink state machine for visual feedback - TX (send): 100ms interval, 3 blinks - RX (receive): 20ms interval, 15 blinks - Document compatibility layer stubs in README (inet_compat, atomic_compat, header stubs)
- Log boot, WiFi, signaling, ICE, DTLS, SCTP, and first TX/RX timestamps - Update README with correct openocd command for RP2350
- Use async WiFi connect with state polling for granular timing - Log WiFi scan, auth/assoc, DHCP phases separately - Add DNS resolution timing before signaling connection - Track CYW43_LINK_JOIN/NOIP/UP state transitions
…flict Also add JSON parse logging, increase bMaxPower to 500mA, and add debug prefix to hid_task output.
cyw43 -> board_init/tusb_init -> WiFi connect -> WebRTC Avoids both GPIO/PIO conflict and iPhone EP0 panic.
tud_task() was only called after PEER_CONNECTION_COMPLETED, leaving ~13s without USB processing. iPhone USB host triggers EP0 panic when requests go unhandled.
core0: tud_task + hid_task + LED (always polling, never blocks) core1: cyw43_arch_poll + peer_connection_loop + datachannel TX
cyw43_arch_init() on core1 so background worker IRQs never block core0's USB interrupts. Wait for tud_mounted() before launching core1 to ensure EP0 enumeration completes cleanly.
Use queue_try_add/queue_try_remove for multicore-safe inter-core communication without manual critical sections.
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.