Skip to content

Commit

Permalink
Increase dnssd kHostNameMaxLength to 40 bytes
Browse files Browse the repository at this point in the history
  • Loading branch information
yunhanw-google committed May 15, 2024
1 parent f4e02d4 commit 14422a2
Showing 1 changed file with 6 additions and 1 deletion.
7 changes: 6 additions & 1 deletion src/lib/dnssd/Constants.h
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,12 @@ namespace Dnssd {
* Matter DNS host settings
*/

inline constexpr size_t kHostNameMaxLength = 16; // MAC or 802.15.4 Extended Address in hex
// Matter spec expects hostname to be MAC or 802.15.4 Extended Address in hex.
// But in latest android nsdManager, it would set hostname with 40 bytes with prefix as android_,
// and there is no existing API to update the hostname, therefore we put temporary workaround with 40 bytes.
// Follow-up with ticket issue https://github.com/project-chip/connectedhomeip/issues/33474
inline constexpr size_t kHostNameMaxLength = 40;
//

/*
* Matter DNS service subtypes
Expand Down

0 comments on commit 14422a2

Please sign in to comment.