fix: router issue with query params (#77) (#78)

Co-authored-by: Pooya Parsa <pyapar@gmail.com>

src/router.ts

@@ -42,7 +42,16 @@ export function createRouter (): Router {
   // Main handle
   router.handler = defineEventHandler((event) => {
     // Match route
-    const matched = _router.lookup(event.req.url || '/')
+
+    // Remove query parameters for matching
+    let path = event.req.url || '/'
+    const queryUrlIndex = path.lastIndexOf('?')
+    if (queryUrlIndex > -1) {
+      path = path.substring(0, queryUrlIndex)
+    }
+
+    const matched = _router.lookup(path)
+
     if (!matched) {
       throw createError({
         statusCode: 404,

test/router.test.ts

@@ -11,6 +11,7 @@ describe('router', () => {
     app = createApp({ debug: false })
     router = createRouter()
       .add('/', () => 'Hello')
+      .add('/test/?/a', () => '/test/?/a')
       .get('/test', () => 'Test (GET)')
       .post('/test', () => 'Test (POST)')
 
@@ -29,6 +30,15 @@ describe('router', () => {
     const res2 = await request.post('/test')
     expect(res2.text).toEqual('Test (POST)')
   })
+  it('Handle url with query parameters', async () => {
+    const res = await request.get('/test?title=test')
+    expect(res.status).toEqual(200)
+  })
+
+  it('Handle url with query parameters, include "?" in url path', async () => {
+    const res = await request.get('/test/?/a?title=test')
+    expect(res.status).toEqual(200)
+  })
 
   it('Not matching route', async () => {
     const res = await request.get('/404')