Conversation
Summary of ChangesHello @Geun-Oh, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request introduces a new Zig-native TLS 1.3 implementation, Highlights
Changelog
Activity
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Code Review
This pull request adds zigtls, a native Zig TLS 1.3 implementation, to the list of awesome Zig projects under the Network section. The addition is valuable. I have one suggestion to make the description more concise and align better with other entries in the list.
| - [williamw520/zigjr](https://github.com/williamw520/zigjr) - A lightweight Zig library for building JSON-RPC 2.0 applications. | ||
| - [cocky-punch/raft](https://github.com/cocky-punch/raft) - Raft consensus algorithm implemented in Zig. | ||
| - [zigcord](https://codeberg.org/lipfang/zigcord) - Typed Discord API for Zig. | ||
| - [zigtls](https://github.com/Geun-Oh/zigtls) - Zig-native TLS Implementation library for edge/load-balancer event loops, with BoGo strict, interop, and reliability gates. |
There was a problem hiding this comment.
For consistency with other entries and for better clarity, consider using a more concise description. The description from the project's GitHub repository, "Zig-native TLS 1.3 implementation for client, server, and QUIC.", is a great alternative as it's more direct and arguably easier to understand for a wider audience.
| - [zigtls](https://github.com/Geun-Oh/zigtls) - Zig-native TLS Implementation library for edge/load-balancer event loops, with BoGo strict, interop, and reliability gates. | |
| - [zigtls](https://github.com/Geun-Oh/zigtls) - Zig-native TLS 1.3 implementation for client, server, and QUIC. |
There was a problem hiding this comment.
I'm about to add QUIC soon, but it doesn't support QUIC currently.
|
Great work! |
Adding zigtls, zig native tls 1.3 implementation.
I have developed it to natively meet all TLS 1.3 requirements, and you can review the details at
https://plaintexting.com/implementations-engineerings/zigtls/.
I'll maintain and update it consistently while adhering to security standards.