simon-id comments

Results 74 comments of


                                            simon-id

add experimental tracing using a native event collector

comments closed, LGTM :+1:

[node.js] Enable path_params blocking tests

@cbeauchesne yes :)

[node.js] Enable path_params blocking tests

@CarlesDD can't approve because it's my own PR but LGTM

WIP Rasp LFI first approach

Note to remember later: we've previously decided to keep plugins out of the appsec code and with the other tracing plugins. Let's consider this question again.

Add pipeline to Mongodb query

@vdpjulien a member from dd-trace-js needs to take over your PR to recreate it and pass the CI, i'll forward it

Add pipeline to Mongodb query

@tlhunter would this be under the responsibility of your team ?

Add pipeline to Mongodb query

@oursodon I pinged again the relevant people, don't hesitate to ask again if you don't get a reply

refactor: move remote_config code to the root of dd-trace

I see there is activity on this PR, FYI I'm planning on reviewing it and adding the appsec changes next week

try supporting node 14 on v5

I don't think there is any roadblock from ASM's perspective, as we backport everything anyway. We'd just have to doublecheck our CI testing the right stuff in the right versions....

update path-to-regexp

Ah this rings a bell, and it looks like the CI is failing so maybe that's the reason we never updated it ? it's worth a few retries