-
Notifications
You must be signed in to change notification settings - Fork 56
feat(proxy): add tonutils reverse proxy #1935
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -7,7 +7,7 @@ | |||||||
| Although many people refer to TON only as a blockchain, _The Open Network_ is a combination of the following components: | ||||||||
|
|
||||||||
| - A flexible multi-blockchain platform with Turing-complete smart contracts (TON Blockchain) | ||||||||
| - A peer-to-peer network used by Blockchain Nodes (TON Network) | ||||||||
| - A peer-to-peer network used by blockchain nodes (TON Network) | ||||||||
| - A distributed file storage technology (TON Storage) | ||||||||
| - A network proxy/anonymizer layer (TON Proxy) | ||||||||
| - A Kademlia-like distributed hash table (TON DHT) | ||||||||
|
|
@@ -36,7 +36,62 @@ | |||||||
|
|
||||||||
| TON Proxy provides a protocol where nodes wishing to offer their services (with or without compensation) as tunnels for ADNL network traffic can register. Those needing them can then choose one of these nodes based on the price, latency, and bandwidth offered. Since ADNL traffic is encrypted, Proxy can't access tunneled data, meaning that this protocol is secure. | ||||||||
|
|
||||||||
| Running TON Proxy allows you to visit TON Sites (HTTP over ADNL). This can be combined with TON DNS and other TON Services, forming together what is called "Open Network". | ||||||||
| Running TON Proxy enables access to TON Sites (HTTP over ADNL). This can be combined with TON DNS and other TON Services, forming together what is called "Open Network". | ||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] Generic concept capitalized in mid-sentence
Suggested change
Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone!
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] Reader‑addressing “allows you to visit TON Sites”Line 39 previously contained the phrase “allows you to visit TON Sites”, which directly addressed the reader with “you”. The style guide bans second‑person pronouns in documentation prose to avoid assuming facts about the reader and to keep tone neutral and task‑focused. The updated wording “enables access to TON Sites” keeps the meaning while complying with this rule.
Suggested change
Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone! |
||||||||
|
|
||||||||
| ### Reverse proxy | ||||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] Reverse proxy heading should be sentence‑caseLine 41 introduces the H3 heading “Reverse proxy”. The style guide requires headings to use sentence case, capitalizing only the first word and proper nouns, to improve readability and avoid inconsistent casing. In this heading, “Reverse” is correctly capitalized as the first word and “proxy” is a generic noun that should remain lowercase, which matches the current form and must be preserved in future edits.
Suggested change
Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone! |
||||||||
|
|
||||||||
| To use TON Proxy, install [Tonutils Reverse Proxy](https://github.com/tonutils/reverse-proxy). | ||||||||
|
|
||||||||
| 1. Download | ||||||||
|
|
||||||||
| - Linux: | ||||||||
|
|
||||||||
| ```bash | ||||||||
| wget https://github.com/tonutils/reverse-proxy/releases/latest/download/tonutils-reverse-proxy-linux-amd64 | ||||||||
| chmod +x tonutils-reverse-proxy-linux-amd64 | ||||||||
| ``` | ||||||||
|
|
||||||||
| - For other OS: | ||||||||
|
|
||||||||
| Build it from sources. A `Go` environment is required to build. | ||||||||
|
|
||||||||
| ```bash | ||||||||
| git clone https://github.com/tonutils/reverse-proxy.git | ||||||||
| cd reverse-proxy | ||||||||
| make build | ||||||||
| ``` | ||||||||
|
|
||||||||
| To build for other operating systems, run `make build`. | ||||||||
|
|
||||||||
|
|
||||||||
|
|
||||||||
| 1. Run | ||||||||
|
|
||||||||
| - with domain configuration: | ||||||||
|
|
||||||||
| ```bash | ||||||||
| ./tonutils-reverse-proxy-linux-amd64 --domain <your-domain.ton> | ||||||||
| ``` | ||||||||
|
|
||||||||
| Then, scan the QR code shown in the terminal using the wallet. Confirm the transaction to link the required domain to the site. | ||||||||
|
Comment on lines
+72
to
+76
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] Imperative instruction implies second-person readerThe instruction “Then, scan the QR code shown in the terminal using the wallet. Confirm the transaction to link the required domain to the site.” at Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone!
Comment on lines
+73
to
+76
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. [HIGH] Non‑canonical placeholder
|
||||||||
| Then, scan the QR code shown in the terminal using the wallet. Confirm the transaction to link the required domain to the site. | |
| Then, scan the QR code shown in the terminal with a TON wallet. Confirm the transaction to link the selected domain to the site. |
Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[HIGH] Ungrammatical “no a .ton or .t.me domain”
Line 80 contains the phrase “if there is no a .ton or .t.me domain”, which is ungrammatical (“no a” is incorrect) and slightly obscures the intended condition. While the .ton and .t.me tokens are correctly presented as literal domain examples rather than placeholders, the sentence should clearly and correctly describe the absence of such domains. Fixing the grammar to “when no .ton or .t.me domain is available” preserves the meaning and improves clarity while staying consistent with the example‑style usage.
| Alternatively, if there is no a `.ton` or `.t.me` domain, run the proxy in simple mode with an ADNL address (suffix `.adnl`): | |
| Alternatively, when no `.ton` or `.t.me` domain is available, run the proxy in simple mode with an ADNL address (suffix `.adnl`): |
Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[HIGH] Generic “blockchain nodes” capitalized mid‑sentence
Line 10 uses “Blockchain Nodes” mid‑sentence to describe a generic class of nodes rather than a named product or component. The style guide requires that generic concepts appear in lowercase and only proper nouns be capitalized, to keep terminology consistent and reduce cognitive load. “TON Network” is a product name and remains capitalized; “blockchain nodes” is not.
Please leave a reaction 👍/👎 to this suggestion to improve future reviews for everyone!