feat(connect): middlewareをtraceするサンプルを追加

connect-trace-example.js

Author: azu

ja/connect/README.md

@@ -23,7 +23,7 @@ Echoサーバとは、送られてきたリクエストの内容をそのまま
 ```
 
 `app.use(middleware)` という形で、_middleware_と呼ばれる関数には`request`や`response`といったオブジェクトが渡されます。
-そのため、リクエストみてフィルタリングしたり、任意のレスポンスを返したり出来るようになっています。
+そのため、リクエストをフィルタリングしたり、ログを取ったり、任意のレスポンスを返したり出来るようになっています。
 
 Echoサーバでは `req.pipe(res);` という形でリクエストをそのままレスポンスとして流す事で実現されています。
 
@@ -47,4 +47,5 @@ Echoサーバでは `req.pipe(res);` という形でリクエストをそのま
 基本的にどの_middleware_も`app.use(middleware)`という形で拡張でき、
 モジュールとして実装すれば再利用もしやすい形となっています。
 
-> **Note** _middleware_となる関数の引数が4つであると、それはエラーハンドリングの_middleware_とするという、Connectの独自のルールがあります。
+> **Note** _middleware_となる関数の引数が4つであると、それはエラーハンドリングの_middleware_とするという、Connect独自のルールがあります。
+

src/connect/connect-example.js

@@ -6,6 +6,7 @@ import assert from "assert";
 import connect from "connect";
 import http from "http";
 import fetch from "node-fetch";
+
 const responseText = "response text";
 let app = connect();
 // add Error handling
@@ -14,6 +15,7 @@ app.use(errorHandler());
 app.use(nosniff());
 // respond to all requests
 app.use(hello(responseText));
+
 //create node.js http server and listen on port
 let server = http.createServer(app).listen(3000, request);
 

src/connect/connect-trace-example.js

@@ -16,17 +16,10 @@ app.use(nosniff());
 // respond to all requests
 app.use(hello(responseText));
 
-//create node.js http server and listen on port
-let server = http.createServer(app).listen(3000, request);
-
-function request() {
-    let closeServer = server.close.bind(server);
-    fetch("http://localhost:3000")
-        .then(res => res.text())
-        .then(text => {
-            assert.equal(text, responseText);
-            server.close();
-        })
-        .catch(console.error.bind(console))
-        .then(closeServer, closeServer);
-}
+// print middleware list
+app.stack.map(({handle}) => console.log(handle));
+/* =>
+ [Function: errorHandling]
+ [Function: nosniff]
+ [Function: hello]
+*/

src/connect/errorHandler.js

@@ -1,6 +1,6 @@
 "use strict";
-export default function errorHandler() {
-    return function (err, req, res, next) {
+export default function () {
+    return function errorHandling(err, req, res, next) {
         console.error(err.stack);
         res.status(500).send(err.message);
         next();

src/connect/hello.js

@@ -1,6 +1,6 @@
 "use strict";
 export default function (text) {
-    return function (req, res) {
+    return function hello(req, res) {
         res.end(text);
     };
 }

src/connect/nosniff.js

@@ -8,7 +8,7 @@ function setHeaders(res, headers) {
     });
 }
 export default function () {
-    return function (req, res, next) {
+    return function nosniff(req, res, next) {
         setHeaders(res, {
             "X-Content-Type-Options": "nosniff"
         });