Skip to content

Commit 2d1f1fe

Browse files
authored
Merge pull request #10 from jkyberneees/improve-compatibility
Improve Node.js HTTP Server compatibility
2 parents ee83e18 + c37d238 commit 2d1f1fe

File tree

5 files changed

+8
-5
lines changed

5 files changed

+8
-5
lines changed

lib/next.js

+2
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,8 @@ function next (middlewares, req, res, index, routerPatterns = {}, defaultRoute,
2222
const pattern = routerPatterns[middleware.id]
2323
if (pattern) {
2424
req.preRouterUrl = req.url
25+
req.preRouterPath = req.path
26+
2527
req.url = req.url.replace(pattern, '')
2628
}
2729
middleware.lookup(req, res, step)

lib/router/sequential.js

+3
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,10 @@ module.exports = (config = {}) => {
5959
// router is being used as a nested router
6060
middlewares.push((req, res, next) => {
6161
req.url = req.preRouterUrl
62+
req.path = req.preRouterPath
63+
6264
delete req.preRouterUrl
65+
delete req.preRouterPath
6366

6467
step()
6568
})

lib/server/low.js

+1-3
Original file line numberDiff line numberDiff line change
@@ -71,16 +71,14 @@ module.exports = (config = {}) => {
7171
class HttpRequest {
7272
constructor (uRequest) {
7373
this.req = uRequest
74-
this.url = uRequest.getUrl()
74+
this.url = uRequest.getUrl() + (uRequest.getQuery() ? '?' + uRequest.getQuery() : '')
7575
this.method = uRequest.getMethod().toUpperCase()
7676
this.body = null
7777
this.headers = {}
7878

7979
uRequest.forEach((k, v) => {
8080
this.headers[k] = v
8181
})
82-
83-
this.query = uRequest.getQuery()
8482
}
8583

8684
getRaw () {

package-lock.json

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "0http",
3-
"version": "2.2.1",
3+
"version": "2.2.2",
44
"description": "Cero friction HTTP request router. The need for speed!",
55
"main": "index.js",
66
"scripts": {

0 commit comments

Comments
 (0)