IronCoreLabs / IronCoreLabs/tenant-security-client-java
Add an `isTransient()` or something similar to `TenantSecurityException`
- Dominant language
- Java
- Stars
- 3
- Forks
- 4
- Avg merge
- 50m
- Merged PRs (30d)
- 4
Description
Adding an `isTransient()` method to [TenantSecurityException](https://github.com/IronCoreLabs/tenant-security-client-java/blob/59f06568cfc6b4ea5b3e0ba7d21a55dc37372500/src/main/java/com/ironcorelabs/tenantsecurity/kms/v1/exception/TenantSecurityException.java#L12) would make it a lot more convenient for consumers of the library to know if they should retry an operation or not.
The major unknown would be what to do for things that are on the borderline, like a KMS auth error, which may go away on retry if the tenant happens to change something on the KMS side, but likely won't until a later time.
Contributor guide
Research direction
Start with src/main/java/com/ironcorelabs/tenantsecurity/kms/v1/exception/TenantSecurityException.java and review how its exceptions represent failures. Define which failures should be considered retryable, including the borderline KMS authentication error, and make the completed behavior clear for library consumers.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- security
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 25/100