Skip to content
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

vtls: deduplicate some DISABLE_PROXY ifdefs #5735

Closed
wants to merge 1 commit into from

Conversation

kdudka
Copy link
Contributor

@kdudka kdudka commented Jul 27, 2020

... in the code of gtls, nss, and openssl (for now)

@bagder
Copy link
Member

bagder commented Sep 30, 2020

@kdudka can you rebase and force-push this to see if the CI failures are "real"?

... in the code of gtls, nss, and openssl (for now)
@kdudka
Copy link
Contributor Author

kdudka commented Sep 30, 2020

Done. I did not know this was waiting for CI to become green.

Btw. is it expected that HTTPS-proxy is advertised in SUPPORT_FEATURES when curl is configured with --disable-proxy?

bagder added a commit that referenced this pull request Sep 30, 2020
bagder added a commit that referenced this pull request Sep 30, 2020
Reported-by: Kamil Dudka
Reviewed-by: Kamil Dudka
Bug: #5735 (comment)
Closes #6029
@kdudka
Copy link
Contributor Author

kdudka commented Oct 1, 2020

I see no related CI failures. CI results of the current master branch look more or less the same: random infrastructure failures on macos and windows.

@bagder
Copy link
Member

bagder commented Oct 1, 2020

Thanks!

@bagder bagder closed this in 7920be9 Oct 1, 2020
@kdudka kdudka deleted the vtls-proxy branch October 1, 2020 14:52
MAntoniak added a commit to MAntoniak/curl that referenced this pull request Feb 25, 2021
- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other tls backend
- create SSL_HOST_PORT
MAntoniak added a commit to MAntoniak/curl that referenced this pull request Feb 25, 2021
- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other tls backend
- create SSL_HOST_PORT
MAntoniak added a commit to MAntoniak/curl that referenced this pull request Feb 25, 2021
- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other tls backend
- create SSL_HOST_PORT
MAntoniak added a commit to MAntoniak/curl that referenced this pull request Apr 12, 2021
- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other tls backend
- create SSL_HOST_PORT
MAntoniak added a commit to MAntoniak/curl that referenced this pull request Apr 12, 2021
- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other tls backend
- create SSL_HOST_PORT
MAntoniak added a commit to MAntoniak/curl that referenced this pull request Apr 15, 2021
- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other tls backend
- create SSL_HOST_PORT
MAntoniak added a commit to MAntoniak/curl that referenced this pull request Apr 15, 2021
- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other tls backend
- create SSL_HOST_PORT
MAntoniak added a commit to MAntoniak/curl that referenced this pull request Apr 22, 2021
- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other tls backend
- create SSL_HOST_PORT
bagder pushed a commit that referenced this pull request Apr 22, 2021
continue from #5735

- using SSL_HOST_NAME, SSL_HOST_DISPNAME, SSL_PINNED_PUB_KEY for other
  tls backend

- create SSL_HOST_PORT

Closes #6660
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants