Skip to content

chore(logging): add structured backend logging for PB Website#481

Open
adityamandre25 wants to merge 5 commits intopointblank-club:stagingfrom
adityamandre25:feature/auth-structured-logging
Open

chore(logging): add structured backend logging for PB Website#481
adityamandre25 wants to merge 5 commits intopointblank-club:stagingfrom
adityamandre25:feature/auth-structured-logging

Conversation

@adityamandre25
Copy link
Copy Markdown

@adityamandre25 adityamandre25 commented Jan 7, 2026 •
edited
Loading

PB-44

Summary: What does this PR do?

Adds structured backend logging to the PB Website using pino.
Logging is standardized across core backend services, and unsafe logging of sensitive data has been removed.
https://linear.app/point-blank/issue/PB-44/setup-logging-for-pb-website

Changes Made

  • Introduced centralized structured logging (pino)
  • Added logging to auth flows, auth guard, DB connection, session initialization, and rate limiter
  • Removed unsafe logging of session secrets

Type of Change

  • New feature (non-breaking change which adds functionality)
  • Code cleanup or refactor

How Has This Been Tested?

  • Manual tests

Test cases:

  1. Verified application starts and emits structured JSON logs to stdout.

Dependencies

  • Added pino for backend logging (lockfile updated).

Comments

  • Backend-only changes.

@github-actions github-actions bot added Dependencies Pull requests that update a dependency file Utils Changes to utility size-XL size label for pr with extra large changes and removed size-XL size label for pr with extra large changes labels Jan 7, 2026
@vaibhavgarg25
Copy link
Copy Markdown
Contributor

vaibhavgarg25 commented Jan 8, 2026

LGTM

@github-actions github-actions bot added size-L size label for pr with large changes and removed size-XL size label for pr with extra large changes size-L size label for pr with large changes labels Jan 19, 2026
@adityamandre25
Copy link
Copy Markdown
Author

adityamandre25 commented Jan 26, 2026

Added pino redaction

@github-actions github-actions bot added size-L size label for pr with large changes and removed size-L size label for pr with large changes labels Jan 26, 2026
@github-actions github-actions bot added size-XL size label for pr with extra large changes and removed size-L size label for pr with large changes labels Jan 26, 2026
@github-actions github-actions bot added the size-XL size label for pr with extra large changes label Jan 27, 2026
@adityamandre25 adityamandre25 force-pushed the feature/auth-structured-logging branch from 56c2796 to b187e7f Compare January 27, 2026 08:37
@github-actions github-actions bot added size-L size label for pr with large changes and removed size-XL size label for pr with extra large changes labels Jan 27, 2026
@adityamandre25 adityamandre25 force-pushed the feature/auth-structured-logging branch from b187e7f to a69335e Compare January 27, 2026 17:27
@github-actions github-actions bot added size-M size label for pr with medium changes and removed size-L size label for pr with large changes labels Jan 27, 2026
@adityamandre25 adityamandre25 force-pushed the feature/auth-structured-logging branch from a69335e to b34c348 Compare January 27, 2026 17:47
@github-actions github-actions bot added size-M size label for pr with medium changes and removed size-M size label for pr with medium changes labels Jan 27, 2026
@adityamandre25 adityamandre25 force-pushed the feature/auth-structured-logging branch from b34c348 to 7a80cc4 Compare January 27, 2026 17:50
@github-actions github-actions bot added size-M size label for pr with medium changes size-XL size label for pr with extra large changes and removed size-M size label for pr with medium changes size-XL size label for pr with extra large changes labels Jan 27, 2026
@Aneesh-Hegde
Copy link
Copy Markdown
Contributor

Aneesh-Hegde commented Feb 10, 2026 •
edited
Loading

Line 14-18(auth.ts) and 5-8(requireAuth.ts) - Log for a function start not needed
Line 23-28(auth.ts) - Also log for api call not needed
Line 70-74(auth.ts) -If the code reaches to this section it is guaranteed that email exist so chances of token not being generated is almost impossible, so not needed

@adityamandre25

@github-actions github-actions bot added size-XL size label for pr with extra large changes and removed size-XL size label for pr with extra large changes labels Feb 11, 2026
@Aneesh-Hegde
Copy link
Copy Markdown
Contributor

Aneesh-Hegde commented Feb 11, 2026

@slashexx LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SkySingh04 SkySingh04 Awaiting requested review from SkySingh04 SkySingh04 is a code owner

@XploY04 XploY04 Awaiting requested review from XploY04 XploY04 is a code owner

@alfiyafatima09 alfiyafatima09 Awaiting requested review from alfiyafatima09 alfiyafatima09 is a code owner

@0Xero7 0Xero7 Awaiting requested review from 0Xero7

@mohit-nagaraj mohit-nagaraj Awaiting requested review from mohit-nagaraj mohit-nagaraj is a code owner

@WasThatRudy WasThatRudy Awaiting requested review from WasThatRudy WasThatRudy is a code owner

@slashexx slashexx Awaiting requested review from slashexx slashexx is a code owner

@ShreyashSri ShreyashSri Awaiting requested review from ShreyashSri ShreyashSri is a code owner

@dhruvpuri-aspora dhruvpuri-aspora Awaiting requested review from dhruvpuri-aspora

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

API Issue related to API routes CI/CD Issues related to CI/CD pipelines, including workflow changes. Config Changes to project configuration files Dependencies Pull requests that update a dependency file Documentation Improvements or additions to documentation Frontend Frontend issue using Tailwind and css Github Actions workflow changes size-L size label for pr with large changes Styling Changes in css and tailwind Utils Changes to utility

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@adityamandre25 @vaibhavgarg25 @Aneesh-Hegde @autistic-avenger