IronCoreLabs / IronCoreLabs/tenant-security-client-java

Add an `isTransient()` or something similar to `TenantSecurityException`

Open
#75 0 comments 0 reactions 0 assignees View on GitHub
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.