Skip to content

docs: expand security threat model - #2

Closed
zeroshade wants to merge 3 commits into
mainfrom
docs/security-threat-model-review
Closed

docs: expand security threat model#2
zeroshade wants to merge 3 commits into
mainfrom
docs/security-threat-model-review

Conversation

@zeroshade

Copy link
Copy Markdown
Owner

Summary

  • adapt the expanded Iceberg Python threat-model structure to Iceberg Go's actual catalog, REST, configuration, IO, and CLI boundaries
  • preserve Iceberg Go-owned credential isolation across separately constructed catalog/client instances while distinguishing caller-shared state
  • clarify private vulnerability reporting, delegated credentials, endpoint routing, resource limits, table provenance, and destructive external-path behavior
  • make scanner calibration conditional and require human review instead of blanket rejection

Review notes

The Go model intentionally does not copy Python's blanket same-process isolation disclaimer or assume universal table-root containment. Availability-only findings remain hardening by default unless they violate a documented Iceberg Go-owned boundary or demonstrate another security impact.

Verification

  • go test ./... — 28 packages passed; 6 packages had no tests
  • git diff --check origin/main..HEAD
  • final task review and whole-branch security-policy review completed with no remaining Critical or Important findings

@zeroshade
zeroshade force-pushed the docs/security-threat-model-review branch from 24b51ea to 318d82d Compare August 26, 2026 19:39
@zeroshade

Copy link
Copy Markdown
Owner Author

Superseded by apache#1886.

@zeroshade zeroshade closed this Aug 26, 2026
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.

1 participant