Skip to content

add security notes for stellar-core v28.0.0 release - #5442

Draft
anupsdf wants to merge 1 commit into
stellar:masterfrom
anupsdf:security-notes-28.0.1
Draft

add security notes for stellar-core v28.0.0 release#5442
anupsdf wants to merge 1 commit into
stellar:masterfrom
anupsdf:security-notes-28.0.1

Conversation

@anupsdf

@anupsdf anupsdf commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

what

Add security notes for stellar-core v28.0.0 release.

Copilot AI balanced review requested due to automatic review settings September 1, 2026 21:37

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds security notes for six fixes shipped in Stellar Core v28.0.0 and v28.0.1.

Changes:

  • Documents Overlay resource-consumption protections.
  • Records Ledger validation safeguards.
  • Notes Herder fee-overflow hardening.
Suppressed comments (1)

docs/software/security-protocol-release-notes.md:73

  • This also discloses v28.0.0 security issues before the document's required 30-day delay (docs/software/security-protocol-release-notes.md:39). The release was published on 2026-08-13, so this section should not be merged before 2026-09-12.
## v28.0.0 (2026-08-13)

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread docs/software/security-protocol-release-notes.md Outdated
Copilot AI review requested due to automatic review settings September 1, 2026 21:42
@anupsdf anupsdf changed the title add security notes for stellar-core v28.0.0 and v28.0.1 releases add security notes for stellar-core v28.0.0 release Sep 1, 2026
@anupsdf
anupsdf force-pushed the security-notes-28.0.1 branch from b6043f9 to b06f247 Compare September 1, 2026 21:44

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread docs/software/security-protocol-release-notes.md
Copilot AI review requested due to automatic review settings September 1, 2026 21:44
@anupsdf
anupsdf force-pushed the security-notes-28.0.1 branch from b06f247 to 9166b6b Compare September 1, 2026 21:46

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Suppressed comments (1)

docs/software/security-protocol-release-notes.md:62

  • This section discloses security issues before the document's 30-day waiting period has elapsed. Line 39 requires waiting 30 days after the fixing release; for the 2026-08-13 release, that is 2026-09-12, while this review is dated 2026-09-01. Please delay merging this section until September 12 or remove it for now.
## v28.0.0 (2026-08-13)

Copilot AI review requested due to automatic review settings September 1, 2026 21:47
@anupsdf
anupsdf enabled auto-merge September 1, 2026 21:47
@anupsdf
anupsdf marked this pull request as draft September 1, 2026 21:48
auto-merge was automatically disabled September 1, 2026 21:48

Pull request was converted to draft

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

# List of releases

## v28.0.0 (2026-08-13)
* `Overlay` - security - Strengthen flow control and peer request processing limits to prevent excessive resource consumption.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

referring to #5362 and #5381 cc. @graydon

* `Overlay` - security - Strengthen flow control and peer request processing limits to prevent excessive resource consumption.
* exploited: no
* mitigation: code fix
* `Ledger` - security - Enforce transaction signature verification limits throughout transaction validation.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

referring to #5393, cc. @bboston7

* `Ledger` - security - Enforce transaction signature verification limits throughout transaction validation.
* exploited: no
* mitigation: code fix
* `Ledger` - security - Ensure consistent validation of transaction XDR across protocol versions.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

referring to #5396 cc. @dmkozh

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