0.7.3

The answerer was getting 403 Forbidden when sending ICE candidates because
the server didn't know who the answerer was yet. ICE gathering starts when
setLocalDescription is called, but we were calling /answer AFTER that.

Fixed by sending the answer to the server BEFORE setLocalDescription:
1. Create answer SDP
2. Send answer to server (registers answererPeerId)
3. Set up ICE handler
4. Set local description (ICE gathering starts)

This ensures the server has answererPeerId set before ICE candidates arrive,
so they're properly authorized.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@xtr-dev/rondevu-client",
-  "version": "0.7.1",
+  "version": "0.7.3",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "@xtr-dev/rondevu-client",
-      "version": "0.7.1",
+      "version": "0.7.3",
       "license": "MIT",
       "dependencies": {
         "@xtr-dev/rondevu-client": "^0.5.1"

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@xtr-dev/rondevu-client",
-  "version": "0.7.1",
+  "version": "0.7.3",
   "description": "TypeScript client for Rondevu topic-based peer discovery and signaling server",
   "type": "module",
   "main": "dist/index.js",

src/peer/answering-state.ts

@@ -25,13 +25,17 @@ export class AnsweringState extends PeerState {
 
       // Create answer
       const answer = await this.peer.pc.createAnswer();
-      await this.peer.pc.setLocalDescription(answer);
 
-      // Send answer to server immediately (don't wait for ICE)
+      // Send answer to server BEFORE setLocalDescription
+      // This registers us as the answerer so ICE candidates will be accepted
       await this.peer.offersApi.answer(offerId, answer.sdp!);
 
-      // Enable trickle ICE - send candidates as they arrive
-      this.setupIceCandidateHandler(offerId);
+      // Enable trickle ICE - set up handler before ICE gathering starts
+      this.setupIceCandidateHandler();
+
+      // Set local description - ICE gathering starts here
+      // Server already knows we're the answerer, so candidates will be accepted
+      await this.peer.pc.setLocalDescription(answer);
 
       // Transition to exchanging ICE
       const { ExchangingIceState } = await import('./exchanging-ice-state.js');

src/peer/creating-offer-state.ts

@@ -24,9 +24,13 @@ export class CreatingOfferState extends PeerState {
         this.peer.emitEvent('datachannel', channel);
       }
 
+      // Enable trickle ICE - set up handler before ICE gathering starts
+      // Handler will check this.peer.offerId before sending
+      this.setupIceCandidateHandler();
+
       // Create WebRTC offer
       const offer = await this.peer.pc.createOffer();
-      await this.peer.pc.setLocalDescription(offer);
+      await this.peer.pc.setLocalDescription(offer); // ICE gathering starts here
 
       // Send offer to server immediately (don't wait for ICE)
       const offers = await this.peer.offersApi.create([{
@@ -36,10 +40,7 @@ export class CreatingOfferState extends PeerState {
       }]);
 
       const offerId = offers[0].id;
-      this.peer.offerId = offerId;
-
-      // Enable trickle ICE - send candidates as they arrive
-      this.setupIceCandidateHandler(offerId);
+      this.peer.offerId = offerId; // Now handler can send candidates
 
       // Transition to waiting for answer
       const { WaitingForAnswerState } = await import('./waiting-for-answer-state.js');

src/peer/state.ts

@@ -35,13 +35,13 @@ export abstract class PeerState {
    * Setup trickle ICE candidate handler
    * Sends local ICE candidates to server as they are discovered
    */
-  protected setupIceCandidateHandler(offerId: string): void {
+  protected setupIceCandidateHandler(): void {
     this.iceCandidateHandler = async (event: RTCPeerConnectionIceEvent) => {
-      if (event.candidate && offerId) {
+      if (event.candidate && this.peer.offerId) {
         const candidateData = event.candidate.toJSON();
         if (candidateData.candidate && candidateData.candidate !== '') {
           try {
-            await this.peer.offersApi.addIceCandidates(offerId, [candidateData]);
+            await this.peer.offersApi.addIceCandidates(this.peer.offerId, [candidateData]);
           } catch (err) {
             console.error('Error sending ICE candidate:', err);
           }