WebThingsIO / registration_server

The registration server for WebThings Gateway.
Mozilla Public License 2.0
79 stars 33 forks source link

Remove unused discovery APIs. #42

Closed mrstegeman closed 6 years ago

mrstegeman commented 6 years ago

Re-use /ping for activity tracking only.

Fixes #40

andrenatal commented 6 years ago

Are we sure we'll never need to use these APIs?

andrenatal commented 6 years ago

I know they were used for foxlink but don't know the reason they existed either and if we'd need them at some point.

mrstegeman commented 6 years ago

We don't use any of the discovery APIs, and I don't believe we ever have. We rely solely on the PageKite tunnel, so we don't do any of the shortest-path stuff that these APIs provided. As discussed in the meeting yesterday, we'd like to not send the user's local IP address to the registration server, which renders these APIs useless.