-
Notifications
You must be signed in to change notification settings - Fork 198
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
remove LegacyPasswd
code and dependency on crypt
#1153
Comments
peterbecich
changed the title
Should
Should Jan 3, 2023
crypt
be replaced with libcrypt
?crypt
be replaced with something else?
peterbecich
changed the title
Should
Can Jan 3, 2023
crypt
be replaced with something else?libcrypt
be replaced with something else?
The easiest way to solve this is by deleting the "old-style crypt() auth", if possible: hackage-server/src/Distribution/Server/Features/LegacyPasswds/Auth.hs Lines 29 to 43 in 5f5b814
|
peterbecich
changed the title
Can
Can Jan 3, 2023
libcrypt
be replaced with something else?libcrypt
be replaced with something else? Or delete "old-style crypt() auth"?
peterbecich
changed the title
Can
Can Jan 3, 2023
libcrypt
be replaced with something else? Or delete "old-style crypt() auth"?libcrypt
be replaced with something else? Or delete "old-style crypt()
auth"?
At this point I think just ripping out the legacypasswd stuff is acceptable -- the migration has been over for a long time now. PR welcome. |
peterbecich
changed the title
Can
remove Jan 9, 2023
libcrypt
be replaced with something else? Or delete "old-style crypt()
auth"?LegacyPasswd
code and dependency on crypt
peterbecich
added a commit
to peterbecich/hackage-server
that referenced
this issue
Jan 16, 2023
peterbecich
added a commit
to peterbecich/hackage-server
that referenced
this issue
Feb 4, 2023
peterbecich
added a commit
to peterbecich/hackage-server
that referenced
this issue
Feb 4, 2023
gbaz
pushed a commit
that referenced
this issue
Feb 24, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
hackage-server
has a dependency on theextra-library
crypt
:hackage-server/hackage-server.cabal
Lines 443 to 444 in 5f5b814
I believe that refers to this function: https://en.wikipedia.org/wiki/Crypt_(C)
It appears that
crypt
is deprecated:https://www.gentoo.org/support/news-items/2021-10-18-libxcrypt-migration-stable.html
It also looks like Nix is moving away from
libcrypt
to a different librarylibxcrypt
: NixOS/nixpkgs#181764The usage of the library in
hackage-server
is here:hackage-server/src/Distribution/Server/Features/LegacyPasswds/Auth.hs
Line 49 in c3312b0
Can we replace
libcrypt
withlibxcrypt
, or replace thisforeign import ccall
with some other library likeThe text was updated successfully, but these errors were encountered: