Issues

Select view

Select search mode

 

Searching for alarms in the v2 API with a reductionKey that includes a comma or semicolon results in a 500 error

Fixed

Description

While attempting to move an integration that uses the v1 alarms REST API to v2 due to poor performance in v1 (), we found that attempting to filtering on reductionKey fails if the query includes a percent-encoded comma or semicolon, despite the fact that search.decode.values is set to true in the CXF-RS config.]

Stack trace attached.

Acceptance / Success Criteria

None

Attachments

1

Lucidchart Diagrams

Details

Assignee

Reporter

HB Grooming Date

HB Backlog Status

Components

Sprint

Affects versions

Priority

PagerDuty

Created June 25, 2020 at 9:08 PM
Updated July 22, 2020 at 7:52 PM
Resolved July 22, 2020 at 7:52 PM

Activity

Show:

Benjamin ReedJuly 22, 2020 at 7:52 PM

Merged to foundation-2019

David SchlenkJuly 13, 2020 at 9:28 PM

@Sandy Skipper I tried that, but the same query does not work on that branch. It might have something to do with wildcards. I talked wtih Ben and Jesse in chat and they said foundation-2019 is enough as it's just doc and test.

Sandy SkipperJuly 7, 2020 at 3:28 PM

please retarget your PR to foundation-2017.

David SchlenkJune 26, 2020 at 1:06 AM

Okayyyy well I was writing the test and noticed that in another one there's mention of double escaping and then I tried using %252C instead of %2C and wouldn't you know it, it works! Anyway I wrote tests for it and then I added some very minimal documentation for the v2 API including a note about this in PR 3050.

David SchlenkJune 25, 2020 at 9:10 PM

I'm working on writing a (failing) test to illustrate the problem and will update this with a link to the branch when ready.