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

gskit: remove #11460

Closed
wants to merge 1 commit into from
Closed

gskit: remove #11460

wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Jul 18, 2023

Motivation

We remove support for building curl with the gskit TLS library in August 2023.

  • This is a niche TLS library, only running on some IBM systems
  • no regular curl contributors use this backend
  • no CI builds use or verify this backend
  • gskit, or the curl adaption for it, lacks many modern TLS features making it an inferior solution
  • build breakages in this code take weeks or more to get detected
  • fixing gskit code is mostly done "flying blind"

Discussion

Debated here: https://curl.se/mail/lib-2023-07/0015.html

To bring it back?

This backend could be brought back to supported status if we see efforts into making sure this backend reaches similar level of support and attention as other TLS backends do. Probably including both support for modern TLS features and CI builds.

@bagder bagder added TLS feature-window A merge of this requires an open feature window labels Jul 18, 2023
@bagder bagder closed this in 78d6232 Aug 7, 2023
@bagder bagder deleted the bagder/rm-gskit branch August 7, 2023 19:00
ptitSeb pushed a commit to wasix-org/curl that referenced this pull request Sep 25, 2023
We remove support for building curl with gskit.

 - This is a niche TLS library, only running on some IBM systems
 - no regular curl contributors use this backend
 - no CI builds use or verify this backend
 - gskit, or the curl adaption for it, lacks many modern TLS features
   making it an inferior solution
 - build breakages in this code take weeks or more to get detected
 - fixing gskit code is mostly done "flying blind"

This removal has been advertized in DEPRECATED in Jan 2, 2023 and it has
been mentioned on the curl-library mailing list.

It could be brought back, this is not a ban. Given proper effort and
will, gskit support is welcome back into the curl TLS backend family.

Closes curl#11460
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cmdline tool feature-window A merge of this requires an open feature window libcurl API tests TLS
Development

Successfully merging this pull request may close these issues.

None yet

1 participant