Skip to content

Conversation

@bagder
Copy link
Member

@bagder bagder commented Sep 17, 2025

It was accidentally broken in commit 0f4c439, shipped since 8.8.0 (May 2024) and yet not a single person has noticed or reported, indicating that we might as well drop support for FTP Kerberos.

Krb5 support was added in 54967d2 (July 2007), and we have been carrying the extra license information around since then for this code. This commit removes the last traces of that code and thus we can remove the extra copyright notices along with it.

Reported-by: Joshua Rogers

@bagder bagder added the FTP label Sep 17, 2025
@github-actions github-actions bot added cmdline tool CI Continuous Integration labels Sep 17, 2025
@bagder bagder changed the title krb5: remove drop support for Kerberos5 FTP Sep 17, 2025
@testclutch

This comment was marked as outdated.

@bagder bagder added the feature-window A merge of this requires an open feature window label Sep 17, 2025
@bagder bagder marked this pull request as ready for review September 17, 2025 08:07
It was accidentally broken in commit 0f4c439, shipped since
8.8.0 (May 2024) and yet not a single person has noticed or reported,
indicating that we might as well drop support for FTP Kerberos.

Krb5 support was added in 54967d2 (July 2007), and we have
been carrying the extra license information around since then for this
code. This commit removes the last traces of that code and thus we can
remove the extra copyright notices along with it.

Reported-by: Joshua Rogers
@github-actions github-actions bot added the tests label Sep 17, 2025
@bagder bagder closed this in 5ab120b Sep 20, 2025
@bagder bagder deleted the bagder/rm-krb5 branch September 20, 2025 21:59
@dfandrich
Copy link
Contributor

dfandrich commented Sep 20, 2025 via email

@xylar xylar mentioned this pull request Nov 5, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous Integration cmdline tool feature-window A merge of this requires an open feature window FTP libcurl API tests

Development

Successfully merging this pull request may close these issues.

3 participants