Skip to content

Csptp proto#2266

Merged
rnijveld merged 6 commits into
mainfrom
csptp-proto
Jul 8, 2026
Merged

Csptp proto#2266
rnijveld merged 6 commits into
mainfrom
csptp-proto

Conversation

@davidv1992

Copy link
Copy Markdown
Member

No description provided.

@davidv1992 davidv1992 changed the base branch from main to organization-rework May 12, 2026 11:46
@davidv1992 davidv1992 linked an issue Jun 12, 2026 that may be closed by this pull request
@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 30.20457% with 580 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.35%. Comparing base (29e6394) to head (79711a9).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
statime-csptp/src/source.rs 0.00% 259 Missing ⚠️
statime-csptp/src/messages.rs 0.00% 180 Missing ⚠️
statime-csptp/src/server.rs 0.00% 75 Missing ⚠️
statime-csptp/src/manager.rs 0.00% 48 Missing ⚠️
statime-csptp/src/platform.rs 0.00% 18 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2266      +/-   ##
==========================================
- Coverage   84.51%   82.35%   -2.17%     
==========================================
  Files          95      101       +6     
  Lines       21608    22414     +806     
==========================================
+ Hits        18261    18458     +197     
- Misses       3347     3956     +609     
Flag Coverage Δ
fuzz-cookie_parsing_sound 0.61% <ø> (ø)
fuzz-duration_from_float 0.26% <ø> (ø)
fuzz-encrypted_client_parsing 7.19% <ø> (-0.13%) ⬇️
fuzz-encrypted_server_parsing 9.88% <ø> (-0.10%) ⬇️
fuzz-handle 11.08% <ø> (-4.97%) ⬇️
fuzz-ipfilter 2.49% <ø> (ø)
fuzz-key_exchange_request_parsing 3.81% <ø> (-1.24%) ⬇️
fuzz-key_exchange_response_parsing 3.67% <ø> (-0.02%) ⬇️
fuzz-packet_keyset 5.65% <ø> (ø)
fuzz-packet_parsing_sound 6.08% <ø> (-1.92%) ⬇️
fuzz-ptp_message_parsing 59.35% <80.00%> (-5.27%) ⬇️
fuzz-record_encode_decode 4.39% <ø> (-0.04%) ⬇️
test-aarch64-apple-darwin 81.29% <29.60%> (-1.92%) ⬇️
test-x86_64-unknown-linux-gnu 81.32% <29.60%> (-1.90%) ⬇️
test-x86_64-unknown-linux-musl 81.21% <29.60%> (-1.91%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@davidv1992 davidv1992 marked this pull request as ready for review June 30, 2026 11:26
@davidv1992 davidv1992 force-pushed the organization-rework branch from c2b9c25 to 0f711b0 Compare June 30, 2026 11:35
Base automatically changed from organization-rework to main June 30, 2026 12:04
This allows writing serialization code for tlvs outside of the
statime-wire crate. This is usefull as those tlvs are not always general
accross PTP variants.
This allows external crates to actually create and read messages.
This wraps the PTP message type to enforce the PTP messages to be valid
CSPTP requests, responses or follow up messages.
@davidv1992 davidv1992 force-pushed the csptp-proto branch 2 times, most recently from 6bbc21a to 1e8bb85 Compare July 1, 2026 07:07
@rnijveld rnijveld added this pull request to the merge queue Jul 8, 2026
Merged via the queue into main with commit 11854dd Jul 8, 2026
37 checks passed
@rnijveld rnijveld deleted the csptp-proto branch July 8, 2026 13:20
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.

Implement basic request-response part of CSPTP protocol

2 participants