Skip to content

Commit

Permalink
Exclude tsp.exe from binskim (#6145)
Browse files Browse the repository at this point in the history
PR adding some baseline thing to give time to fix issues,
#6134

we can't actually fix those as we don't build this binary we just inject
our code into node's so just ignoring the validation of the `tsp.exe`
  • Loading branch information
timotheeguerin authored Feb 25, 2025
1 parent e0cf97c commit 4dcc26b
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions eng/common/pipelines/ci.yml
Original file line number Diff line number Diff line change
@@ -12,6 +12,8 @@ pr:
extends:
template: /eng/common/pipelines/templates/1es-redirect.yml
parameters:
BinSkimSettings:
targetPathExclusionPattern: "tsp.exe" # Flag issue with node binary which we can't fix https://github.com/nodejs/node/issues/42100
stages:
- stage: InitStage
displayName: Initialize
2 changes: 2 additions & 0 deletions eng/tsp-core/pipelines/publish.yml
Original file line number Diff line number Diff line change
@@ -15,6 +15,8 @@ pr: none
extends:
template: /eng/common/pipelines/templates/1es-redirect.yml
parameters:
BinSkimSettings:
targetPathExclusionPattern: "tsp.exe" # Flag issue with node binary which we can't fix https://github.com/nodejs/node/issues/42100
variables:
- template: /eng/tsp-core/pipelines/templates/variables/globals.yml@self

0 comments on commit 4dcc26b

Please sign in to comment.