UnityPlugin icon indicating copy to clipboard operation
UnityPlugin copied to clipboard

Expose service IP and port as variables in service providers

Open RodolpheHoudas-UL opened this issue 2 years ago • 3 comments

Summary

This replaces the _serverNameSpace with _serverIP and _serverPort. _serverNameSpace was used back in Orion when it was using named pipes. It is now using TCP and although LeapC still uses a variable called server_namespace, it can be used to connect to a service listening on a different IP/port by passing a JSON payload containing tracking_server_ip and tracking_server_port:

Contributor Tasks

  • [ ] Create or edit test cases here
  • [x] Add a CHANGELOG entry for this change.
  • [x] Ensure documentation requirements are met e.g., public API is commented.
  • [x] Consider any licensing/other legal implications for this MR e.g. notices required by any new libraries.
  • [x] Add any relevant labels such as breaking to this MR.

Reviewer Tasks

Add any instructions or tasks for the reviewer such as specific test considerations before this can be merged.

Use emojis in review threads to communicate intent and help contributors.

  • [ ] Run all tests associated with the ticket.
  • [x] Code reviewed.
  • [x] Non-code assets e.g. Unity assets/scenes reviewed.
  • [x] Documentation has been reviewed.
  • [ ] Approve with a comment of any additional tests run or any observations.

Closes JIRA Issue

Closes UNITY-1217

RodolpheHoudas-UL avatar Aug 31 '23 14:08 RodolpheHoudas-UL

I've tested this with the localhost server and a server on a NUC on the same network (in-Editor and as a built app). It worked successfully.

robinhawkins-UH avatar Oct 30 '23 14:10 robinhawkins-UH

I think it would be valuable to be able to programmatically set the IP and Port a LeapServiceProvider connects to. Only being able to set this at development time means a developer is always necessary to make that change. If it was doable programmatically, it can be changed in-app or via config with limited technical resources.

robinhawkins-UH avatar Oct 30 '23 16:10 robinhawkins-UH