5htp-core 0.5.0 → 0.5.1

package/src/server/services/router/index.ts

@@ -414,22 +414,6 @@ declare type Routes = {
         // Hook
         await this.runHook('request', request);
 
-        // TODO: move to tracking
-        /*const now = new Date;
-
-        // Identify Guest & create log entry
-        const username = request.user?.name;
-        let clientId: string = request.cookies.clientId;
-        const newClient = !(typeof clientId === 'string' && clientId.length <= 39)
-        if (newClient) {
-            clientId = uuid();
-            res.cookie('clientId', clientId, { expires: new Date(253402300000000) }); // Never expires
-        }
-
-        const keepLogs = request.ip !== '86.76.176.80';
-        if (!keepLogs)  
-            requestId = 'admin';*/
-
         // Create request context so we can access request context across all the request-triggered libs
         context.run({ channelType: 'request', channelId: requestId }, async () => {
 
@@ -455,33 +439,6 @@ declare type Routes = {
             // Data
             res.send(response.data);
 
-            // TODO: move to tracking
-            /*if (newClient)
-                console.client({
-                    id: clientId,
-                    ip: request.ip,
-                    user: username,
-                    device: request.deviceString(),
-                    meet: now,
-                    activity: now,
-                });
-
-            console.request({
-
-                id: requestId,
-                date: now,
-
-                method: request.method,
-                url: request.path,
-                data: request.data,
-
-                ip: request.ip,
-                user: request.user?.name,
-                clientId,
-
-                statusCode: response.statusCode,
-                time: Date.now() - now.valueOf()
-            });*/
         });
     }
 
@@ -502,12 +459,15 @@ declare type Routes = {
 
     public async resolve(request: ServerRequest<this>): Promise<ServerResponse<this>> {
 
-        console.info(LogPrefix, request.ip, request.method, request.domain, request.path);
+        const logId = LogPrefix + ' ' + (request.isVirtual ? ' ---- ' : '') + request.ip + ' ' + request.method + ' ' + request.domain + ' ' + request.path;
+        console.info(logId);
+        const timeStart = Date.now();
 
         if (this.status === 'starting') {
             console.log(LogPrefix, `Waiting for servert to be resdy before resolving request`);
             await this.started;
         }
+
         try {
 
             const response = new ServerResponse<this>(request);
@@ -530,12 +490,14 @@ declare type Routes = {
 
                 // Run on resolution hooks. Ex: authentication check
                 await this.runHook('resolved', route);
+                const timeEndResolving = Date.now();
 
                 // Create response
                 await response.runController(route);
-                if (response.wasProvided)
-                    // On continue l'itération des routes, sauf si des données ont été fournie dans la réponse (.json(), .html(), ...)
+                if (response.wasProvided) {
+                    this.printTakenTime(logId, timeStart, timeEndResolving);
                     return response;
+                }
             }
 
             throw new NotFound();
@@ -551,10 +513,20 @@ declare type Routes = {
                     error.details.origin = errOrigin;
             }
 
+            this.printTakenTime(logId, timeStart);
             throw error;
         }
     }
 
+    private printTakenTime = (logId: string, timeStart: number, timeEndResolving?: number) => {
+
+        if (this.app.env.name === 'server') return;
+
+        console.log(logId + ' ' + Math.round(Date.now() - timeStart) + 'ms' + 
+            (timeEndResolving === undefined ? '' : ' | Routing: ' + Math.round(timeEndResolving - timeStart))
+        );
+    }
+
     private async resolveApiBatch( fetchers: TFetcherList, request: ServerRequest<this> ) {
 
         // TODO: use api.fetchSync instead
@@ -601,9 +573,14 @@ declare type Routes = {
             if (this.app.env.profile === 'prod')
                 e.message = "We encountered an internal error, and our team has just been notified. Sorry for the inconvenience.";
 
-        // Pour déboguer les erreurs HTTP
-        } else if (code !== 404 && this.app.env.profile === "dev")
-            console.warn(e);
+        } else {
+
+            // For debugging HTTP errors
+            if (this.app.env.profile === "dev")
+                console.warn(e);
+
+            await this.app.runHook('error.' + code, e, request);
+        }
 
         // Return error based on the request format
         if (request.accepts("html")) {

package/src/server/services/router/request/index.ts

@@ -135,6 +135,17 @@ export default class ServerRequest<
         return locale ? locale.toUpperCase() : 'EN'
     }
 
+    public cookie( key: string, consume: boolean = false ) {
+
+        const value = this.req.cookies[ key ];
+
+        if (consume)
+            this.res.clearCookie(key);
+
+        return value;
+
+    }
+
     /*----------------------------------
    - TESTS
    ----------------------------------*/

package/src/server/services/router/response/index.ts

@@ -81,6 +81,8 @@ export default class ServerResponse<
     public statusCode: number = 200;
     public headers: {[cle: string]: string} = {}
     public cookie: express.Response["cookie"];
+    public clearCookie: express.Response["clearCookie"];
+    public canonicalUrl: URL;
 
     // If data was provided by at lead one controller
     public wasProvided = false;
@@ -90,15 +92,22 @@ export default class ServerResponse<
         super(request);
 
         this.cookie = this.request.res.cookie.bind(this.request.res);
+        this.clearCookie = this.request.res.clearCookie.bind(this.request.res);
 
         this.router = request.router;
         this.app = this.router.app;
+
+        this.canonicalUrl = new URL(request.url);
+        this.canonicalUrl.search = '';
     }
 
     public async runController( route: TAnyRoute, additionnalData: {} = {} ) {
 
         this.route = route;
 
+        // Update canonical url
+        this.updateCanonicalUrl(route);
+
         // Create response context for controllers
         const context = await this.createContext(route);
 
@@ -138,6 +147,18 @@ export default class ServerResponse<
             this.router.cache[ chunkId ] = this.data;
     }
 
+    private updateCanonicalUrl( route: TAnyRoute ) {
+
+        if (!route.options.canonicalParams)
+            return;
+
+        for (const key of route.options.canonicalParams) {
+            const paramValue = this.request.data[ key ];
+            if (paramValue !== undefined)
+                this.canonicalUrl.searchParams.set(key, paramValue);
+        }
+    }
+
     /*----------------------------------
     - INTERNAL
     ----------------------------------*/

package/src/server/services/router/response/page/document.tsx

@@ -57,9 +57,6 @@ export default class DocumentRenderer<TRouter extends Router> {
 
     public async page( html: string, page: Page, response: ServerResponse<TRouter> ) {
 
-        // TODO: can be customized via page / route config
-        const canonicalUrl = response.request.req.url;
-
         let attrsBody = {
             className: [...page.bodyClass].join(' '),
         };
@@ -94,7 +91,7 @@ export default class DocumentRenderer<TRouter extends Router> {
                     {/* Page */}
                     <title>{page.title}</title>
                     <meta content={page.description} name="description" />
-                    <link rel="canonical" href={canonicalUrl} />
+                    <link rel="canonical" href={response.canonicalUrl} />
 
                     {/* SEO, social medias, OG tags, ...  */}
                     {page.head.map(({ $, ...attrs }) => (

package/src/server/services/router/response/page/index.tsx

@@ -120,8 +120,12 @@ export default class ServerPage<TRouter extends Router = Router> extends PageRes
 
     private buildMetas() {
 
+        const shouldIndex = this.context.response.statusCode < 300;
+
         const metas = {
 
+            robots: shouldIndex ? 'index' : 'noindex',
+
             'og:type': 'website',
             'og:locale': this.app.identity.locale,
             'og:site_name': this.app.identity.web.title,