luciano/reitit
1
0
Fork 0
mirror of https://github.com/metosin/reitit.git synced 2025-12-18 08:51:12 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #443 from skibe/resource-handler-path-size-fix

Browse source 
Fix resource handler path matching
This commit is contained in:
Tommi Reiman 2020-10-16 11:24:48 +03:00 committed by GitHub
commit 21df83c804
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
modules/reitit-ring/src/reitit/ring.cljc
View file

@ -221,8 +221,8 @@
handler (if path
(fn [request]
(let [uri (:uri request)]
(if-let [path (if (>= (count uri) path-size) (subs uri path-size))]
(path-or-index-response path uri))))
(if (.startsWith uri path)
(path-or-index-response (subs uri path-size) uri))))
(fn [request]
(let [uri (:uri request)
path (-> request :path-params parameter)]

2
test/clj/reitit/http_test.clj
View file

@ -491,6 +491,8 @@
(testing "not found"
(let [response (app (request "/not-found"))]
(is (= 404 (:status response))))
(let [response (app {:uri "/XXXXX/hello.json" :request-method :get})]
(is (= 404 (:status response)))))
(testing "3-arity"