8000 ✨Minecraft traps by dxrcy · Pull Request #62 · rozukke/lace · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

✨Minecraft traps #62

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
Merged
merged 20 commits into from
Apr 18, 2025
Merged

✨Minecraft traps #62

merged 20 commits into from
Apr 18, 2025

Conversation

dxrcy
Copy link
Contributor
@dxrcy dxrcy commented Apr 14, 2025

Socket is lazily connected on first execution of a Minecraft trap.

Reads and writes to TCP stream without allocating temporaries for request/response data.

  • StreamIter wraps TcpStream::read to buffer reads and yield individual bytes.
  • IntegerReader parses integers from a byte iterator (StreamIter).
  • CleanMemoryStr wraps MemoryStr to sanitize string arguments in ECLI requests.

Mostly copied from mcrs, although exclusion of non-scalar responses allows for more specialized parsing.

dxrcy added 20 commits February 12, 2025 11:55
@dxrcy
feat: implement minecraft traps
53a91dc
@dxrcy
opt: read int responses without buffer
a9d446d
@dxrcy
refactor: make integer reader much better
ed4e50c
@dxrcy
feat: make errors consistent
dc3e335
@dxrcy
opt: use format_args! for commands to avoid alloc
3b7d11c
@dxrcy
fix: sanitize string for CHAT
56b9a66
@dxrcy
chore: document mc items
5a7dbaa
@dxrcy
opt: use buffered read for response
77056cc
@dxrcy
opt: reduce response buffer size
0922d11
@dxrcy
chore: tidy up and comments
0003fb3
@dxrcy
opt: streamline CHAT string read to avoid alloc
4b6ae82
@dxrcy
opt: replace allocating BufReader with fixed-size buffer
d74f3fe
@dxrcy
Merge remote-tracking branch 'upstream/main' into mc
adc5070
@dxrcy
fix: merged imports
f956b0b
@dxrcy
feat(eval): allow mc traps
048efc2
@dxrcy
lint: make mc::READ_BUFFER_SIZE value more explicit
c7909aa
@dxrcy
fix: fix read buffer bug
62f4625
@dxrcy
feat: exception on unexpected socket eof
088098d
@dxrcy
lint: clippy
b67494d
@dxrcy
doc: fix regex comment
20d65c3
@dxrcy dxrcy changed the base branch from main to minecraft April 16, 2025 00:08
rozukke
rozukke approved these changes Apr 18, 2025
View reviewed changes
Copy link
Owner
@rozukke rozukke left a comment

Choose a reason for hiding this comment

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

tyvm students will now experience the full swag of our next-level tooling

@rozukke rozukke merged commit 6f5b900 into rozukke:minecraft Apr 18, 2025
3 checks passed
@dxrcy dxrcy deleted the mc branch April 18, 2025 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rozukke rozukke rozukke 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.
2 participants
@dxrcy @rozukke
0