Skip to content

Fix test due to scan-pr environment enforcement V3#1265

Open
eranturgeman wants to merge 4 commits intojfrog:v3_erfrom
eranturgeman:fix-oidc-workflow-v3
Open

Fix test due to scan-pr environment enforcement V3#1265
eranturgeman wants to merge 4 commits intojfrog:v3_erfrom
eranturgeman:fix-oidc-workflow-v3

Conversation

@eranturgeman
Copy link
Copy Markdown
Collaborator

@eranturgeman eranturgeman commented Mar 26, 2026

  • 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 PR adds environment to the OIDC yml file according to recent security hardening with pull_request_target trigger that activates frogbot scan-pr flow

@eranturgeman eranturgeman mentioned this pull request Mar 26, 2026
4 tasks
@eranturgeman eranturgeman added safe to test Approve running integration tests on a pull request ignore for release Automatically generated release notes labels Mar 26, 2026
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Mar 26, 2026
@eranturgeman eranturgeman added the safe to test Approve running integration tests on a pull request label Mar 29, 2026
@github-actions github-actions bot removed the safe to test Approve running integration tests on a pull request label Mar 29, 2026
@eranturgeman eranturgeman changed the title fix test due to scan-pr environment enforcement Fix test due to scan-pr environment enforcement V3 Mar 29, 2026
Copy link
Copy Markdown
Contributor

@attiasas attiasas left a comment

Choose a reason for hiding this comment

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

LGTM

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