Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

OpenThread: avoid hardcoded Golioth IPv6 address #449

Merged
merged 1 commit into from
Jan 5, 2024
Merged

Conversation

MarkoPura
Copy link
Contributor

Avoid hardcoding the IPv6 Golioth System Server address by leveraging OpenThread DNS to retrieve the address before connecting to Golioth Cloud. As a side benefit, NAT64 prefix used by the Thred Border Router is set while synthesizing the address.

Copy link

github-actions bot commented Dec 21, 2023

Visit the preview URL for this PR (updated for commit 938082f):

https://golioth-zephyr-sdk-doxygen-dev--pr449-thread-dns-5uz8k9ea.web.app

(expires Fri, 12 Jan 2024 14:00:48 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a389eefadf4b4b68a539327b3459dd66c142cf49

@MarkoPura MarkoPura self-assigned this Dec 21, 2023
net/golioth/system_client.c Outdated Show resolved Hide resolved
net/golioth/system_client.c Outdated Show resolved Hide resolved
@MarkoPura MarkoPura force-pushed the thread_dns branch 4 times, most recently from 901be12 to bfccbc3 Compare January 5, 2024 13:50
Leverage the OpenThread DNS to synthesize the Golioth System Server
IPv6 address avoiding hardcoded IP address in applications.
NAT64 prefix used by the Thred Border Router is set while
synthesizing the address.

Signed-off-by: Marko Puric <[email protected]>
@MarkoPura MarkoPura marked this pull request as ready for review January 5, 2024 14:02
@MarkoPura MarkoPura requested a review from sam-golioth January 5, 2024 14:02
Copy link
Contributor

@sam-golioth sam-golioth left a comment

Choose a reason for hiding this comment

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

Looks good!

@MarkoPura MarkoPura merged commit 42408c7 into main Jan 5, 2024
10 of 12 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants