[PR #440] [MERGED] refactor: Replace consola with pino #28891

Closed
opened 2026-04-17 20:20:08 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/better-auth/better-auth/pull/440
Author: @tklein1801
Created: 11/6/2024
Status: Merged
Merged: 11/18/2024
Merged by: @Bekacru

Base: mainHead: main


📝 Commits (10+)

  • 92bcce7 refactor: Replace consola with pino
  • 35b7cc1 refactor: Implement adapter for logging
  • d10f51f fix: Resolve issues due to previous commit
  • 41750b8 Merge branch 'main' into main
  • 7781c30 chore: Remove pkg-lock and unused dependency
  • 0ef5a26 feat: Provide ability to print success messages
  • 70c8eec fix: Use correct log-level
  • 2da89e7 Merge branch 'main' into main
  • 6e627bd test: Update snapshot due to logger changes
  • 7f511d1 Merge branch 'main' into main

📊 Changes

28 files changed (+204 additions, -124 deletions)

View changed files

📝 packages/better-auth/package.json (+1 -1)
📝 packages/better-auth/src/__snapshots__/init.test.ts.snap (+0 -3)
📝 packages/better-auth/src/api/index.ts (+30 -21)
📝 packages/better-auth/src/api/middlewares/origin-check.ts (+2 -3)
📝 packages/better-auth/src/api/rate-limiter/index.ts (+1 -2)
📝 packages/better-auth/src/api/routes/callback.ts (+5 -5)
📝 packages/better-auth/src/api/routes/session.ts (+13 -3)
📝 packages/better-auth/src/api/routes/sign-in.ts (+2 -2)
📝 packages/better-auth/src/api/routes/sign-up.ts (+2 -3)
📝 packages/better-auth/src/db/get-migration.ts (+2 -1)
📝 packages/better-auth/src/init.ts (+11 -13)
📝 packages/better-auth/src/oauth2/state.ts (+4 -5)
📝 packages/better-auth/src/plugins/admin/admin.test.ts (+1 -1)
📝 packages/better-auth/src/plugins/email-otp/index.ts (+4 -3)
📝 packages/better-auth/src/plugins/generic-oauth/index.ts (+8 -4)
📝 packages/better-auth/src/plugins/jwt/jwt.test.ts (+1 -1)
📝 packages/better-auth/src/plugins/organization/organization.test.ts (+1 -1)
📝 packages/better-auth/src/plugins/organization/routes/crud-invites.ts (+1 -2)
📝 packages/better-auth/src/plugins/passkey/index.ts (+1 -1)
📝 packages/better-auth/src/plugins/phone-number/index.ts (+1 -2)

...and 8 more files

📄 Description

So far, the options for customizing in logging are very limited or only log-level and activation can be set. It should be discussed which aspects of the logger should be configurable by the user

closes #124


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/better-auth/better-auth/pull/440 **Author:** [@tklein1801](https://github.com/tklein1801) **Created:** 11/6/2024 **Status:** ✅ Merged **Merged:** 11/18/2024 **Merged by:** [@Bekacru](https://github.com/Bekacru) **Base:** `main` ← **Head:** `main` --- ### 📝 Commits (10+) - [`92bcce7`](https://github.com/better-auth/better-auth/commit/92bcce756ade276deb46df8795f9f5da81b88de7) refactor: Replace `consola` with `pino` - [`35b7cc1`](https://github.com/better-auth/better-auth/commit/35b7cc138c0e76c90b5c9029f486214706ac215e) refactor: Implement adapter for logging - [`d10f51f`](https://github.com/better-auth/better-auth/commit/d10f51ff5c78e91ddfe91054b0979db9368fcafe) fix: Resolve issues due to previous commit - [`41750b8`](https://github.com/better-auth/better-auth/commit/41750b849218d52dd6648b2adfb67e143125aa13) Merge branch 'main' into main - [`7781c30`](https://github.com/better-auth/better-auth/commit/7781c3023d3042490d52495188e46991a4964b85) chore: Remove pkg-lock and unused dependency - [`0ef5a26`](https://github.com/better-auth/better-auth/commit/0ef5a26ebc86b515f376b05a275c6b57fd7002e7) feat: Provide ability to print success messages - [`70c8eec`](https://github.com/better-auth/better-auth/commit/70c8eecbf64a714db2b15219f11e8c5983afc0e5) fix: Use correct log-level - [`2da89e7`](https://github.com/better-auth/better-auth/commit/2da89e7a7f3b061e1588bab1df25b1f7a96b36b8) Merge branch 'main' into main - [`6e627bd`](https://github.com/better-auth/better-auth/commit/6e627bd3f307b59bc9010750d5a690354b2b4f93) test: Update snapshot due to logger changes - [`7f511d1`](https://github.com/better-auth/better-auth/commit/7f511d1a838eb0b219db06da30f8629640cd19c0) Merge branch 'main' into main ### 📊 Changes **28 files changed** (+204 additions, -124 deletions) <details> <summary>View changed files</summary> 📝 `packages/better-auth/package.json` (+1 -1) 📝 `packages/better-auth/src/__snapshots__/init.test.ts.snap` (+0 -3) 📝 `packages/better-auth/src/api/index.ts` (+30 -21) 📝 `packages/better-auth/src/api/middlewares/origin-check.ts` (+2 -3) 📝 `packages/better-auth/src/api/rate-limiter/index.ts` (+1 -2) 📝 `packages/better-auth/src/api/routes/callback.ts` (+5 -5) 📝 `packages/better-auth/src/api/routes/session.ts` (+13 -3) 📝 `packages/better-auth/src/api/routes/sign-in.ts` (+2 -2) 📝 `packages/better-auth/src/api/routes/sign-up.ts` (+2 -3) 📝 `packages/better-auth/src/db/get-migration.ts` (+2 -1) 📝 `packages/better-auth/src/init.ts` (+11 -13) 📝 `packages/better-auth/src/oauth2/state.ts` (+4 -5) 📝 `packages/better-auth/src/plugins/admin/admin.test.ts` (+1 -1) 📝 `packages/better-auth/src/plugins/email-otp/index.ts` (+4 -3) 📝 `packages/better-auth/src/plugins/generic-oauth/index.ts` (+8 -4) 📝 `packages/better-auth/src/plugins/jwt/jwt.test.ts` (+1 -1) 📝 `packages/better-auth/src/plugins/organization/organization.test.ts` (+1 -1) 📝 `packages/better-auth/src/plugins/organization/routes/crud-invites.ts` (+1 -2) 📝 `packages/better-auth/src/plugins/passkey/index.ts` (+1 -1) 📝 `packages/better-auth/src/plugins/phone-number/index.ts` (+1 -2) _...and 8 more files_ </details> ### 📄 Description So far, the options for customizing in logging are very limited or only log-level and activation can be set. It should be discussed which aspects of the logger should be configurable by the user closes #124 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-04-17 20:20:08 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/better-auth#28891