department-of-veterans-affairs / connect_vbms

VBMS SOAP API client
Other
17 stars 9 forks source link

Add retry attempt to all VBMS requests on SSL error #156

Closed joofsh closed 7 years ago

joofsh commented 7 years ago

SSL errors are causing ~.1% of requests to VBMS to fail. It is unclear the timeline for VBMS to fix this. In the meantime, we should add a 1 attempt retry to all our VBMS requests.

Related sentry errors: https://sentry.ds.va.gov/department-of-veterans-affairs/caseflow-certification/issues/391/ https://sentry.ds.va.gov/department-of-veterans-affairs/efolder/issues/376/ https://sentry.ds.va.gov/department-of-veterans-affairs/caseflow-certification/issues/409/

joofsh commented 7 years ago

Haven't seen an SSL error in 24 hours. Closing this issue

mmlumba commented 7 years ago

More related SSL issues: https://sentry.ds.va.gov/department-of-veterans-affairs/caseflow-certification/issues/368/ https://sentry.ds.va.gov/department-of-veterans-affairs/efolder/issues/436/ https://sentry.ds.va.gov/department-of-veterans-affairs/efolder/issues/366/