Page MenuHomeSoftware Heritage

Remove visit/ endpoint and keep only: visit/nodes visit/paths
ClosedPublic

Authored by haltode on Jul 29 2019, 4:10 PM.

Details

Summary

Returning both nodes and paths adds unnecessary convoluted code in the
client and server code. In addition, the global 'visit/' endpoint is not
listed in any of our use-case, and having the two separate sub-endpoints
(visit/nodes and visit/paths) is enough.

Note: the server code was not fully updated after recent code clean up, so the
visit/ global endpoint needed: a new function on its own (repeating again the
code in visitNodes and visitPaths) and a custom class for the JSON output
format.

Diff Detail

Repository
rDGRPH Compressed graph representation
Branch
rm-visit-root-endpoint
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 7063
Build 9949: arc lint + arc unit