Skip to content

Commit

Permalink
curl_tool: Added krb5 to the supported features
Browse files Browse the repository at this point in the history
  • Loading branch information
@captain-caveman2k
captain-caveman2k committed Nov 7, 2014
1 parent eda919f commit e7bcfb3
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions src/tool_help.c
View file
Expand Up @@ -269,6 +269,7 @@ static const struct feat feats[] = {
{"NTLM_WB", CURL_VERSION_NTLM_WB},
{"SSL", CURL_VERSION_SSL},
{"krb4", CURL_VERSION_KERBEROS4},
{"krb5", CURL_VERSION_KERBEROS5},
{"libz", CURL_VERSION_LIBZ},
{"CharConv", CURL_VERSION_CONV},
{"TLS-SRP", CURL_VERSION_TLSAUTH_SRP},
Expand Down

1 comment on commit e7bcfb3

@michael-o
Copy link
Contributor

Choose a reason for hiding this comment

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

I would rather call that Kerberos, USE_KERBEROS, CURL_VERSION_KERBEROS, etc. Nothing else is known these says. Kerberos 4 is virtually dead. Kerberos 5 de facto standard now. Krb4 is about to be removed from MIT Kerberos and Heimdal, and from curl. Windows does not support it at all.

Should I create a PR for all previous commits to clean that up?

Please sign in to comment.