Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add: test code #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

add: test code #1

wants to merge 3 commits into from

Conversation

gassara-kys
Copy link
Contributor

@gassara-kys gassara-kys commented Dec 27, 2023

This is test PR.

}

func osCmdInjection(userInput string) error {
cmd := exec.Command(userInput) // OS command injection
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

問題のコードを発見しました。修正が必要か確認してください🙏

コードスキャン結果

  • Semgrep CheckID:
    go.lang.security.audit.dangerous-exec-command.dangerous-exec-command
  • 説明:
    Detected non-static command inside Command. Audit the input to 'exec.Command'. If unverified user data can reach this call site, this is a code injection vulnerability. A malicious actor can inject a malicious script to execute arbitrary code.

RISKENで確認

より詳細な情報や生成AIによる解説はRISKENコンソール上で確認できます。

By RISKEN review

}

const (
TEST_SECRET_KEY = "AKIAIMNOJVGFDXXXE4OA" // aws test key. https://github.com/gitleaks/gitleaks
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

シークレット情報が含まれている可能性があります👀

シークレットスキャン結果

  • シークレットタイプ: AWS
  • 説明:
    対象データがテスト用のダミーデータや公開可能な情報であるか確認してください。
    もし、有効なシークレット情報の場合はキーのローテーションや権限の削除(キーの無効化)を行ってください。
    どうしてもシークレット情報をコミットする必要がある場合は、プライベートリポジトリになっていることを確認しアクセスできる人を限定してください。

RISKENで確認

より詳細な情報や生成AIによる解説はRISKENコンソール上で確認できます。

By RISKEN review

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.

None yet

1 participant