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

huge sync with main branch #1120

Merged
merged 144 commits into from
Nov 29, 2024
Merged

huge sync with main branch #1120

merged 144 commits into from
Nov 29, 2024

Conversation

JaylinYu
Copy link
Member

No description provided.

JaylinYu and others added 30 commits September 5, 2024 16:53
… reset timestamp of duplicated msg

Signed-off-by: jaylin <[email protected]>
work with ea5cf0c50229fba8d742326cf387a95e1e075976

Signed-off-by: jaylin <[email protected]>
gdamore and others added 20 commits October 29, 2024 18:00
This will be used in UDP.  It also lets us reduce some unnecessary
code paths for redundant library initialization.
This adds endian awareness at compile time, and defines some little
endian versions of some macros. We antiicpate making more use of
little endian in new protocols to reduce the "endian tax", as nearly
every modern system is little endian these days.
This will be used for some follow up work (UDP).
Copy link
Member Author

@JaylinYu JaylinYu left a comment

Choose a reason for hiding this comment

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

will tag then merge

run: cmake ${{ steps.vcpkg.outputs.vcpkg-cmake-config }} -D NNG_ENABLE_TLS=ON -B build
# TLS on Windows is not good because vcpkg is *old* mbedTLS 2.28, and we don't pass some newer tests.
# We should explore testing WolfSSL here instead.
run: cmake ${{ steps.vcpkg.outputs.vcpkg-cmake-config }} -B build
Copy link
Member Author

Choose a reason for hiding this comment

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

we shall stick to old 2.28 in nanonng

Copy link

codecov bot commented Nov 27, 2024

Codecov Report

Attention: Patch coverage is 44.89796% with 27 lines in your changes missing coverage. Please review.

Project coverage is 63.30%. Comparing base (e7108f2) to head (9bf4b32).
Report is 33 commits behind head on sdv_master.

Files with missing lines Patch % Lines
src/sp/protocol/mqtt/nmq_mqtt.c 0.00% 5 Missing ⚠️
src/sp/transport/mqtt/broker_tcp.c 44.44% 5 Missing ⚠️
src/sp/transport/mqtts/broker_tls.c 0.00% 3 Missing ⚠️
src/supplemental/mqtt/mqtt_msg.c 0.00% 3 Missing ⚠️
src/supplemental/mqtt/mqtt_qos_db_api.c 0.00% 3 Missing ⚠️
src/core/idhash.c 86.66% 2 Missing ⚠️
src/supplemental/util/idhash.c 50.00% 2 Missing ⚠️
src/core/tcp.c 50.00% 1 Missing ⚠️
src/core/url.c 0.00% 1 Missing ⚠️
src/supplemental/tls/mbedtls/tls.c 0.00% 1 Missing ⚠️
... and 1 more
Additional details and impacted files
@@              Coverage Diff               @@
##           sdv_master    #1120      +/-   ##
==============================================
- Coverage       63.48%   63.30%   -0.18%     
==============================================
  Files             135      135              
  Lines           39291    39460     +169     
==============================================
+ Hits            24945    24982      +37     
- Misses          14346    14478     +132     

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

@JaylinYu JaylinYu merged commit 3375b52 into sdv_master Nov 29, 2024
10 of 15 checks passed
@JaylinYu JaylinYu deleted the jaylin/sdv_master branch November 29, 2024 08:23
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.

5 participants