Add REST API and use it for bootstrap data #176
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Extend tauriond so that it can run a REST server. The server has just one endpoint (at least for now), /bootstrap.json.gz, which returns the getbootstrapdata JSON (as per the RPC method) with gzip compression.
Keep a copy of the bootstrap data for the REST API in memory, and only refresh it with a certain interval. (If we are still catching up, no cache is kept.)
Instead of passing getbootstrapdata through Charon, use the new REST API to get the data from tauriond in client mode.
Add in letsencrypt.pem, which holds the --cafile that can be used with the Taurion REST client on the production server (rest.taurion.io).
Extend the charon.py integration test to also exercise getbootstrapdata via the new REST interface between client and server.
domob1812
added
performance
This was referenced Sep 30, 2020
domob1812
added a commit
that referenced
this pull request
Oct 2, 2020
Add REST API and use it for bootstrap data
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This implements #162: It adds an optional REST API to the GSP, which has just one endpoint (for now at least):
/bootstrap.json.gz, serving thegetbootstrapdataresult as gzip-compressed JSON. Note that the data is only refreshed every once in a while, and cached in-between. So it will not exactly matchgetbootstrapdata, but will be a "somewhat fresh" data result which is all we need.In addition,
getbootstrapdataas RPC method is no longer supported by the Charon server. The Charon client will instead try to retrieve it from the REST API, which is a lot more efficient for that big data chunk.