WebHook URLが接続できない場合のエラー処理が適切でない可能性がある

[y-natsume-d]

本プラグインをインストール後にテスト用にWebHookのURLを追加してみたところ、URLの入力ミスで接続できず、WebHookへのリクエスト送信時に以下のエラーが画面に出力されました。

Argument 1 passed to Plugin\Api\Service\WebHookService::Plugin\Api\Service\{closure}() must be an instance of GuzzleHttp\Exception\RequestException, instance of GuzzleHttp\Exception\ConnectException given

該当コードでは RequestException を想定しているのですが、その手前でエラーになった場合 ConnectException が渡ってくる可能性があるので \Exception を指定したほうが良いかと思いますが、エラーハンドリングの想定スコープが分からなかったためissueのみ上げさせていただきます。

環境

eccube/api 2.1.3

[y-natsume-d]

webhookに登録したエンドポイントがシステムダウンしてconnection errorになる場合も上記エラーが返ってきてしまい、結果的にメインのリクエストまで500エラーになってしまうので良くないと感じました。

[kiy0taka]

4.2では対応済み。#128