Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add a full-route key to the request map #210

Closed
wants to merge 1 commit into from
Closed

Add a full-route key to the request map #210

wants to merge 1 commit into from
+49 −12

Commits on May 11, 2022

  1. Add a full-route key to the request map 

    This provides access to the full route that contains the common prefix.
The full-route info is added to the :compojure/full-route key in the
request map.

We use :compojure/route to get the route info and add it to the
attributes of our metrics and traces, but when compojure.core/context
is used, we are not able to get the parameters that are not
instantiated. This change adds :compojure/full-route key that serves
similarly as the existing :compojure/route does - the only difference
is that the new key has a common prefix.

Co-authored-by: Liam Chen <[email protected]>
Co-authored-by: Claire Alvis <[email protected]>
    @liamchzh @calvis
    liamchzh and calvis committed May 11, 2022
    Configuration menu
    Copy the full SHA
    5abd2eb View commit details
    Browse the repository at this point in the history