Skip to content

[doc](security) Add SECURITY.md threat model entry point#63553

Merged
CalvinKirs merged 1 commit into
apache:masterfrom
CalvinKirs:security-model-fix
May 23, 2026
Merged

[doc](security) Add SECURITY.md threat model entry point#63553
CalvinKirs merged 1 commit into
apache:masterfrom
CalvinKirs:security-model-fix

Conversation

@CalvinKirs
Copy link
Copy Markdown
Member

What problem does this PR solve?

Issue Number: None

Related PR: None

Problem Summary: Security tooling and reviewers expect a root SECURITY.md entry point. The threat model already defined SECURITY.md coexistence under M16, but the repository did not provide the conventional file.

Release note

None

Check List (For Author)

  • Test: Manual test
    • Ran git diff --cached --check and verified the SECURITY.md -> threat-model.md discovery chain.
  • Behavior changed: No
  • Does this need documentation: No, documentation-only change.

What problem does this PR solve?

Issue Number: close #xxx

Related PR: #xxx

Problem Summary:

Release note

None

Check List (For Author)

  • Test

    • Regression test
    • Unit Test
    • Manual test (add detailed scripts or steps below)
    • No need to test or manual test. Explain why:
      • This is a refactor/code format and no logic has been changed.
      • Previous test can cover this change.
      • No code files have been changed.
      • Other reason
  • Behavior changed:

    • No.
    • Yes.
  • Does this need documentation?

    • No.
    • Yes.

Check List (For Reviewer who merge this PR)

  • Confirm the release note
  • Confirm test cases
  • Confirm document
  • Add branch pick label

### What problem does this PR solve?

Issue Number: None

Related PR: None

Problem Summary: Security tooling and reviewers expect a root SECURITY.md entry point. The threat model already defined SECURITY.md coexistence under M16, but the repository did not provide the conventional file.

### Release note

None

### Check List (For Author)

- Test: Manual test
    - Ran `git diff --cached --check` and verified the SECURITY.md -> threat-model.md discovery chain.
- Behavior changed: No
- Does this need documentation: No, documentation-only change.
@CalvinKirs CalvinKirs requested a review from zclllyybb as a code owner May 23, 2026 00:15
@hello-stephen
Copy link
Copy Markdown
Contributor

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?

@morningman
Copy link
Copy Markdown
Contributor

skip buildall

@CalvinKirs
Copy link
Copy Markdown
Member Author

/review

Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

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

Reviewed PR 63553.

No blocking issues found.

Critical checkpoint conclusions:

  • Goal/test: Adds a root SECURITY.md entry point and updates threat-model references consistently; documentation-only manual verification is sufficient.
  • Scope/focus: Change is small, focused, and clear.
  • Concurrency/lifecycle/config/compatibility/transactions/data writes/protocol: Not applicable to this documentation-only change.
  • Parallel paths: AGENTS.md, SECURITY.md, and threat-model.md consistently route security triage through SECURITY.md to threat-model.md.
  • Tests/results: No executable tests required; no generated test results changed.
  • Observability/performance: Not applicable.
  • User focus: No additional user-provided review focus.

@github-actions github-actions Bot added the approved Indicates a PR has been approved by one committer. label May 23, 2026
@github-actions
Copy link
Copy Markdown
Contributor

PR approved by at least one committer and no changes requested.

@github-actions
Copy link
Copy Markdown
Contributor

PR approved by anyone and no changes requested.

@CalvinKirs CalvinKirs merged commit 9a58cde into apache:master May 23, 2026
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants