payjp / payjp-node

PAY.JP server-side JavaScript & Typescirpt SDK :tada: (Note. will remove npm beta tag) Tutorial:

https://pay.jp/docs/started

MIT License

33 stars 8 forks source link

Closed feiz closed 2 years ago

feiz commented 2 years ago

概要

https://pay.jp/docs/api/#rate-limit への対応です。ステータスコード429のレスポンスを受け取った場合に、一定のdelayをはさみつつ再帰的に再リクエストを行います。

optionに maxRetry retryInitialDelay retryMaxDelay を追加。
- maxRetryのデフォルトは0=リトライなし=導入前と同じ動作となります。
typescriptおよび諸々の依存ライブラリをアップグレード

darai0512 commented 2 years ago

仕様面のretryがoptionである点は大賛成（他SDKで強制のものもあるのでoptionに直したい）です。ロジックも問題ないです。 Lintのwarningは頑張れば解決できますが、折を見て...

retryがoptionであるべき理由

既にretry実装している加盟店への悪影響
retryが不要な運用やユーザー体験をしている加盟店への悪影響
- 決済で10秒以上進まないのは利用者的には怖いので、それなら一度エラーとしてあげたい、など
他社でも同じ対応。リトライは基本的にビジネスロジックである
- https://github.com/stripe/stripe-dotnet/issues/1886