Skip to content

Remove scratch notes files accidentally committed - #21784

Merged
mgodwan merged 1 commit into
opensearch-project:mainfrom
KhishorekumarBS:remove-scratch-md-files
May 21, 2026
Merged

Remove scratch notes files accidentally committed#21784
mgodwan merged 1 commit into
opensearch-project:mainfrom
KhishorekumarBS:remove-scratch-md-files

Conversation

@KhishorekumarBS

Copy link
Copy Markdown
Contributor

Description

Remove scratch md files

Signed-off-by: Khishorekumar BS <bkhishor@amazon.com>
@KhishorekumarBS
KhishorekumarBS requested a review from a team as a code owner May 21, 2026 15:07
@github-actions

github-actions Bot commented May 21, 2026

Copy link
Copy Markdown
Contributor

PR Code Analyzer ❗

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

PathLineSeverityDescription
prune-block-cache-testing.md18lowDocumentation (now deleted) contained a curl-pipe-to-shell pattern for installing Rust (`curl ... https://sh.rustup.rs | sh`). While standard for Rust installation, such patterns in shared repo docs can encourage unsafe practices. File is being removed, which is the correct action.
prune-block-cache-testing.md32lowHardcoded developer home directory paths (`/home/bkhishor/`) are present throughout the deleted testing notes, leaking an internal developer username. The file deletion resolves this information disclosure.

The table above displays the top 10 most important findings.

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


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.

@github-actions

github-actions Bot commented May 21, 2026

Copy link
Copy Markdown
Contributor

PR Reviewer Guide 🔍

(Review updated until commit 0ebd982)

Here are some key observations to aid the review process:

🧪 No relevant tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown
Contributor

Persistent review updated to latest commit 0ebd982

@mgodwan
mgodwan merged commit 7ea0d77 into opensearch-project:main May 21, 2026
23 checks passed
KhishorekumarBS added a commit to KhishorekumarBS/OpenSearch that referenced this pull request Jul 3, 2026
…#21784)

Signed-off-by: Khishorekumar BS <bkhishor@amazon.com>
Co-authored-by: Khishorekumar BS <bkhishor@amazon.com>
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