fix(logger): removing spaces from logger (#3334)

honojs · Aug 31, 2024 · 6c19429 · 6c19429
1 parent 72f4c9d
commit 6c19429
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 12 deletions.
diff --git a/src/middleware/logger/index.test.ts b/src/middleware/logger/index.test.ts
@@ -34,31 +34,31 @@ describe('Logger by Middleware', () => {
     const res = await app.request('http://localhost/empty')
     expect(res).not.toBeNull()
     expect(res.status).toBe(200)
-    expect(log.startsWith('  --> GET /empty \x1b[32m200\x1b[0m')).toBe(true)
+    expect(log.startsWith('--> GET /empty \x1b[32m200\x1b[0m')).toBe(true)
     expect(log).toMatch(/m?s$/)
   })
 
   it('Log status 200 with small body', async () => {
     const res = await app.request('http://localhost/short')
     expect(res).not.toBeNull()
     expect(res.status).toBe(200)
-    expect(log.startsWith('  --> GET /short \x1b[32m200\x1b[0m')).toBe(true)
+    expect(log.startsWith('--> GET /short \x1b[32m200\x1b[0m')).toBe(true)
     expect(log).toMatch(/m?s$/)
   })
 
   it('Log status 200 with big body', async () => {
     const res = await app.request('http://localhost/long')
     expect(res).not.toBeNull()
     expect(res.status).toBe(200)
-    expect(log.startsWith('  --> GET /long \x1b[32m200\x1b[0m')).toBe(true)
+    expect(log.startsWith('--> GET /long \x1b[32m200\x1b[0m')).toBe(true)
     expect(log).toMatch(/m?s$/)
   })
 
   it('Time in seconds', async () => {
     const res = await app.request('http://localhost/seconds')
     expect(res).not.toBeNull()
     expect(res.status).toBe(200)
-    expect(log.startsWith('  --> GET /seconds \x1b[32m200\x1b[0m')).toBe(true)
+    expect(log.startsWith('--> GET /seconds \x1b[32m200\x1b[0m')).toBe(true)
     expect(log).toMatch(/1s/)
   })
 
@@ -70,7 +70,7 @@ describe('Logger by Middleware', () => {
     const res = await app.request('http://localhost/notfound')
     expect(res).not.toBeNull()
     expect(res.status).toBe(404)
-    expect(log.startsWith('  --> GET /notfound \x1b[33m404\x1b[0m')).toBe(true)
+    expect(log.startsWith('--> GET /notfound \x1b[33m404\x1b[0m')).toBe(true)
     expect(log).toMatch(/m?s$/)
   })
 })
@@ -111,31 +111,31 @@ describe('Logger by Middleware in NO_COLOR', () => {
     const res = await app.request('http://localhost/empty')
     expect(res).not.toBeNull()
     expect(res.status).toBe(200)
-    expect(log.startsWith('  --> GET /empty 200')).toBe(true)
+    expect(log.startsWith('--> GET /empty 200')).toBe(true)
     expect(log).toMatch(/m?s$/)
   })
 
   it('Log status 200 with small body', async () => {
     const res = await app.request('http://localhost/short')
     expect(res).not.toBeNull()
     expect(res.status).toBe(200)
-    expect(log.startsWith('  --> GET /short 200')).toBe(true)
+    expect(log.startsWith('--> GET /short 200')).toBe(true)
     expect(log).toMatch(/m?s$/)
   })
 
   it('Log status 200 with big body', async () => {
     const res = await app.request('http://localhost/long')
     expect(res).not.toBeNull()
     expect(res.status).toBe(200)
-    expect(log.startsWith('  --> GET /long 200')).toBe(true)
+    expect(log.startsWith('--> GET /long 200')).toBe(true)
     expect(log).toMatch(/m?s$/)
   })
 
   it('Time in seconds', async () => {
     const res = await app.request('http://localhost/seconds')
     expect(res).not.toBeNull()
     expect(res.status).toBe(200)
-    expect(log.startsWith('  --> GET /seconds 200')).toBe(true)
+    expect(log.startsWith('--> GET /seconds 200')).toBe(true)
     expect(log).toMatch(/1s/)
   })
 
@@ -147,7 +147,7 @@ describe('Logger by Middleware in NO_COLOR', () => {
     const res = await app.request('http://localhost/notfound')
     expect(res).not.toBeNull()
     expect(res.status).toBe(404)
-    expect(log.startsWith('  --> GET /notfound 404')).toBe(true)
+    expect(log.startsWith('--> GET /notfound 404')).toBe(true)
     expect(log).toMatch(/m?s$/)
   })
 })
diff --git a/src/middleware/logger/index.ts b/src/middleware/logger/index.ts
@@ -55,8 +55,8 @@ function log(
 ) {
   const out =
     prefix === LogPrefix.Incoming
-      ? `  ${prefix} ${method} ${path}`
-      : `  ${prefix} ${method} ${path} ${colorStatus(status)} ${elapsed}`
+      ? `${prefix} ${method} ${path}`
+      : `${prefix} ${method} ${path} ${colorStatus(status)} ${elapsed}`
   fn(out)
 }