Skip to content
This repository was archived by the owner on Jun 5, 2025. It is now read-only.

Make VCertException error messages more helpful #23

Merged
tr1ck3r merged 1 commit into from
Oct 29, 2020
Merged

Make VCertException error messages more helpful #23

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 0 additions & 22 deletions src/main/java/com/venafi/vcert/sdk/VCertClient.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,8 +103,6 @@ public void ping() throws VCertException {
connector.ping();
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -117,8 +115,6 @@ public void authenticate(Authentication auth) throws VCertException {
connector.authenticate(auth);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -131,8 +127,6 @@ public ZoneConfiguration readZoneConfiguration(String zone) throws VCertExceptio
return connector.readZoneConfiguration(zone);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -146,8 +140,6 @@ public CertificateRequest generateRequest(ZoneConfiguration config, CertificateR
return connector.generateRequest(config, request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -157,8 +149,6 @@ public String requestCertificate(CertificateRequest request, String zone) throws
return connector.requestCertificate(request, zone);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -172,8 +162,6 @@ public String requestCertificate(CertificateRequest request, ZoneConfiguration z
return connector.requestCertificate(request, zoneConfiguration);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -186,8 +174,6 @@ public PEMCollection retrieveCertificate(CertificateRequest request) throws VCer
return connector.retrieveCertificate(request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -200,8 +186,6 @@ public void revokeCertificate(RevocationRequest request) throws VCertException {
connector.revokeCertificate(request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -214,8 +198,6 @@ public String renewCertificate(RenewalRequest request) throws VCertException {
return connector.renewCertificate(request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -228,8 +210,6 @@ public ImportResponse importCertificate(ImportRequest request) throws VCertExcep
return connector.importCertificate(request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -242,8 +222,6 @@ public Policy readPolicyConfiguration(String zone) throws VCertException {
return connector.readPolicyConfiguration(zone);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}
}
26 changes: 1 addition & 25 deletions src/main/java/com/venafi/vcert/sdk/VCertTknClient.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,6 @@ public TokenInfo getAccessToken(Authentication auth) throws VCertException{
return connector.getAccessToken(auth);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -113,13 +111,11 @@ public TokenInfo getAccessToken() throws VCertException{
return connector.getAccessToken();
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

@Override
public TokenInfo refreshAccessToken(String applicationId) throws VCertException{
public TokenInfo refreshAccessToken(String applicationId) throws VCertException {
return connector.refreshAccessToken(applicationId);
}

Expand All @@ -137,8 +133,6 @@ public void ping() throws VCertException {
connector.ping();
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -151,8 +145,6 @@ public ZoneConfiguration readZoneConfiguration(String zone) throws VCertExceptio
return connector.readZoneConfiguration(zone);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -166,8 +158,6 @@ public CertificateRequest generateRequest(ZoneConfiguration config, CertificateR
return connector.generateRequest(config, request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -177,8 +167,6 @@ public String requestCertificate(CertificateRequest request, String zone) throws
return connector.requestCertificate(request, zone);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -192,8 +180,6 @@ public String requestCertificate(CertificateRequest request, ZoneConfiguration z
return connector.requestCertificate(request, zoneConfiguration);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -206,8 +192,6 @@ public PEMCollection retrieveCertificate(CertificateRequest request) throws VCer
return connector.retrieveCertificate(request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -220,8 +204,6 @@ public void revokeCertificate(RevocationRequest request) throws VCertException {
connector.revokeCertificate(request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -234,8 +216,6 @@ public String renewCertificate(RenewalRequest request) throws VCertException {
return connector.renewCertificate(request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -248,8 +228,6 @@ public ImportResponse importCertificate(ImportRequest request) throws VCertExcep
return connector.importCertificate(request);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}

Expand All @@ -262,8 +240,6 @@ public Policy readPolicyConfiguration(String zone) throws VCertException {
return connector.readPolicyConfiguration(zone);
} catch (FeignException e) {
throw VCertException.fromFeignException(e);
} catch (Exception e) {
throw new VCertException("Unexpected exception", e);
}
}
}