Skip to content

Add ci.opensearch.org/m2/ mirror to buildscript and project repos (sql) - #5667

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 (sql)#5667
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 (sql)

Issues Resolved

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

@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 8a16fa7.

PathLineSeverityDescription
build.gradle71highA new Maven repository URL (https://ci.opensearch.org/m2/) is added in three build contexts (buildscript, project, and subprojects). While the domain matches existing trusted repos, the path /m2/ differs from the existing /maven2/ and snapshot paths already present. Per mandatory policy, all new package registry sources must be flagged regardless of apparent legitimacy — maintainers should verify this endpoint is an officially sanctioned OpenSearch artifact mirror and not a path that could serve attacker-controlled artifacts.

The table above displays the top 10 most important findings.

Total: 1 | Critical: 0 | High: 1 | 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 skip-changelog skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. labels Aug 1, 2026
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon
peterzhuamazon merged commit 79d532d into opensearch-project:main Aug 1, 2026
39 checks passed
@peterzhuamazon
peterzhuamazon deleted the fix-maven-429 branch August 1, 2026 17:10
@github-project-automation github-project-automation Bot moved this from 👀 In Review to ✅ Done in Engineering Effectiveness Board Aug 1, 2026
peterzhuamazon added a commit that referenced this pull request Aug 1, 2026
…l) (#5667) (#5668)

(cherry picked from commit 79d532d)

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>
@peterzhuamazon peterzhuamazon added release v3.8.0 Issues and PRs related to version v3.8.0 labels Aug 1, 2026
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-changelog 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