Skip to content

feat: proxy support for scanner sub-processes#42

Merged
Nitr4x merged 11 commits into
mainfrom
copilot/create-gitlab-ci-example
Apr 27, 2026
Merged

feat: proxy support for scanner sub-processes#42
Nitr4x merged 11 commits into
mainfrom
copilot/create-gitlab-ci-example

Conversation

Copilot AI commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Description

Adds HTTPS proxy support documentation and fixes the engine initialization exit code.

  • Added a --cap-add SYS_PTRACE note to the Running with Docker section of the README, explaining it is required when an HTTPS proxy is configured so that OpenGrep can bootstrap its bundled Python interpreter. Examples are provided for docker run, Docker Compose, and GitHub Actions.
  • Changed os.Exit(1) to os.Exit(-2) in main.go when eng.Initialize fails, to use a distinct exit code for engine initialization failures.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that causes existing functionality to change)
  • Documentation update
  • Refactor / code cleanup
  • CI / build change

How Has This Been Tested?

go build ./...
go vet ./...
go test ./...

Checklist

  • My code follows the project's coding standards (go vet ./... passes)
  • I have run the existing test suite and all tests pass (go test ./...)
  • I have added or updated tests that cover my changes
  • I have updated documentation where relevant (README, comments, etc.)
  • I have not introduced new secrets, credentials, or sensitive data
  • My changes do not break backward compatibility (or I have documented breaking changes above)

Screenshots / Output (if applicable)

N/A

Copilot AI and others added 6 commits April 24, 2026 16:04
Comment thread main.go Outdated
eng.Initialize(config)
if err := eng.Initialize(config); err != nil {
logger.Error(err.Error())
os.Exit(1)

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.

@copilot We should exit -2

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Changed to os.Exit(-2) in commit 0b20b49.

Copilot AI requested a review from Nitr4x April 27, 2026 15:34
@Nitr4x Nitr4x marked this pull request as ready for review April 27, 2026 15:37
@Nitr4x Nitr4x merged commit 13309e4 into main Apr 27, 2026
2 checks passed
@Nitr4x Nitr4x deleted the copilot/create-gitlab-ci-example branch April 27, 2026 15:37
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.

2 participants