Outbound query parameters in Stripes Connect

Description

At present, while Stripes Connect substitutes URL parameters into the path part of its requests (e.g. /users/:query), it does not substitute them into the query part (e.g. /users/?query=:query)-- and mod-users on the server side only recognises the latter, so we need it in order to do searching.

Environment

None

Potential Workaround

None

Checklist

hide

TestRail: Results

Activity

Show:

Mike TaylorNovember 28, 2016 at 4:42 PM

Mike TaylorNovember 28, 2016 at 4:41 PM

Having discussed with Jason, it is now apparent that which I thought https://folio-org.atlassian.net/browse/STRIPES-91#icft=STRIPES-91 was about (allowing Stripes Connect to see the query parameter parts of the UI URL) is already in place – we think those parameters are available via the props of routed components. Instead it turns out that https://folio-org.atlassian.net/browse/STRIPES-91#icft=STRIPES-91 is about passing query parameters to the Okapi URI. In other words, it's the same issue as this one, so I am closing this as a duplicate.

Mike TaylorNovember 25, 2016 at 1:23 PM
Edited

It's not 100% clear what the syntax should look like for placeholders in query parameters. If we just use the same :name syntax that we use in path components, yielding manifest entries like /users/?query=:query then is that too lax? Might we for some reason want a literal : in the query part of a URL?

Anyway, as a first pass, it probably suffices just to blindly substitute any sequence of the form /:([a-z0-9])+/ with the named parameter from the UI URL.

Mike TaylorNovember 25, 2016 at 1:20 PM

Note that this issue is the mirror image of https://folio-org.atlassian.net/browse/STRIPES-91#icft=STRIPES-91: that one is about extending Stripes Connect so that it can ingest query parameters from the UI URL; this one is about extending it so it can emit them in WSAPI URLs.

Duplicate

Details

Assignee

Reporter

Labels

Priority

TestRail: Cases

Open TestRail: Cases

TestRail: Runs

Open TestRail: Runs

Created November 25, 2016 at 1:19 PM
Updated November 28, 2016 at 4:42 PM
Resolved November 28, 2016 at 4:42 PM
TestRail: Cases
TestRail: Runs

Flag notifications