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

[ICD]Add SAT Common TXT key #28123

Merged
merged 9 commits into from
Jul 21, 2023
Merged

Conversation

jmartinez-silabs
Copy link
Member

@jmartinez-silabs jmartinez-silabs commented Jul 20, 2023

fixes #28025
Add the Session Active Threshold key in the operational discovery advertisement

When the ICD server is enabled on the device, It uses its Active Mode Threshold attribute else it defaults to 4000 milliseconds

The Retransmission mechanisms also use that value as minimal Active Time (default stays like before at 4 seconds)

Copy link
Contributor

@mkardous-silabs mkardous-silabs left a comment

Choose a reason for hiding this comment

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

LGTM!

src/lib/dnssd/tests/TestTxtFields.cpp Outdated Show resolved Hide resolved
src/lib/dnssd/tests/TestTxtFields.cpp Outdated Show resolved Hide resolved
src/messaging/ReliableMessageProtocolConfig.h Show resolved Hide resolved
@mergify mergify bot merged commit 47bd885 into project-chip:master Jul 21, 2023
@jmartinez-silabs jmartinez-silabs deleted the add_SAT_key branch July 21, 2023 13:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

[ICD] Implement the Session Active Threshold Discovery Key
5 participants