Skip to content

Conversation

@eduardoboronat-okta
Copy link
Contributor

Semgrep is Okta's static application security testing tool. It was previously recommended to add Semgrep directly into CI/CD pipelines or workflows. However, the ProdSec team has enhanced the tooling, and Semgrep scanning is now built into the platform via a GitHub Application.

ℹ️ Details

  • You'll find all of your PRs are already being scanned by a dedicated Semgrep step. Given this, you no longer need to run Semgrep via a Github Action workflow.
  • ⚠️ This PR attempts to help remove it for you. Please review this change carefully to ensure it doesn't impact any other jobs. Any adjustments needed to make this PR pass is the responsibility of the owners of the repository. (You are also welcome to close this PR and remove Semgrep CI separately).

Created by Sourcegraph batch change eduardo.boronat/Remove-Semgrep-GHA-non-EMU.

@eduardoboronat-okta eduardoboronat-okta requested a review from a team as a code owner August 6, 2025 14:49
@ankita10119
Copy link
Contributor

@eduardoboronat-okta

Thanks for raising this PR!

We’ve recently made some commits to master, so this branch is currently out of sync.
Once you’ve rebased or updated the branch, we’ll be good to proceed with merging.

@ankita10119
Copy link
Contributor

Heads-up on Snyk warning:

This PR fails the Snyk security check due to a known vulnerability in form-data@4.0.0, which is a transitive dependency

The vulnerable path is through deprecated packages like request, which currently block us from upgrading form-data

  • We’re aware of the issue and are tracking it separately (e.g., evaluating whether we can remove or replace request)

For now, we can proceed with this merge since it does not introduce new vulnerabilities, and no direct fix is currently possible.

@ankita10119 ankita10119 merged commit 9d85f7c into master Sep 10, 2025
6 of 7 checks passed
@ankita10119 ankita10119 deleted the Remove-Semgrep-GHA-non-EMU branch September 10, 2025 09:33
@ankita10119 ankita10119 mentioned this pull request Sep 12, 2025
7 tasks
ankita10119 added a commit that referenced this pull request Sep 12, 2025
### Changed 

- Bump karma from 6.4.3 to 6.4.4
- Bump pbkdf2 from 3.1.2 to 3.1.3
- Bump validator from 13.15.0 to 13.15.15
- Bump sha.js from 2.4.11 to 2.4.12
- Bump cipher-base from 1.0.4 to 1.0.6
- Bump codecov/codecov-action from 5.4.3 to 5.5.1
- Bump puppeteer from 24.9.0 to 24.19.0
- Bump tmp from 0.2.3 to 0.2.5
- bump fsevents to latest(SEC- 2161)
- Bump eslint-plugin-react from 7.34.1 to 7.37.5
- Bump @grpc/grpc-js and @google-cloud/translate

### Fixed

- Fix: social connection names not showing displayName correctly
[\#2651](#2651)
([omarquazi-okta](https://github.com/omarquazi-okta))
- Update old Twitter icon and name to "X"
[\#2649](#2649)
([omarquazi-okta](https://github.com/omarquazi-okta))
- Fix issue 2546 - TypeError: Super expression must either be null or a
function [\#2578](#2578)
([Hworden](https://github.com/Hworden))
- Fix: Accessibility Issues #2624
[\#2642](#2642)
([ankita10119](https://github.com/ankita10119))
- fix: Rename shop strategy
[\#2641](#2641)
([omarquazi-okta](https://github.com/omarquazi-okta))
- Fix release pipeline cdn
[\#2628](#2628)
([developerkunal](https://github.com/developerkunal))
- Fix Release PIPELINE [\#2627](#2627)
([developerkunal](https://github.com/developerkunal))
- chore: update .gitignore and Makefile for Puppeteer cache and config
directories [\#2626](#2626)
([developerkunal](https://github.com/developerkunal))
- Fix Makefile for Puppeteer cache support
[\#2625](#2625)
([developerkunal](https://github.com/developerkunal))


### Removed


- chore(ci): Remove Semgrep GHA Workflow
[\#2650](#2650)
([eduardoboronat-okta](https://github.com/eduardoboronat-okta))

### Security

- security: Remove vulnerable node-es-module-loader dependency
(SEC-2160) [\#2629](#2629)
([harekrishnarai](https://github.com/harekrishnarai))



### Testing

<!--
Please describe how this can be tested by reviewers. Be specific about
anything not tested and reasons why. If this library has unit and/or
integration testing, tests should be added for new functionality and
existing tests should complete without errors.
-->

* [ ] This change adds unit test coverage
* [ ] This change adds integration test coverage
* [ ] This change has been tested on the latest version of the
platform/language

### Checklist

* [x] I have read the [Auth0 general contribution
guidelines](https://github.com/auth0/open-source-template/blob/master/GENERAL-CONTRIBUTING.md)
* [x] I have read the [Auth0 Code of
Conduct](https://github.com/auth0/open-source-template/blob/master/CODE-OF-CONDUCT.md)
* [x] All code quality tools/guidelines have been run/followed
* [x] All relevant assets have been compiled
@ankita10119 ankita10119 mentioned this pull request Sep 12, 2025
7 tasks
ankita10119 added a commit that referenced this pull request Sep 12, 2025
### Changed 

- Bump karma from 6.4.3 to 6.4.4
- Bump pbkdf2 from 3.1.2 to 3.1.3
- Bump validator from 13.15.0 to 13.15.15
- Bump sha.js from 2.4.11 to 2.4.12
- Bump cipher-base from 1.0.4 to 1.0.6
- Bump codecov/codecov-action from 5.4.3 to 5.5.1
- Bump puppeteer from 24.9.0 to 24.19.0
- Bump tmp from 0.2.3 to 0.2.5
- bump fsevents to latest(SEC- 2161)
- Bump eslint-plugin-react from 7.34.1 to 7.37.5
- Bump @grpc/grpc-js and @google-cloud/translate

### Fixed

- Fix: social connection names not showing displayName correctly
[\#2651](#2651)
([omarquazi-okta](https://github.com/omarquazi-okta))
- Update old Twitter icon and name to "X"
[\#2649](#2649)
([omarquazi-okta](https://github.com/omarquazi-okta))
- Fix issue 2546 - TypeError: Super expression must either be null or a
function [\#2578](#2578)
([Hworden](https://github.com/Hworden))
- Fix: Accessibility Issues #2624
[\#2642](#2642)
([ankita10119](https://github.com/ankita10119))
- fix: Rename shop strategy
[\#2641](#2641)
([omarquazi-okta](https://github.com/omarquazi-okta))
- Fix release pipeline cdn
[\#2628](#2628)
([developerkunal](https://github.com/developerkunal))
- Fix Release PIPELINE [\#2627](#2627)
([developerkunal](https://github.com/developerkunal))
- chore: update .gitignore and Makefile for Puppeteer cache and config
directories [\#2626](#2626)
([developerkunal](https://github.com/developerkunal))
- Fix Makefile for Puppeteer cache support
[\#2625](#2625)
([developerkunal](https://github.com/developerkunal))


### Removed


- chore(ci): Remove Semgrep GHA Workflow
[\#2650](#2650)
([eduardoboronat-okta](https://github.com/eduardoboronat-okta))

### Security

- security: Remove vulnerable node-es-module-loader dependency
(SEC-2160) [\#2629](#2629)
([harekrishnarai](https://github.com/harekrishnarai))



### Testing

<!--
Please describe how this can be tested by reviewers. Be specific about
anything not tested and reasons why. If this library has unit and/or
integration testing, tests should be added for new functionality and
existing tests should complete without errors.
-->

* [ ] This change adds unit test coverage
* [ ] This change adds integration test coverage
* [ ] This change has been tested on the latest version of the
platform/language

### Checklist

* [x] I have read the [Auth0 general contribution
guidelines](https://github.com/auth0/open-source-template/blob/master/GENERAL-CONTRIBUTING.md)
* [x] I have read the [Auth0 Code of
Conduct](https://github.com/auth0/open-source-template/blob/master/CODE-OF-CONDUCT.md)
* [x] All code quality tools/guidelines have been run/followed
* [x] All relevant assets have been compiled
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.

4 participants