Skip to content

Add ci.opensearch.org/m2/ mirror for plugin resolution (security-analytics) - #1751

Merged
peterzhuamazon merged 2 commits into
opensearch-project:3.8from
shreyah963:update-plugin-m2
Jul 31, 2026
Merged

Add ci.opensearch.org/m2/ mirror for plugin resolution (security-analytics)#1751
peterzhuamazon merged 2 commits into
opensearch-project:3.8from
shreyah963:update-plugin-m2

Conversation

@shreyah963

@shreyah963 shreyah963 commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

Description

Add ci.opensearch.org/m2/ mirror for plugin resolution (security-analytics)

Issues Resolved

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

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@github-actions

github-actions Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

PR Code Analyzer ❗

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

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

PathLineSeverityDescription
build-tools/repositories.gradle9highNew Maven repository source added: https://ci.opensearch.org/m2/. Per mandatory flagging rules, all package registry additions must be flagged regardless of apparent legitimacy. This URL path (/m2/) differs from the existing OpenSearch repo (/maven2/) and could resolve to a different artifact set. Maintainers must verify this path is intentional and controlled by the OpenSearch project.
settings.gradle9highNew Maven repository source added to pluginManagement: https://ci.opensearch.org/m2/. This mirrors the change in repositories.gradle and affects Gradle plugin resolution in addition to dependency resolution. Plugins loaded via pluginManagement have deep build-time access; an attacker controlling this path could inject malicious build plugins. Maintainers must verify ownership and contents of this URL before merging.

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.

…ytics)

Signed-off-by: shreyah963 <shreyab963@gmail.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
@peterzhuamazon peterzhuamazon added the skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis. label Jul 31, 2026
@peterzhuamazon

Copy link
Copy Markdown
Member

Expected new mirror for gradle plugins.

@peterzhuamazon
peterzhuamazon merged commit 2061a21 into opensearch-project:3.8 Jul 31, 2026
20 of 22 checks passed
peterzhuamazon added a commit that referenced this pull request Jul 31, 2026
…ytics) (#1751) (#1754)

* Add ci.opensearch.org/m2/ mirror for plugin resolution (security-analytics)



* Address order issues



---------




(cherry picked from commit 2061a21)

Signed-off-by: shreyah963 <shreyab963@gmail.com>
Signed-off-by: Peter Zhu <zhujiaxi@amazon.com>
Signed-off-by: opensearch-ci-bot <opensearch-infra@amazon.com>
Co-authored-by: Shreya Bhatta <shreyab963@gmail.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 main skip-diff-analyzer Maintainer to skip code-diff-analyzer check, after reviewing issues in AI analysis.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants