Skip to content

Conversation

@pburkholder
Copy link
Contributor

Changes proposed in this pull request:

  • Unauthenticated users should have the marketplace hidden.

security considerations

Improves security. Currently unauthenticated users can view stuff

Unauthenticated users should have the marketplace hidden.

Signed-off-by: Peter Burkholder <[email protected]>
@pburkholder pburkholder requested a review from a team as a code owner February 3, 2025 22:54
@pburkholder
Copy link
Contributor Author

Never mind. Closing. We'll need this so we can have our plans be self documenting, instead of manually copied over to the site.

@pburkholder pburkholder closed this Feb 3, 2025
@jameshochadel
Copy link
Contributor

Did you have a particular threat model in mind here? Once we port all offerings to the CSB, we'll have an auto-generated docs page for them that's separate from CAPI. Just curious what you're considering.

@pburkholder pburkholder deleted the peterb/hide-marketplace branch February 4, 2025 14:25
@pburkholder
Copy link
Contributor Author

I agree 100%, that's why, after a moment's further consideration, I had already closed this PR

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