-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Default to use an IPV4 address #25668
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
pullapprove
bot
requested review from
amitnj,
andy31415,
anush-apple,
arkq,
bzbarsky-apple,
carol-apple,
chrisdecenzo,
chshu,
chulspro,
Damian-Nordic,
dhrishi,
electrocucaracha,
emargolis,
franck-apple,
gjc13,
harimau-qirex,
harsha-rajendran,
hawk248,
jelderton,
jepenven-silabs,
jmartinez-silabs,
jmeg-sfy,
joonhaengHeo,
jtung-apple,
kcoppock,
kkasperczyk-no,
ksperling-apple and
lazarkov
March 13, 2023 21:12
pullapprove
bot
requested review from
woody-apple,
xylophone21,
younghak-hwang,
yufengwangca and
yunhanw-google
March 13, 2023 21:12
sharadb-amazon
approved these changes
Mar 13, 2023
PR #25668: Size comparison from d04e396 to 8dc3883 Increases (6 builds for bl702, cc13x2_26x2, nrfconnect, psoc6)
Decreases (3 builds for cc13x2_26x2, efr32)
Full report (34 builds for bl602, bl702, cc13x2_26x2, cc32xx, cyw30739, efr32, k32w, linux, mbed, nrfconnect, psoc6, qpg)
|
cliffamzn
force-pushed
the
ipv4
branch
3 times, most recently
from
March 14, 2023 03:18
50f3754
to
4185cd5
Compare
PR #25668: Size comparison from 1ce3a1e to 4185cd5 Decreases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
PR #25668: Size comparison from fa2e709 to 6203fd5 Decreases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
PR #25668: Size comparison from 0c5d81d to 917d926 Full report (1 build for cc32xx)
|
PR #25668: Size comparison from e778a03 to 930683f Increases (1 build for psoc6)
Decreases (5 builds for bl602, bl702, efr32, psoc6)
Full report (31 builds for bl602, bl702, cc13x2_26x2, cc32xx, cyw30739, efr32, k32w, linux, mbed, psoc6, qpg)
|
chrisdecenzo
approved these changes
Mar 16, 2023
On some devices it appears that the commissioner may not have an appropriate IPV6 address and attempts to send a user directed commissioning request to a device using its IPV6 address. Its a bit unclear how this situation happens but in theory it could occur. In most cases this should be safe to default to an ipv4 address if it is available. The implementation of the user directed commissioning server does something similar.
PR #25668: Size comparison from 42a607c to ad98cf4 Decreases (1 build for cc32xx)
Full report (1 build for cc32xx)
|
yufengwangca
approved these changes
Mar 16, 2023
lecndav
pushed a commit
to lecndav/connectedhomeip
that referenced
this pull request
Mar 22, 2023
* Default to use an IPV4 address On some devices it appears that the commissioner may not have an appropriate IPV6 address and attempts to send a user directed commissioning request to a device using its IPV6 address. Its a bit unclear how this situation happens but in theory it could occur. In most cases this should be safe to default to an ipv4 address if it is available. The implementation of the user directed commissioning server does something similar. * Expand on the error messaging to clarify ipv4 selection
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
On some devices it appears that the commissioner may not have an appropriate IPV6 address and attempts to send a user directed commissioning request to a device using its IPV6 address.
Its a bit unclear how this situation happens but in theory it could occur. In most cases this should be safe to default to an ipv4 address if it is available.
The implementation of the user directed commissioning server does something similar.
In the test setup we had, an iOS device was acting as the commissionee (sending the UDC packet). One interface (the loop back) has both an IPv6 and IPv4 address, the WiFi interface however only had an IPv4 address.
We observed the iOS device attempting to send the UDC packet back over the IPv6 address to the commissioner but tracing over wireshark showed no packets being sent.
This setup could only be reproduced with one particular travel router and the same scenario could not be reproduced on other setups.