Skip to content

test 1560: test RFC4291 style IPv6 IPv4-mapped addresses#22136

Closed
bagder wants to merge 5 commits into
masterfrom
bagder/test-rfc4291
Closed

test 1560: test RFC4291 style IPv6 IPv4-mapped addresses#22136
bagder wants to merge 5 commits into
masterfrom
bagder/test-rfc4291

Conversation

@bagder

@bagder bagder commented Jun 22, 2026

Copy link
Copy Markdown
Member

No description provided.

@bagder

bagder commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

It appear's musl's inet_ntop() call does not output ::[ipv4] like the others do...

bagder added 4 commits June 22, 2026 16:31
…d format

An attempt to detect a non-compatible inet_ntop() call for IPv6 output.
@bagder bagder marked this pull request as ready for review June 22, 2026 14:58
@bagder

bagder commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

@vszakats you feel like helping me make a cmake version of this test? Without it, the cmake build will fail test 1560 when using musl, and possibly other libc implementations without support for this.

@vszakats

Copy link
Copy Markdown
Member

Sure, looking into it.

@testclutch

Copy link
Copy Markdown

Analysis of PR #22136 at 83f54d13:

Test 3106 failed, which has NOT been flaky recently, so there could be a real issue in this PR. Note that this test has failed in 12 different CI jobs (the link just goes to one of them). Note that this CI job has had a number of other flaky tests recently (2, to be specific) so it may be that this failure is rather a systemic issue with this job and not with this specific PR.

Generated by Testclutch

@bagder

bagder commented Jun 22, 2026

Copy link
Copy Markdown
Member Author

It can be noted that POSIX does not actually mention the ::127.0.0.1 style glibc and others (and curl's internal version) use.

The musl inet_ntop implementation makes no attempt to.

@vszakats

Copy link
Copy Markdown
Member

FYI working on the CMake parts in → #22137

@vszakats

Copy link
Copy Markdown
Member

I think we should merge this first and I'll follow-up with #22137 after.

@bagder

bagder commented Jun 25, 2026

Copy link
Copy Markdown
Member Author

Ok, let's do that! 👍

@bagder bagder closed this in bf5cc6a Jun 25, 2026
bagder added a commit that referenced this pull request Jun 25, 2026
@bagder bagder deleted the bagder/test-rfc4291 branch June 25, 2026 09:03
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.

3 participants