Add set_reconnect_url to Client

[tyilo]

No description provided.

[1c3t3a]

Sorry for the late review and thanks for the patch! I don't really understand this patch though, why do we need this instead of setting a different address when we create the builder? This doesn't give any benefit in my eyes.

[tyilo]

Sorry for the late review and thanks for the patch! I don't really understand this patch though, why do we need this instead of setting a different address when we create the builder? This doesn't give any benefit in my eyes.

I forgot to link it, but this fixes #251 where I have described why I need this feature.

[tyilo]

Ping @1c3t3a

[1c3t3a]

Sorry for the delay!! I have some comments :) Can you please also rebase and resolve the merge conflicts?

[tyilo]

Rebased and updated the code.

[codecov]

Codecov Report

Merging #252 (d3eeae4) into main (e6e1ae6) will increase coverage by 0.21%.
The diff coverage is 96.03%.

❗ Current head d3eeae4 differs from pull request most recent head 684e497. Consider uploading reports for the commit 684e497 to get more accurate results

@@            Coverage Diff             @@
##             main     #252      +/-   ##
==========================================
+ Coverage   90.90%   91.12%   +0.21%     
==========================================
  Files          37       37              
  Lines        4629     4719      +90     
==========================================
+ Hits         4208     4300      +92     
+ Misses        421      419       -2     

Impacted Files	Coverage Δ
socketio/src/client/builder.rs	92.56% <ø> (ø)
socketio/src/client/client.rs	97.18% <95.60%> (-0.37%)	⬇️
socketio/src/lib.rs	100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

[1c3t3a]

LGTM :) Thanks you!

[tyilo]

Thanks for merging :)