Skip to content

Add OTEL support#102

Open
IAmJSD wants to merge 3 commits into
neondatabase:mainfrom
IAmJSD:otel
Open

Add OTEL support#102
IAmJSD wants to merge 3 commits into
neondatabase:mainfrom
IAmJSD:otel

Conversation

@IAmJSD

@IAmJSD IAmJSD commented Sep 11, 2024

Copy link
Copy Markdown

Adds opentelemetry support to the serverless driver. I need to do further testing, but would appreciate testing/feedback and this seems to pass the latency tests so dropping here

@orca-security-us orca-security-us Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 1   low 0   info 0 View in Orca
☢️ The following Vulnerabilities (CVEs) have been detected
PACKAGE FILE CVE ID INSTALLED VERSION FIXED VERSION
high path-to-regexp ./package-lock.json CVE-2024-45296 2.2.1 0.1.10, 8.0.0 View in code

@jawj

jawj commented Sep 17, 2024

Copy link
Copy Markdown
Collaborator

@IAmJSD Thanks for this. We're discussing it internally and will come back to you soon.

@IAmJSD

IAmJSD commented Sep 20, 2024

Copy link
Copy Markdown
Author

@jawj No problem! Let me know when there are any updates/changes you want me to make!

(ps: cool to see someone from Brighton replying - small world lol 👍)

@manticarodrigo

Copy link
Copy Markdown

@jawj @IAmJSD is there an existing solve for this? Trying to get Sentry query monitoring working which would normally auto-instrument pg (node-postgres) but does not seemingly do it for the neon serverless driver.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants