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

deprecate ca bundle #31

Open
darai0512 opened this issue Jun 22, 2021 · 0 comments
Open

deprecate ca bundle #31

darai0512 opened this issue Jun 22, 2021 · 0 comments

Comments

@darai0512
Copy link
Contributor

概要

SDKがca bundle(ルート証明書リスト)を持つのをやめる。
https://github.com/payjp/payjp-php/tree/master/data

背景

SDKはリクエスト時にSSL関連のエラーが発生すると、バンドル済のcaリストを利用してリトライする。
この仕様にした理由は不明。
またこのリトライ時はpayjp_client_infoに独自メッセージが書かれるが、今年このリトライ経由でのリクエストはなかった。

廃止を推す理由

caリストの参照方法がunix系のパス参照(../.. など)のため、Windows系サーバー利用者では参照エラーが発生してしまう(問い合わせあり)。

リストは4年前から更新されてない。手動でアップデートし続けるのも厳しいと思われる。

上のパス周りを直し、リストを自動アップデート化する価値はあるか?

SSL関連のエラーが、caリストを直すことで解決することは稀であると思われる。過去の問い合わせ対応でも、おおよそルーター設定を見直してもらって解決している。
何よりこの方法でリクエストが成功する状態は不健全である。毎回必ず失敗リクエストを投げ続けるのも不健全。

よって廃止した方が良いと思われる。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant