This repository was archived by the owner on Dec 3, 2023. It is now read-only.
- Notifications
You must be signed in to change notification settings - Fork 36
This repository was archived by the owner on Dec 3, 2023. It is now read-only.
SSLException isn't retryable #182
Copy link
Copy link
Closed
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.
Description
Hi,
Storage Java library users are failing on a retryable exception SocketException: Socket closed' errors
which raised as a javax.net.ssl.SSLException
. SSLException isn't considered retryable in BaseServiceException.java.
Are there any concerns with making this exception retryable along with:
- SocketTimeoutException
- SocketException
- SSLHandshakeException
I'll send a PR to introduce this change for discussion.
Metadata
Metadata
Assignees
Labels
priority: p2Moderately-important priority. Fix may not be included in next release.Moderately-important priority. Fix may not be included in next release.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.Error or flaw in code with unintended results or allowing sub-optimal usage patterns.