Null access fix for BaseClient exception handlers. #649

Merged
merged 1 commit into from Jun 8, 2017

Conversation

Projects
None yet
3 participants
@maclema
Contributor

maclema commented May 16, 2017

Fix for null access on httpRequest in DataFormatException and IOException handlers - passing UNKNOWN as the verb and URI when the httpRequest is null, this is better than a null access being thrown when calling .execute()

Update BaseClient.java
Fix for null access on httpRequest in DataFormatException and IOException handlers.
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls May 16, 2017

Coverage Status

Coverage decreased (-0.006%) to 77.362% when pulling 06919f9 on iceoss:rest-client-patch into b2996d3 on jamesagnew:master.

Coverage Status

Coverage decreased (-0.006%) to 77.362% when pulling 06919f9 on iceoss:rest-client-patch into b2996d3 on jamesagnew:master.

@jamesagnew

This comment has been minimized.

Show comment
Hide comment
@jamesagnew

jamesagnew Jun 8, 2017

Owner

Thanks for the contribution!! Merging it now.

Owner

jamesagnew commented Jun 8, 2017

Thanks for the contribution!! Merging it now.

@jamesagnew jamesagnew merged commit 7d5fbe7 into jamesagnew:master Jun 8, 2017

1 of 2 checks passed

coverage/coveralls Coverage decreased (-0.006%) to 77.362%
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

jamesagnew added a commit that referenced this pull request Jun 8, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment