Skip to content

docs: add multi-tenancy considerations to README#846

Merged
toddbaert merged 1 commit into
mainfrom
docs/multi-tenancy-considerations
Jun 4, 2026
Merged

docs: add multi-tenancy considerations to README#846
toddbaert merged 1 commit into
mainfrom
docs/multi-tenancy-considerations

Conversation

@toddbaert
Copy link
Copy Markdown
Member

Adds an IMPORTANT admonition signposting GHSA-398h-7f66-3h4p and the implicit cross-namespace FeatureFlagSource / InProcessConfiguration resolution behavior. Links to the advisory for the full threat model and to #795 for per-tenant operator install.

@toddbaert toddbaert force-pushed the docs/multi-tenancy-considerations branch from fe545cb to 24adb33 Compare June 4, 2026 19:23
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request adds a 'Multi-tenancy considerations' section to the README.md file to document security implications and threat models regarding cross-namespace resolution. The review feedback suggests spelling out 'The OpenFeature Operator' instead of using the abbreviation 'OFO' and prefixing a relative link with './' to improve readability and maintain consistency with other links in the document.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread README.md Outdated
Adds an IMPORTANT admonition signposting GHSA-398h-7f66-3h4p and the
implicit cross-namespace FeatureFlagSource / InProcessConfiguration
resolution behavior. Links to the advisory for the full threat model
and to #795 for per-tenant operator install.

Signed-off-by: Todd Baert <todd.baert@dynatrace.com>
@toddbaert toddbaert force-pushed the docs/multi-tenancy-considerations branch from 24adb33 to a7afe5d Compare June 4, 2026 19:26
Copy link
Copy Markdown
Member

@lukas-reining lukas-reining left a comment

Choose a reason for hiding this comment

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

Yeah this makes sense.

@toddbaert toddbaert added this pull request to the merge queue Jun 4, 2026
Merged via the queue into main with commit 32b3b65 Jun 4, 2026
13 checks passed
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.

3 participants