Repository navigation
Merge staging branch into main(AST-0000) - #1614
Merged
Merged
Conversation
…oks (AST-180812) (#1603)
* Update github.com/andybalholm/brotli from v1.2.0 to v1.2.7 (AST-189161) Indirect dependency pulled in via anchore/syft -> anchore/archiver. No API removals; archiver only uses NewWriterLevel/NewReader, which are unchanged. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * Update Docker base image checkmarx/bash:5.3 to latest digest (AST-189161) Bumps the SHA256-pinned base image to the 5.3-r14 build pushed on 2026-11-04. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * Replace .trivyignore entries with current binary findings (AST-189161) Ignore GO-2026-5932 (x/crypto/openpgp via helm provenance), CVE-2026-78661 and CVE-2026-97045 (x/net/html via syft Maven cataloger). Remove the previous Wolfi base-image and syft/docker entries, which no longer apply. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * Restore syft and docker entries in .trivyignore (AST-189161) CVE-2026-33481 (syft temp cleanup) and CVE-2026-33997 (moby plugin privilege bypass) are still reported against the binary. Neither has an adoptable fix: syft >= 1.42.3 reintroduces the binary-size regression reverted in AST-188320 and docker/docker has no 29.x release on its module path. Justifications and reachability analysis are recorded in the file. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> * Bump golang.org/x/net to v0.62.0 and trim .trivyignore (AST-189161) Bump the indirect golang.org/x/net dependency to v0.62.0, which fixes CVE-2026-78661 and CVE-2026-97045, so their ignore entries are dropped. Keep only the ignores that cannot be fixed directly: GO-2026-5932 (x/crypto openpgp, via helm), CVE-2026-33481 (syft) and CVE-2026-33997 (docker). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com> Co-authored-by: atishj99 <atish.jadhav@checkmarx.com>
Co-authored-by: Aniket Shinde <aniket.shinde@checkmarx.com>
…(AST-189161) Replace `uses: ./.github/workflows/ci-tests.yml` with the `$/` form to satisfy the zizmor `self-repository` audit (added in zizmor v1.30.0). Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
* Handle missing feature flags gracefully (code 100 response) When the API returns a 404 with error code 100 and message "flag with the given name is not found", return a disabled flag model instead of nil to prevent nil pointer dereference panics in calling code. Missing flags are treated as enabled (default) to ensure safe feature flag fallback behavior. Fixes panic: runtime error: invalid memory address or nil pointer dereference when GetSpecificFlag is called and the flag doesn't exist on the server. * Refactor: use existing WebError struct instead of inline type Replace inline error response struct with the existing WebError struct from ast.go for consistency with the rest of the codebase. This improves maintainability and follows the established pattern in wrappers like bfl-http.go and codebashing-http.go. * handle remove ff * Remove INCREASE_FILE_UPLOAD_LIMIT flag - now always enabled for large files Removed the INCREASE_FILE_UPLOAD_LIMIT feature flag that controlled multipart uploads. The flag is now permanently enabled - files > 5GB always use multipart upload, smaller files use single upload. This simplifies the code by removing the flag check and associated tests. Changes: - Removed flag constant from feature-flags.go - Updated scan.go to always check file size (no flag check) - Consolidated 5 tests to 3 tests (removed redundant flag-based tests) - Behavior unchanged: multipart for >5GB, single for <=5GB No regression - functionality preserved, code simplified. * removed status code handling * updated docker image --------- Co-authored-by: Anurag Dalke <120229307+cx-anurag-dalke@users.noreply.github.com>
Staging has a newer docker image version (fad83378373438) that was intentionally updated. Keeping staging's version when merging staging into main. Conflict resolution for #1614: Merge staging branch into main Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
Both are newly disclosed. glibc comes from the checkmarx/bash base image (fix in 2.44-r16, no rebuilt image yet); stdlib is fixed in Go 1.26.11. Remove once the base image is rebuilt and the Go toolchain is bumped. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
cx-aniket-shinde
approved these changes
Nov 12, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
By submitting this pull request, you agree to the terms within the Checkmarx Code of Conduct. Please review the contributing guidelines for guidance on creating high-quality pull requests.
Description
Please provide a summary of the changes and the related issue. Include relevant motivation and context.
Type of Change
Related Issues
Link any related issues or tickets.
Checklist
Screenshots (if applicable)
Add screenshots to help explain your changes.
Additional Notes
Add any other relevant information.