Skip to content

Add ci.opensearch.org/m2/ mirror to buildscript and project repos (security-analytics) - #1755

Merged
peterzhuamazon merged 1 commit into
opensearch-project:mainfrom
peterzhuamazon:fix-maven-429
Aug 1, 2026
Merged

Add ci.opensearch.org/m2/ mirror to buildscript and project repos (security-analytics)#1755
peterzhuamazon merged 1 commit into
opensearch-project:mainfrom
peterzhuamazon:fix-maven-429

Conversation

@peterzhuamazon

Copy link
Copy Markdown
Member

Description

Add ci.opensearch.org/m2/ mirror to buildscript and project repos (security-analytics)

Issues Resolved

opensearch-project/opensearch-build#6278 (comment)

…curity-analytics)

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@github-actions

github-actions Bot commented Aug 1, 2026

Copy link
Copy Markdown
Contributor

PR Code Analyzer ❗

AI-powered 'Code-Diff-Analyzer' found issues on commit ec03a83.

Hard block: Issues at High severity or above will block this PR from merging.

PathLineSeverityDescription
build.gradle53highNew Maven repository URL 'https://ci.opensearch.org/m2/' added to buildscript repositories. This is a package source change that must be verified — artifacts resolved from this endpoint could be substituted or tampered with, and the path '/m2/' differs from the existing '/maven2/' endpoint whose legitimacy was previously established.
build.gradle186highNew Maven repository URL 'https://ci.opensearch.org/m2/' added to publishing repositories. Same unverified endpoint added to the publishing block, which controls where build artifacts are deployed. Adding an unverified repository to the publishing configuration could redirect artifact uploads to an unintended or attacker-controlled location.

The table above displays the top 10 most important findings.

Total: 2 | Critical: 0 | High: 2 | Medium: 0 | Low: 0


Pull Requests Author(s): Please update your Pull Request according to the report above.

Repository Maintainer(s): You can bypass diff analyzer by adding label skip-diff-analyzer after reviewing the changes carefully, then re-run failed actions. To re-enable the analyzer, remove the label, then re-run all actions.


⚠️ Note: The Code-Diff-Analyzer helps protect against potentially harmful code patterns. Please ensure you have thoroughly reviewed the changes beforehand.

Thanks.

@peterzhuamazon peterzhuamazon added the skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. label Aug 1, 2026
@peterzhuamazon

Copy link
Copy Markdown
Member Author

expected mirror update.

@peterzhuamazon
peterzhuamazon merged commit 54b0e74 into opensearch-project:main Aug 1, 2026
35 of 36 checks passed
@github-project-automation github-project-automation Bot moved this from 👀 In Review to ✅ Done in Engineering Effectiveness Board Aug 1, 2026
@peterzhuamazon
peterzhuamazon deleted the fix-maven-429 branch August 1, 2026 17:29
peterzhuamazon added a commit that referenced this pull request Aug 1, 2026
…curity-analytics) (#1755) (#1756)

(cherry picked from commit 54b0e74)

Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com>
Co-authored-by: Peter Zhu <zhujiaxi@amazon.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport 3.8 enhancement New feature or request release skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. v3.8.0 Issues and PRs related to version v3.8.0

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

2 participants