Write v2.0.0 bouncer spec#129
Merged
Merged
Conversation
hellais
reviewed
Mar 18, 2019
|
|
||
| "front": | ||
| (optional) `string` indicating the real host to connect | ||
| to when using a cloudfronting service. |
Member
There was a problem hiding this comment.
Let's say "domain fronting" instead of "cloudfronting" as cloudfront is specific to the Amazon vendor.
hellais
reviewed
Mar 18, 2019
hellais
left a comment
hellais
left a comment
Member
There was a problem hiding this comment.
I looks good to me. I left a minor nitpick comment in the text.
Another thing to think about is if we want to specify versioning for the API in the form of /api/v1/XXX. I think by having versioning in the URL it allows us to upgrade the endpoints in a more flexible way and ensuring we keep backward compatibility for legacy endpoints as soon as there are still clients.
hellais
added a commit
that referenced
this pull request
Apr 8, 2019
* origin/master: (73 commits) What to do when bouncer and/or collector are down (#130) Write v2.0.0 bouncer spec (#129) Write v2.0.0 spec of the collector protocol (#128) Minor grammar edits Readme.md: add a minimal explanation Fix telegram spec to increase clarity ts-020-telegram.md: don't flag semantics as JSON ts-020-telegram.md: format JSON example Add text about possibility second response is firewalled Update and rename tq-xxx-attempt-tfo.md to tq-033-attempt-tfo.md Update and rename tq-xxx-attempt-h2-upgrade.md to tq-032-attempt-h2-upgrade.md Make sure I update the technique number Rename tq-xxx-attempt-ecn.md to tq-031-attempt-ecn.md Adds technique to attempt a download using ECN Adds technique to attempt a download using TCP fast open Adds technique to attempt an HTTP/2 download Expand tq-017 with details and examples Edit tq-007 to add BCP152 link Editing of tq-025 tq-024 more links ...
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
No description provided.