3.0.0-2

.editorconfig

@@ -0,0 +1,9 @@
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = false
+
+[src/**.ts]
+charset = utf-8
+indent_style = tab

.github/workflows/publish-pre.yml

@@ -1,4 +1,4 @@
-name: Publish (main)
+name: Publish (pre)
 
 on:
   release:
@@ -20,11 +20,11 @@ jobs:
       - name: Publish to npmjs
         env:
           NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
-        run: npm publish --tag latest --access public
+        run: npm publish --tag pre --access public
       - uses: actions/setup-node@v1
         with:
           registry-url: https://npm.pkg.github.com/
       - name: Publish to GPR
         env:
           NODE_AUTH_TOKEN: ${{secrets.GITHUB_TOKEN}}
-        run: npm publish --tag latest --access public
+        run: npm publish --tag pre --access public

README.md

@@ -1,6 +1,6 @@
 # Cloudflare Workers Router
 
-Cloudflare Workers Router is a super lightweight router (1.3K gzipped) with middleware support and **ZERO dependencies** for [Cloudflare Workers](https://workers.cloudflare.com/).
+Cloudflare Workers Router is a super lightweight router (1.0K gzipped) with middleware support and **ZERO dependencies** for [Cloudflare Workers](https://workers.cloudflare.com/).
 
 When I was trying out Cloudflare Workers I almost immediately noticed how fast it was compared to other serverless offerings. So I wanted to build a full-fledged API to see how it performs doing real work, but since I wasn't able to find a router that suited my needs I created my own.
 
@@ -18,7 +18,7 @@ I worked a lot with [Express.js](https://expressjs.com/) in the past and really
 ## Features
 
 - ZERO dependencies
-- Lightweight (1.3K gzipped)
+- Lightweight (1.0K gzipped)
 - Fully written in TypeScript
 - Integrated Debug-Mode & CORS helper
 - Built specifically around Middlewares
@@ -40,8 +40,9 @@ export interface Env {
     //
     // Example binding to R2. Learn more at https://developers.cloudflare.com/workers/runtime-apis/r2/
     // MY_BUCKET: R2Bucket
-}
 
+    SECRET_TOKEN: string
+}
 
 // Initialize router
 const router = new Router<Env>()
@@ -50,61 +51,61 @@ const router = new Router<Env>()
 router.cors()
 
 // Register global middleware
-router.use(({ req, res, next }) => {
-    res.headers.set('X-Global-Middlewares', 'true')
-    next()
+router.use(() => {
+    return new Response(null, {
+        headers: {
+            'X-Global-Middlewares': 'true'
+        }
+    })
 })
 
 // Simple get
-router.get('/user', ({ req, res }) => {
-    res.body = {
-        data: {
+router.get('/user', () => {
+    return Response.json({
         id: 1,
         name: 'John Doe'
-        }
-    }
+    })
 })
 
 // Post route with url parameter
-router.post('/user/:id', ({ req, res }) => {
+router.post('/user/:id', ({ req }) => {
 
     const userId = req.params.id
 
-    // Do stuff...
+    // Do stuff
 
-    if (errorDoingStuff) {
-        res.status = 400
-        res.body = {
-            error: 'User did stupid stuff!'
-        }
-        return
+    if (!true) {
+        return Response.json({
+            error: 'Error doing stuff!'
+        }, { status: 400 })
     }
 
-    res.status = 204
+    return Response.json({ userId }, { status: 204 })
 })
 
 // Delete route using a middleware
-router.delete('/user/:id', ({ req, res, next }) => {
+router.delete('/user/:id', ({ env, req }) => {
+    const { SECRET_TOKEN } = env
 
-    if (!apiTokenIsCorrect) {
-        res.status = 401
-        return
-    }
+    if (req.headers.get('Authorization') === SECRET_TOKEN)
+        return new Response(null, { status: 401 })
 
-    await next()
-}, (req, res) => {
+}, ({ req }) => {
 
   const userId = req.params.id
 
   // Do stuff...
+
+  return Response.json({ userId })
 })
 
 // Listen Cloudflare Workers Fetch Event
 export default {
-    async fetch(request: Request, env: Env): Promise<Response> {
-        return router.handle(env, request)
+    async fetch(request: Request, env: Env, ctx: ExecutionContext) {
+        return router.handle(request, env, ctx)
     }
 }
+
 ```
 
 <details>
@@ -120,59 +121,56 @@ const router = new Router()
 router.cors()
 
 // Register global middleware
-router.use(({ req, res, next }) => {
-    res.headers.set('X-Global-Middlewares', 'true')
-    next()
+router.use(() => {
+    return new Response(null, {
+        headers: {
+            'X-Global-Middlewares': 'true'
+        }
+    })
 })
 
 // Simple get
-router.get('/user', ({ req, res }) => {
-    res.body = {
-        data: {
+router.get('/user', () => {
+    return Response.json({
         id: 1,
         name: 'John Doe'
-        }
-    }
+    })
 })
 
 // Post route with url parameter
-router.post('/user/:id', ({ req, res }) => {
-
+router.post('/user/:id', ({ req }) => {
     const userId = req.params.id
 
-    // Do stuff...
+    // Do stuff
 
     if (errorDoingStuff) {
-            res.status = 400
-            res.body = {
-                error: 'User did stupid stuff!'
-            }
-            return
+        return Response.json({
+            error: 'Error doing stuff!'
+        }, { status: 400 })
     }
 
-    res.status = 204
+    return Response.json({ userId }, { status: 204 })
 })
 
 // Delete route using a middleware
-router.delete('/user/:id', ({ req, res, next }) => {
+router.delete('/user/:id', ({ env, req }) => {
+    const { SECRET_TOKEN } = env
 
-    if (!apiTokenIsCorrect) {
-        res.status = 401
-        return
-    }
-
-    await next()
-}, (req, res) => {
+    if (req.headers.get('Authorization') === SECRET_TOKEN)
+        return new Response(null, { status: 401 })
 
+}, ({ req }) => {
     const userId = req.params.id
 
     // Do stuff...
+
+    return Response.json({ userId })
 })
 
 // Listen Cloudflare Workers Fetch Event
 export default {
-    async fetch(request, env) {
-        return router.handle(env, request)
+    async fetch(request, env, ctx) {
+        return router.handle(request, env, ctx)
     }
 }
 ```
@@ -206,7 +204,7 @@ Handler is a `function` which will be called for every request.
 
 #### `ctx`
 
-Object containing `env`, [`req`](#req-object), [`res`](#res-object), `next`
+Object containing `env`, [`req`](#req-object)
 
 
 ### `router.cors([config])`
@@ -247,16 +245,16 @@ Supports the use of dynamic parameters, prefixed with a `:` (i.e. `/user/:userId
 
 #### `handlers` (function, optional)
 
-An unlimited number of functions getting [`req`](#req-object) and [`res`](#res-object) passed into them.
+An unlimited number of functions getting [`ctx`](#ctx-object) passed into them.
 
 
 ### `ctx`-Object
+
 Key       | Type                | Description
 --------- | ------------------- | -----------
 `env`     | `object`            | Environment
 `req`     | `req`-Object        | Request Object
-`res`     | `res`-Object        | Response Object
-`next`    | `next`-Handler      | Next Handler
+`ctx`     | `ctx`-Object        | Cloudflare's `ctx`-Object
 
 
 ### `req`-Object
@@ -270,16 +268,6 @@ Key       | Type                | Description
 `query`   | `object`            | Object containing all query parameters
 
 
-### `res`-Object
-
-Key         | Type                | Description
------------ | ------------------- | -----------
-`body`      | `object` / `string` | Either set an `object` (will be converted to JSON) or a string
-`headers`   | `Headers`           | Response [Headers Object](https://developer.mozilla.org/en-US/docs/Web/API/Headers)
-`status`    | `integer`           | Return status code (default: `204`)
-`webSocket` | `WebSocket`         | Upgraded websocket connection
-
-
 ## Setup
 
 Please follow Cloudflare's [Get started guide](https://developers.cloudflare.com/workers/get-started/guide/) to install wrangler.
@@ -335,8 +323,8 @@ const router = new Router<Env>()
 // TODO: add your routes here
 
 export default {
-    async fetch(request: Request, env: Env, ctx: ExecutionContext): Promise<Response> {
-        return router.handle(env, request)
+    async fetch(request: Request, env: Env, ctx: ExecutionContext) {
+        return router.handle(request, env, ctx)
     }
 }
 ```
@@ -360,7 +348,7 @@ const router = new Router()
 
 export default {
     async fetch(request, env, ctx) {
-        return router.handle(env, request)
+        return router.handle(request, env, ctx)
     }
 }
 ```

package-lock.json

@@ -1,12 +1,12 @@
 {
   "name": "@tsndr/cloudflare-worker-router",
-  "version": "2.3.0",
+  "version": "3.0.0-2",
   "lockfileVersion": 2,
   "requires": true,
   "packages": {
     "": {
       "name": "@tsndr/cloudflare-worker-router",
-      "version": "2.3.0",
+      "version": "3.0.0-2",
       "license": "MIT",
       "devDependencies": {
         "@cloudflare/workers-types": "^3.13.0",

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tsndr/cloudflare-worker-router",
-  "version": "2.3.0",
+  "version": "3.0.0-2",
   "description": "",
   "main": "index.js",
   "types": "index.d.ts",

src/index.ts

@@ -1,45 +1,43 @@
 /**
- * Route Object
- *
- * @typedef Route
- * @property {string} method HTTP request method
- * @property {string} url URL String
- * @property {RouterHandler[]} handlers Array of handler functions
- */
- export interface Route<TEnv> {
+* Route Object
+*
+* @typedef Route
+* @property {string} method HTTP request method
+* @property {string} url URL String
+* @property {RouterHandler[]} handlers Array of handler functions
+*/
+export interface Route<TEnv> {
 	method: string
 	url: string
 	handlers: RouterHandler<TEnv>[]
 }
 
 /**
- * Router Context
- *
- * @typedef RouterContext
- * @property {RouterEnv} env Environment
- * @property {RouterRequest} req Request Object
- * @property {RouterResponse} res Response Object
- * @property {RouterNext} next Next Handler
- */
-export interface RouterContext<TEnv> {
+* Router Context
+*
+* @typedef RouterContext
+* @property {RouterEnv} env Environment
+* @property {RouterRequest} req Request Object
+* @property {ExecutionContext} ctx Context Object
+*/
+export interface RouterContext<TEnv = any> {
 	env: TEnv
 	req: RouterRequest
-    res: RouterResponse
-    next: RouterNext
+	ctx?: ExecutionContext
 }
 
 /**
- * Request Object
- *
- * @typedef RouterRequest
- * @property {string} url URL
- * @property {string} method HTTP request method
- * @property {RouterRequestParams} params Object containing all parameters defined in the url string
- * @property {RouterRequestQuery} query Object containing all query parameters
- * @property {Headers} headers Request headers object
- * @property {string | any} body Only available if method is `POST`, `PUT`, `PATCH` or `DELETE`. Contains either the received body string or a parsed object if valid JSON was sent.
- * @property {IncomingRequestCfProperties} [cf] object containing custom Cloudflare properties. (https://developers.cloudflare.com/workers/examples/accessing-the-cloudflare-object)
- */
+* Request Object
+*
+* @typedef RouterRequest
+* @property {string} url URL
+* @property {string} method HTTP request method
+* @property {RouterRequestParams} params Object containing all parameters defined in the url string
+* @property {RouterRequestQuery} query Object containing all query parameters
+* @property {Headers} headers Request headers object
+* @property {string | any} body Only available if method is `POST`, `PUT`, `PATCH` or `DELETE`. Contains either the received body string or a parsed object if valid JSON was sent.
+* @property {IncomingRequestCfProperties} [cf] object containing custom Cloudflare properties. (https://developers.cloudflare.com/workers/examples/accessing-the-cloudflare-object)
+*/
 export interface RouterRequest {
 	url: string
 	method: string
@@ -52,71 +50,44 @@ export interface RouterRequest {
 }
 
 /**
- * Request Parameters
- *
- * @typedef RouterRequestParams
- */
+* Request Parameters
+*
+* @typedef RouterRequestParams
+*/
 export interface RouterRequestParams {
 	[key: string]: string
 }
 
 /**
- * Request Query
- *
- * @typedef RouterRequestQuery
- */
+* Request Query
+*
+* @typedef RouterRequestQuery
+*/
 export interface RouterRequestQuery {
 	[key: string]: string
 }
 
 /**
- * Response Object
- *
- * @typedef RouterResponse
- * @property {Headers} headers Response headers object
- * @property {number} [status=204] Return status code (default: `204`)
- * @property {string | any} [body] Either an `object` (will be converted to JSON) or a string
- * @property {Response} [raw] A response object that is to be returned, this will void all other res properties and return this as is.
- */
-export interface RouterResponse {
-    headers: Headers
-    status?: number
-    body?: string | any
-    raw?: Response,
-    webSocket?: WebSocket
-}
-
-/**
- * Next Function
- *
- * @callback RouterNext
- * @returns {Promise<void>}
- */
-export interface RouterNext {
-    (): Promise<void>
-}
-
-/**
- * Handler Function
- *
- * @callback RouterHandler
- * @param {RouterContext} ctx
- * @returns {Promise<void> | void}
- */
+* Handler Function
+*
+* @callback RouterHandler
+* @param {RouterContext} ctx
+* @returns {Promise<Response | void> Response | void}
+*/
 export interface RouterHandler<TEnv = any> {
-    (ctx: RouterContext<TEnv>): Promise<void> | void
+	(ctx: RouterContext<TEnv>): Promise<Response | void> | Response | void
 }
 
 /**
- * CORS Config
- *
- * @typedef RouterCorsConfig
- * @property {string} [allowOrigin="*"] Access-Control-Allow-Origin (default: `*`)
- * @property {string} [allowMethods="*"] Access-Control-Allow-Methods (default: `*`)
- * @property {string} [allowHeaders="*"] Access-Control-Allow-Headers (default: `*`)
- * @property {number} [maxAge=86400] Access-Control-Max-Age (default: `86400`)
- * @property {number} [optionsSuccessStatus=204] Return status code for OPTIONS request (default: `204`)
- */
+* CORS Config
+*
+* @typedef RouterCorsConfig
+* @property {string} [allowOrigin="*"] Access-Control-Allow-Origin (default: `*`)
+* @property {string} [allowMethods="*"] Access-Control-Allow-Methods (default: `*`)
+* @property {string} [allowHeaders="*"] Access-Control-Allow-Headers (default: `*`)
+* @property {number} [maxAge=86400] Access-Control-Max-Age (default: `86400`)
+* @property {number} [optionsSuccessStatus=204] Return status code for OPTIONS request (default: `204`)
+*/
 export interface RouterCorsConfig {
 	allowOrigin?: string
 	allowMethods?: string
@@ -126,11 +97,11 @@ export interface RouterCorsConfig {
 }
 
 /**
- * Router
- *
- * @public
- * @class
- */
+* Router
+*
+* @public
+* @class
+*/
 export class Router<TEnv = any> {
 
 	/**
@@ -167,7 +138,7 @@ export class Router<TEnv = any> {
 
 	/**
 	* CORS enabled
-     *-
+	*
 	* @protected
 	* @type {boolean}
 	*/
@@ -325,6 +296,18 @@ export class Router<TEnv = any> {
 		return this
 	}
 
+	private setCorsHeaders(headers: Headers = new Headers()): Headers {
+		if (this.corsConfig.allowOrigin && !headers.has('Access-Control-Allow-Origin'))
+			headers.set('Access-Control-Allow-Origin', this.corsConfig.allowOrigin)
+		if (this.corsConfig.allowMethods && !headers.has('Access-Control-Allow-Methods'))
+			headers.set('Access-Control-Allow-Methods', this.corsConfig.allowMethods)
+		if (this.corsConfig.allowHeaders && !headers.has('Access-Control-Allow-Headers'))
+			headers.set('Access-Control-Allow-Headers', this.corsConfig.allowHeaders)
+		if (this.corsConfig.maxAge && !headers.has('Access-Control-Max-Age'))
+			headers.set('Access-Control-Max-Age', this.corsConfig.maxAge.toString())
+		return headers
+	}
+
 	/**
 	* Register route
 	*
@@ -393,8 +376,7 @@ export class Router<TEnv = any> {
 	* @param {any} [extend]
 	* @returns {Promise<Response>}
 	*/
-    public async handle(env: TEnv, request: Request, extend: any = {}): Promise<Response> {
-        try {
+	public async handle(request: Request, env: TEnv, ctx?: ExecutionContext, extend: any = {}): Promise<Response> {
 		const req: RouterRequest = {
 			...extend,
 			method: request.method,
@@ -406,76 +388,37 @@ export class Router<TEnv = any> {
 			body: ''
 		}
 
-            const headers = new Headers()
 		const route = this.getRoute(req)
 
-            if (this.corsEnabled) {
-                if (this.corsConfig.allowOrigin)
-                    headers.set('Access-Control-Allow-Origin', this.corsConfig.allowOrigin)
-                if (this.corsConfig.allowMethods)
-                    headers.set('Access-Control-Allow-Methods', this.corsConfig.allowMethods)
-                if (this.corsConfig.allowHeaders)
-                    headers.set('Access-Control-Allow-Headers', this.corsConfig.allowHeaders)
-                if (this.corsConfig.maxAge)
-                    headers.set('Access-Control-Max-Age', this.corsConfig.maxAge.toString())
-
-                if (!route && req.method === 'OPTIONS') {
-                    return new Response(null, {
-                        headers,
-                        status: this.corsConfig.optionsSuccessStatus
-                    })
-                }
-            }
-
 		if (!route)
 			return new Response(this.debugMode ? 'Route not found!' : null, { status: 404 })
 
-            if (['POST', 'PUT', 'PATCH'].includes(req.method)) {
-                if (req.headers.has('Content-Type') && req.headers.get('Content-Type')!.includes('json')) {
-                    try {
-                        req.body = await request.json()
-                    } catch {
-                        req.body = {}
-                    }
-                } else {
-                    try {
-                        req.body = await request.text()
-                    } catch {
-                        req.body = ''
-                    }
-                }
+		if (this.corsEnabled && req.method === 'OPTIONS') {
+			return new Response(null, {
+				headers: this.setCorsHeaders(),
+				status: this.corsConfig.optionsSuccessStatus
+			})
 		}
 
-            const res: RouterResponse = { headers }
 		const handlers = [...this.globalHandlers, ...route.handlers]
-            let prevIndex = -1
 
-            const runner = async (index: number) => {
-                if (index === prevIndex)
-                    throw new Error('next() called multiple times')
+		let response: Response | undefined
 
-                prevIndex = index
+		for (const handler of handlers) {
+			const res = await handler({ env, req, ctx })
 
-                if (typeof handlers[index] === 'function')
-                    await handlers[index]({ env, req, res, next: async () => await runner(index + 1) })
+			if (res) {
+				response = res
+				break
+			}
 		}
 
-            await runner(0)
+		if (!response)
+			return new Response(this.debugMode ? 'Handler did not return a Response!' : null, { status: 404 })
 
-            if (typeof res.body === 'object') {
-                if (!res.headers.has('Content-Type'))
-                    res.headers.set('Content-Type', 'application/json')
+		if (this.corsEnabled)
+			this.setCorsHeaders(response.headers)
 
-                res.body = JSON.stringify(res.body)
-            }
-
-            if (res.raw)
-                return res.raw
-
-            return new Response([101, 204, 205, 304].includes(res.status || (res.body ? 200 : 204)) ? null : res.body, { status: res.status, headers: res.headers, webSocket: res.webSocket || null })
-        } catch(err) {
-            console.error(err)
-            return new Response(this.debugMode && err instanceof Error ? err.stack : '', { status: 500 })
-        }
+		return response
 	}
 }

tsconfig.json

@@ -15,7 +15,6 @@
         "alwaysStrict": true,
         "noUnusedLocals": true,
         "noUnusedParameters": true,
-        "noImplicitReturns": true,
         "noFallthroughCasesInSwitch": true,
         "esModuleInterop": true,
         "preserveConstEnums": true,