Skip to content

Keep all JF_SAST* env vars when sanitizing JF env#1342

Merged
eranturgeman merged 2 commits into
jfrog:v3_erfrom
eranturgeman:keep-new-sast-env-var-after-clearing-env
Jun 4, 2026
Merged

Keep all JF_SAST* env vars when sanitizing JF env#1342
eranturgeman merged 2 commits into
jfrog:v3_erfrom
eranturgeman:keep-new-sast-env-var-after-clearing-env

Conversation

@eranturgeman

Copy link
Copy Markdown
Collaborator
  • All tests passed. If this feature is not already covered by the tests, I added new tests.
  • This pull request is on the dev branch.
  • I used gofmt for formatting the code before submitting the pull request.
  • Update documentation about new features / new supported technologies

This env var is required to be kept to enable new Sast scanner mode that required getting its value through this env var

@eranturgeman eranturgeman added safe to test Approve running integration tests on a pull request ignore for release Automatically generated release notes labels Jun 2, 2026
@eranturgeman eranturgeman requested a review from attiasas June 2, 2026 11:08
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jun 2, 2026
Comment thread utils/getconfiguration.go
@eranturgeman eranturgeman added the safe to test Approve running integration tests on a pull request label Jun 4, 2026
@github-actions github-actions Bot removed the safe to test Approve running integration tests on a pull request label Jun 4, 2026

@attiasas attiasas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@eranturgeman eranturgeman merged commit 2d069ca into jfrog:v3_er Jun 4, 2026
41 of 54 checks passed
@eranturgeman eranturgeman changed the title keep JF_SAST_DEFAULT_SCAN_MODE when sanitizing JF env Keep all JF_SAST* env vars when sanitizing JF env Jun 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ignore for release Automatically generated release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants