Skip to content

Conversation

Vadman97
Copy link
Contributor

@Vadman97 Vadman97 commented Aug 7, 2025

Summary

How did you test this change?

Are there any deployment considerations?

Comment on lines +208 to +218
setTimeout(async () => {
const { registerInstrumentations } = await import('@opentelemetry/instrumentation')

// Load instrumentations one by one with delays
const instrumentations = []

// Load fetch instrumentation
try {
const fetchModule = await this.loadInstrumentation('fetch')
instrumentations.push(new fetchModule.FetchInstrumentation())
} catch (e) {

Check notice

Code scanning / devskim

If untrusted data (data from HTTP requests, user submitted files, etc.) is included in an setTimeout statement it can allow an attacker to inject their own code. Note

Review setTimeout for untrusted data
Comment on lines +223 to +227
setTimeout(async () => {
try {
const xhrModule = await this.loadInstrumentation('xhr')
instrumentations.push(new xhrModule.XMLHttpRequestInstrumentation())
} catch (e) {

Check notice

Code scanning / devskim

If untrusted data (data from HTTP requests, user submitted files, etc.) is included in an setTimeout statement it can allow an attacker to inject their own code. Note

Review setTimeout for untrusted data
Comment on lines +233 to +237
setTimeout(async () => {
try {
const docModule = await this.loadInstrumentation('document')
instrumentations.push(new docModule.DocumentLoadInstrumentation())
} catch (e) {

Check notice

Code scanning / devskim

If untrusted data (data from HTTP requests, user submitted files, etc.) is included in an setTimeout statement it can allow an attacker to inject their own code. Note

Review setTimeout for untrusted data
Comment on lines +386 to +388
setTimeout(() => {
const nav = performance.getEntriesByType('navigation')[0] as PerformanceNavigationTiming
if (nav) {

Check notice

Code scanning / devskim

If untrusted data (data from HTTP requests, user submitted files, etc.) is included in an setTimeout statement it can allow an attacker to inject their own code. Note

Review setTimeout for untrusted data
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.

1 participant