feat: implement websocket

[leanmendoza]

No description provided.

[codecov-commenter]

Codecov Report

Patch coverage: 7.10% and project coverage change: -1.28% ⚠️

Comparison is base (9bbb32b) 19.49% compared to head (3ca4757) 18.22%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #45      +/-   ##
==========================================
- Coverage   19.49%   18.22%   -1.28%     
==========================================
  Files          58       59       +1     
  Lines        6790     7299     +509     
==========================================
+ Hits         1324     1330       +6     
- Misses       5466     5969     +503     

Files Changed	Coverage Δ
rust/decentraland-godot-lib/src/av/ffmpeg_util.rs	0.00% <0.00%> (ø)
...ust/decentraland-godot-lib/src/av/video_context.rs	0.00% <0.00%> (ø)
rust/decentraland-godot-lib/src/av/video_stream.rs	0.00% <0.00%> (ø)
...aland-godot-lib/src/comms/communication_manager.rs	1.14% <0.00%> (ø)
rust/decentraland-godot-lib/src/comms/wallet.rs	0.00% <0.00%> (ø)
rust/decentraland-godot-lib/src/comms/ws_room.rs	0.00% <0.00%> (ø)
...ust/decentraland-godot-lib/src/dcl/crdt/message.rs	0.00% <0.00%> (ø)
rust/decentraland-godot-lib/src/dcl/js/mod.rs	0.00% <0.00%> (ø)
...decentraland-godot-lib/src/dcl/js/websocket/mod.rs	0.00% <0.00%> (ø)
...ot-lib/src/scene_runner/components/avatar_shape.rs	0.00% <0.00%> (ø)
... and 13 more

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[kuruk-mm]

According to the current SceneRuntime, the Explorer should only allows wss://. It would be great to maintain that limitation to have the same behaviour on all platforms (we should do the same for fetch, too).

Another thing that we can add later is the permissions for using WebSocket that are in the scene.json. I want to add a TODO for that.

[leanmendoza]

According to the current SceneRuntime, the Explorer should only allows wss://. It would be great to maintain that limitation to have the same behaviour on all platforms (we should do the same for fetch, too).

Another thing that we can add later is the permissions for using WebSocket that are in the scene.json. I want to add a TODO for that.

Thanks for the review. Nice catch, we're going to document that extra part of the runtime.