Project-OSRM / osrm-backend

Open Source Routing Machine - C++ backend
http://map.project-osrm.org
BSD 2-Clause "Simplified" License
6.34k stars 3.35k forks source link

Deprecate `hints` parameter #4741

Open TheMarex opened 6 years ago

TheMarex commented 6 years ago

The hints parameter is probably one of the worst concepts of our API that exposes internal data structures to the client and allows them to pass this information back to us. This breaks a few encapsulations, means we need to ship a clunky Base64 encoder and have our internal PhantomNode size impact the size of our response objects.

It is straight forward to remove this parameter from the HTTP API but the C++ API is another problem. Since we encode internal state in the Hints object that is part of a public interface, we can't just remove this class. I don't see an easy way to do this without breaking the API.

daniel-j-h commented 6 years ago

Related: