CODEWITHSUNDEEP
clojuregetquery-stringcompojure
Mantas Vidutis
Mantas Vidutis

Reputation: 16794

Compojure Routes losing params info

My code:

(defn json-response [data & [status]]
    {:status (or status 200)
     :headers {"Content-Type" "application/json"}
     :body (json/generate-string data)})

(defroutes checkin-app-handler
  (GET "/:code" [code & more] (json-response {"code" code "params" more})))

When I load the file to the repl and run this command, the params seems to be blank:

$ (checkin-app-handler {:server-port 8080 :server-name "127.0.0.1" :remote-addr "127.0.0.1" :uri "/123" :query-string "foo=1&bar=2" :scheme :http :headers {} :request-method :get})
> {:status 200, :headers {"Content-Type" "application/json"}, :body "{\"code\":\"123\",\"params\":{}}"}

What am I doing wrong? I need to get at the query-string, but the params map is always empty..

Upvotes: 4

Views: 445

Answers (1)

Bendlas
Bendlas

Reputation: 796

In order to get the query string parsed into the params map, you need to use the params middleware:

(ns n
  (:require [ring.middleware.params :as rmp]))

(defroutes checkin-app-routes
  (GET "" [] ...))

(def checkin-app-handler
  (-> #'checkin-app-routes
      rmp/wrap-params
      ; .. other middlewares
      ))

Note, that the usage of the var (#'checkin-app-routes) is not strictly necessary, but it makes the routes closure, wrapped inside the middlewares, pick up the changes, when you redefine the routes.

IOW you can also write 

(def checkin-app-handler
  (-> checkin-app-routes
      rmp/wrap-params
      ; .. other middlewares
      ))

but then, you need to redefine the handler too, when interactively redefining the routes.

Upvotes: 5

Related Questions