Skip to content

[Imported] Fix HTTP definition. #443

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.

Sign up for GitHub

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

Jump to bottom
Open
ido777 wants to merge 98 commits into
base: main
Choose a base branch
from
Open

[Imported] Fix HTTP definition. #443

ido777 wants to merge 98 commits into from

Conversation

@ido777
Copy link
Owner

@ido777 ido777 commented Apr 6, 2025

Imported from donnemartin/system-design-primer#357

Original author: @tsmirnov

HTTP is built on top of TCP only

donnemartin and others added 30 commits December 3, 2017 07:09
@donnemartin
Update Japanese translation link (#118)
c3f080f
@koudaiii @donnemartin
ja: Fix typo (#119)
a4973a0
@donnemartin
Make some minor wording/formatting changes (#120)
748944c
@fluency03
add Twitter Handles 3,000 Images Per Second
dd124c4
@fluency03 @donnemartin
Add Netflix in Company Architectures (#122)
11b9d72
@donnemartin
Add Vietnamese translation link (#128)
8702199
@fluency03 @donnemartin
Add Scaling Uber in Company Architecture (#123)
156a67e
@donnemartin
Update master-slave section anchor (#129)
ebcec73
@fluency03 @donnemartin
Add Facebook Live Streams (#125)
0263906
@donnemartin
Update contributing guidelines - PR squash (#135)
12946d3
@hakaari @donnemartin
Update re:Invent url (#137)
61bc6f7
@fluency03
Merge branch 'master' into add-twitter
e86ed77
@kevingo @donnemartin
Add Traditional Chinese translation (#133)
9712a12
@kevingo @donnemartin
zh-TW: Add proper line break to links (#139)
82868d3
@donnemartin
Add Greek translation link (#140)
7272a9b
@donnemartin
Convert all .py files to be valid Python (#98)
de3c13c
@robertding @donnemartin
zh-Hans: Fix typos (#141)
94c158e
@donnemartin
Add Spanish translation link (#142)
7c38532
@donnemartin
Add missing colons to class methods (#143)
3079562
@donnemartin
Add missing self variables to Deck of Cards solution (#145)
d90b562
@donnemartin
Add deque import to Social Graph solution (#147)
7e316ce
@donnemartin
Change LARGE to VehicleSize.LARGE in parking lot solution (#146)
f552598
@htkuan @donnemartin
zh-TW: Fix comment format (#155)
f038dbb
@donnemartin
Fix coding errors (#149)
d7d0e9d
@hulikau @donnemartin
Fix dict KeyError (#152)
f05373a
@hulikau @donnemartin
Fix dict KeyError (#153)
7a4dbb1
@donnemartin
Adding missing self variable (#158)
4a0d7fb
@donnemartin
Add missing enum imports (#157)
02a20c6
@donnemartin
Change variable seller to category in Mint solution (#159)
2368ee0
@bursonyang @donnemartin
zh-Hans: Fix typo (#161)
38cb1be
spuyet and others added 25 commits May 11, 2019 12:42
@spuyet @donnemartin
Translate language list (#252)
fc8568a
@donnemartin
Add Ebook generation script (#207)
3c2b831
@mebjas @donnemartin
Add availability in numbers section (#237)
d2384bd
@donnemartin
Update language lists in translations (#280)
9058471
@donnemartin
Add Hebrew translation link (#286)
f5c39e0
@kevinxuv @donnemartin
zh-Hans: Translate Pastebin solution (#273)
6a5d2fe
@Wintus @donnemartin
ja: Fix typo of Big-O notation in KVS section (#292)
014a4a0
@satob
JA: Fix mistranslation in Horizontal scaling section
9ffe72e 
- The Japanese translation is ambiguous about “vertical scaling” means scaling out or scaling up.
- The word “expensive” is missing in the Japanese translation.
@satob
JA: Fix mistranslation in Reverse proxy (web server) section
71f1693 
- Fix mistranslation of parallel structure. (not information/blacklist/limit, but hide/blacklist/limit)
@satob
ja: Fix mistranslation in "Horizontal scaling"
4956d22
@satob @donnemartin
JA: Fix mistranslation in Weak consistency section (#299)
06eeeb7
@satob @donnemartin
JA: Fix mistranslation in Push CDNs section (#300)
218a23d
@donnemartin
ja: Fix translation in horizontal scaling section (#301)
87912ed 
* JA: Fix mistranslation in Horizontal scaling section

- The Japanese translation is ambiguous about “vertical scaling” means scaling out or scaling up.
- The word “expensive” is missing in the Japanese translation.

* ja: Fix mistranslation in "Horizontal scaling"
@donnemartin
Merge pull request #302 from satob/ReverseProxy
316c00c
@satob @donnemartin
JA: Fix mistranslation in Federation section (#303)
46634d1
@satob @donnemartin
ja: Fix translation in "Anki flashcards" (#306)
779b466
@satob @donnemartin
ja: Fix translation in “Disadvantage(s): load balancer” (#307)
b44a018
@satob @donnemartin
ja: Fix translation in Service Discovery section (#308)
8bfc344
@duynguyenhoang @donnemartin
Add API security checklist (#328)
1972808
@brandonJY @donnemartin
zh-Hans: Change translation in SQL tuning (#318)
ce29593
@satob @donnemartin
ja: Fix mistranslation in SQL tuning section (#305)
7f56907
@cclauss @donnemartin
Change raise NotImplemented to raise NotImplementedError (#345)
24707af
@ygdanchoi @donnemartin
Fix broken round robin links (#351)
ba69584
@donnemartin
Fix broken CAP theorem link (#355)
105fa53
@tsmirnov
fix http
010ba4c
Copilot AI review requested due to automatic review settings April 6, 2025 20:40
@ido777
Copy link
Owner Author

ido777 commented Apr 6, 2025

Imported comment from @AleksandrRadchenko

HTTP is typically built over TCP, but it is not a must. See, for example: https://stackoverflow.com/a/323401/6100749

Copilot AI reviewed Apr 6, 2025
View reviewed changes
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@ido777 ido777 force-pushed the imported-pr-357 branch from d32f752 to 010ba4c Compare May 14, 2025 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@anandfresh anandfresh anandfresh approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.