Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

Fix ReST "count" requests, refactor PUT/POST to use Post/Redirect/Get pattern

Description

Reported in support ticket https://mynms.opennms.com/Ticket/Display.html?id=1557

The following message in output.log points straight to the problem:

==> output.log <==Aug 15, 2012 12:10:18 PM
com.sun.jersey.spi.container.ContainerResponse writeSEVERE: A message body
writer for Java class java.lang.Integer, and Java type int, and MIME media type
text/plain was not found

Changing the provider methods in RequisitionRestService to return a String rather than an int, and wrapping the return value in Integer.toString(int), resolves the problem. I'll commit the fix momentarily.

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Benjamin Reed August 27, 2012 at 10:17 AM

FYI, along with these changes, I've refactored the ReST interfaces to use the Post/Redirect/Get pattern (see http://en.wikipedia.org/wiki/Post/Redirect/Get for details).

Jeff Gehlbach August 16, 2012 at 6:03 PM

Fix pushed to 1.10.

Jeff Gehlbach August 16, 2012 at 5:50 PM

In a quick check before committing, I found that the corresponding foreignSources/count and foreignSources/deployed/count endpoints suffer from the same bug. Fixing that as well.

Fixed

Details

Assignee

Reporter

Labels

Components

Fix versions

Affects versions

Priority

PagerDuty

Created August 16, 2012 at 5:47 PM
Updated January 27, 2017 at 4:19 PM
Resolved August 16, 2012 at 6:03 PM