Skip to content

fix(bg/paymentSession): prefix logs per instance#1058

Merged
sidvishnoi merged 2 commits into
mainfrom
logger-per-instance
May 22, 2025
Merged

fix(bg/paymentSession): prefix logs per instance#1058
sidvishnoi merged 2 commits into
mainfrom
logger-per-instance

Conversation

@sidvishnoi
Copy link
Copy Markdown
Member

@sidvishnoi sidvishnoi commented May 21, 2025

Context

Found useful when working on payment streams.

Changes proposed in this pull request

Create an instance of logger per PaymentSession.

Before After
image image

Might make requestId shorter in future 😄

@sidvishnoi sidvishnoi requested a review from DarianM May 21, 2025 11:06
@github-actions github-actions Bot added area: background Improvements or additions to extension background script area: shared Changes to shared libraries and utilities labels May 21, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 21, 2025

Extension builds preview

Name Link
Latest commit 23ff3a0
Latest job logs Run #15160696693
Chrome (990.26KB)Download
Firefox (990.29KB)Download

@sidvishnoi sidvishnoi merged commit 21e38bc into main May 22, 2025
10 checks passed
@sidvishnoi sidvishnoi deleted the logger-per-instance branch May 22, 2025 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: background Improvements or additions to extension background script area: shared Changes to shared libraries and utilities

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants