daviddias / webrtc-explorer

:earth_africa: P2P Network Routing Overlay designed for the Web platform (browsers)
MIT License
288 stars 32 forks source link

Reconnect error #8

Open cvorland opened 9 years ago

cvorland commented 9 years ago

Hi,

Hoping you have an idea on what is happening here.

In Chrome, using the re-connection option,

this.reconnection(opts.reconnection !== false);

results in the following error when attempting to connect to another peer:

_Failed to set remote answer sdp: Called in wrong state: STATEINPROGRESS

If I manually reconnect and re-register the peer, I also get this error when connecting to another peer.

cvorland commented 9 years ago

After leaving it disconnect & reconnect for a couple days, I am seeing this error now:

_Failed to set remote answer sdp: Called in wrong state: STATERECEIVEDACCEPT

and also:

(node) warning: possible EventEmitter memory leak detected. 11 listeners added. Use emitter.setMaxListeners() to increase limit.

daviddias commented 9 years ago

Hi @nutsci sorry I haven't got back to you, have been very busy with the dev of another project. This code is some months old and some of APIs might have changes that change the behaviour, I would have to revisit everything and understand what is going on.

also webrtc-explorer as is, don't have 'gracious leaves', as I never had the chance to implement it, so a peer that goes down, because of network or another reason, might have a havoc behaviour when it tried to reconnect

cvorland commented 9 years ago

Thanks for the reply,

Will deal with a workaround for now and keep an eye on development. Thanks!