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

Ntpv5 packet parsing #1125

Closed
wants to merge 32 commits into from
Closed

Ntpv5 packet parsing #1125

wants to merge 32 commits into from

Conversation

tdittr
Copy link
Contributor

@tdittr tdittr commented Oct 9, 2023

No description provided.

@tdittr tdittr self-assigned this Oct 9, 2023
@tdittr tdittr mentioned this pull request Oct 9, 2023
21 tasks
@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Attention: 143 lines in your changes are missing coverage. Please review.

Comparison is base (0dafaf9) 85.33% compared to head (e58ca4c) 85.06%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1125      +/-   ##
==========================================
- Coverage   85.33%   85.06%   -0.28%     
==========================================
  Files          54       57       +3     
  Lines       14740    15437     +697     
==========================================
+ Hits        12579    13132     +553     
- Misses       2161     2305     +144     
Files Coverage Δ
ntp-proto/src/lib.rs 100.00% <ø> (ø)
ntp-proto/src/packet/crypto.rs 90.37% <ø> (ø)
ntp-proto/src/packet/v5/extension_fields.rs 100.00% <100.00%> (ø)
ntp-proto/src/packet/error.rs 41.17% <33.33%> (-0.76%) ⬇️
ntp-proto/src/packet/extension_fields.rs 93.54% <96.37%> (ø)
ntp-proto/src/packet/v5/error.rs 21.42% <21.42%> (ø)
ntp-proto/src/packet/v5/mod.rs 89.14% <89.14%> (ø)
ntp-proto/src/packet/mod.rs 84.18% <24.32%> (-4.80%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

ntp-proto/src/packet/v5.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/v5.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@folkertdev folkertdev left a comment

Choose a reason for hiding this comment

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

cool stuff!

ntp-proto/src/packet/extensionfields.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/extensionfields.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/mod.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/mod.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/v5.rs Outdated Show resolved Hide resolved
@marlonbaeten marlonbaeten marked this pull request as ready for review October 17, 2023 12:41
ntp-proto/Cargo.toml Outdated Show resolved Hide resolved
ntp-proto/src/packet/v5/mod.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/v5/extension_fields.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/v5/mod.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/v5/mod.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/extension_fields.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/extension_fields.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/extension_fields.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/mod.rs Outdated Show resolved Hide resolved
ntp-proto/src/packet/mod.rs Outdated Show resolved Hide resolved
@tdittr
Copy link
Contributor Author

tdittr commented Oct 18, 2023

Replaced by rebased version in #1141

@tdittr tdittr closed this Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants