Skip to content

tests: require IPv6 for 1265, 1324, 2086#17014

Closed
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:t-fix-noipv6
Closed

tests: require IPv6 for 1265, 1324, 2086#17014
vszakats wants to merge 1 commit into
curl:masterfrom
vszakats:t-fix-noipv6

Conversation

@vszakats
Copy link
Copy Markdown
Member

@vszakats vszakats commented Apr 10, 2025

And also require HTTP. Also add IPv6 to the keywords.

Fixing:

Linux AM openssl !ipv6 !--libcurl:

FAIL 1265: 'NO_PROXY with IPv6 numerical address' HTTP, HTTP proxy, http_proxy, NO_PROXY, noproxy
FAIL 1324: 'HTTP with --resolve and [ipv6address]' HTTP, HTTP GET, --resolve
FAIL 2086: 'Pre-request callback for HTTP IPv6' HTTP, IPv6

Ref: https://github.com/curl/curl/actions/runs/14378524385/job/40318328714?pr=17012#step:41:3789

Follow-up to a09e491 #17005

Fixing:

Linux AM openssl !ipv6 !--libcurl:
```
FAIL 1265: 'NO_PROXY with IPv6 numerical address' HTTP, HTTP proxy, http_proxy, NO_PROXY, noproxy
FAIL 1324: 'HTTP with --resolve and [ipv6address]' HTTP, HTTP GET, --resolve
FAIL 2086: 'Pre-request callback for HTTP IPv6' HTTP, IPv6
```
Ref: https://github.com/curl/curl/actions/runs/14378524385/job/40318328714?pr=17012#step:41:3789
@vszakats vszakats changed the title tests: require IPv6 for 1247, 1324, 2086 tests: require IPv6 for 1265, 1324, 2086 Apr 10, 2025
@github-actions github-actions Bot added the tests label Apr 10, 2025
@vszakats
Copy link
Copy Markdown
Member Author

vszakats commented Apr 10, 2025

Not sure, why this did not come up earlier. Might be a side-effect of?
a09e491 #17005

This is where these 3 started to fail, also within the PR. Hm, is this possible?

@vszakats
Copy link
Copy Markdown
Member Author

The macOS fallout is unrelated, some more info here:
#17012 (comment)

Copy link
Copy Markdown
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

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

Yeah, I think these simply failed to start previously and thus ran under the radar.

@vszakats vszakats closed this in 0e8bf75 Apr 10, 2025
@vszakats vszakats deleted the t-fix-noipv6 branch April 10, 2025 14:25
nbaws pushed a commit to nbaws/curl that referenced this pull request Apr 26, 2025
And also require HTTP. Also add `IPv6` to the keywords.

Fixing:

Linux AM openssl !ipv6 !--libcurl:
```
FAIL 1265: 'NO_PROXY with IPv6 numerical address' HTTP, HTTP proxy, http_proxy, NO_PROXY, noproxy
FAIL 1324: 'HTTP with --resolve and [ipv6address]' HTTP, HTTP GET, --resolve
FAIL 2086: 'Pre-request callback for HTTP IPv6' HTTP, IPv6
```
Ref: https://github.com/curl/curl/actions/runs/14378524385/job/40318328714?pr=17012#step:41:3789

Follow-up to a09e491 curl#17005

Closes curl#17014
nbaws pushed a commit to nbaws/curl that referenced this pull request Apr 26, 2025
And also require HTTP. Also add `IPv6` to the keywords.

Fixing:

Linux AM openssl !ipv6 !--libcurl:
```
FAIL 1265: 'NO_PROXY with IPv6 numerical address' HTTP, HTTP proxy, http_proxy, NO_PROXY, noproxy
FAIL 1324: 'HTTP with --resolve and [ipv6address]' HTTP, HTTP GET, --resolve
FAIL 2086: 'Pre-request callback for HTTP IPv6' HTTP, IPv6
```
Ref: https://github.com/curl/curl/actions/runs/14378524385/job/40318328714?pr=17012#step:41:3789

Follow-up to a09e491 curl#17005

Closes curl#17014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants