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.

Sign up for GitHub

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

Jump to bottom

Resolved SIGSEGV error by lowering the version. #117

Merged

cateiru merged 1 commit into main from fix/pw-hash

Jul 19, 2022

Member

cateiru commented Jul 19, 2022 •

edited

Loading

issue:

🔧 修正点

ref. ranisalt/node-argon2#302

Alpine Linuxでargon2を実行するとSIGSEGV errorで異常終了してしまう問題があったため、バージョンを0.27.2に下げた。

& Dockerfileを少し修正した。

✅ 自己チェック

動作確認ができているか
新しく追加したメソッドにユニットテストを追加しているか
静的解析、テストは成功しているか
Files changedを自分で確認してミスがないか

📸 スクリーンショット（オプション）

⚠️ レビュー時に注意して欲しいこと（オプション）

🗣️ 補足など（オプション）


          https://github.com/ranisalt/node-argon2/issues/302

c961d2b

cateiru self-assigned this

cateiru merged commit e87fcb6 into main

cateiru deleted the fix/pw-hash branch

July 19, 2022 15:09

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet