Skip to content
This repository has been archived by the owner on Apr 5, 2022. It is now read-only.

Null pointer in links #29

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Null pointer in links #29

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
4f70571
fix for NullPointerException if links parameter is <null>
darklynx Apr 29, 2013
b6cbcd6
Improved logging for RuntimeExceptions that may happen during executi…
darklynx Apr 29, 2013
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
5 changes: 4 additions & 1 deletion src/main/java/org/springframework/data/rest/shell/commands/HttpCommands.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -377,6 +377,9 @@ public String execute(final HttpMethod method,
}
// Calling this method recursively results in hang, so just retry once.
response = restTemplate.execute(requestUri, method, helper, helper);
} catch(RuntimeException re) {
LOG.error(re.getMessage(), re);
throw re;
} finally {
restTemplate.setErrorHandler(origErrHandler);
}
Expand Down Expand Up @@ -615,7 +618,7 @@ private RequestHelper(Object body, MediaType contentType) {

contextCmds.variables.put("responseBody", lastResult);

if(lastResult instanceof Map && ((Map)lastResult).containsKey("links")) {
if(lastResult instanceof Map && ((Map)lastResult).containsKey("links") && ((Map)lastResult).get("links") != null) {
Links linksobj;
if(contextCmds.variables.containsKey("links")) {
linksobj = (Links)contextCmds.variables.get("links");
Expand Down