Skip to content
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

Update Documentation #2127

Merged
merged 1 commit into from
Aug 28, 2023
Merged

Update Documentation #2127

merged 1 commit into from
Aug 28, 2023

Conversation

majestrate
Copy link
Contributor

stub pr for docs that dan is adding

@dr7ana dr7ana marked this pull request as ready for review January 30, 2023 14:51
@dr7ana dr7ana changed the title documentation Update Documentation Jan 30, 2023
@dr7ana dr7ana requested a review from jagerman February 1, 2023 23:47
@dr7ana dr7ana self-assigned this Feb 2, 2023
@dr7ana dr7ana added the documentation related to documentation label Feb 2, 2023
@majestrate
Copy link
Contributor Author

majestrate commented Feb 14, 2023

random note for addition to this pr:

relay up/down has: 32 bytes (nounce) +3 bytes bencode, + 16 bytes (txid) + 3 bytes bencode + 16 bytes (rxid) + 3 bytes bencode + 3 bytes bencode for link layer message kind (1:a) + 2 bytes for 'de' + the data you are sending as ciphertext padded to nearest 128 bytes + 4 or 5 bytes bencode. 
the smallest possible relay up/down message is 210 plaintext then add the current wire protocol overhead of 108 bytes for the node to node transmission header for non fragmented transmission.
the smallest possible sized udp packet for node to node data via a path ends up being 328 bytes.
as long as your routing message is under 128 bytes it will be padded to 128 when inside the relay/up down message

@majestrate
Copy link
Contributor Author

i think we can merge this.

@jagerman jagerman changed the base branch from dev to quic-wip August 28, 2023 16:23
- re-structure docs
- add diagrams
@dr7ana dr7ana merged commit 16814c8 into oxen-io:quic-wip Aug 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants