Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enable self-service ops collections via special case #39

Merged
merged 3 commits into from
Jan 19, 2024

Conversation

jgraettinger
Copy link
Member

This is a medium-term ... hack, which enables users to read the ops logs and stats of their own tasks. When reading the especially-enumerated collections, we bypass traditional prefix authorization and instead authorize over the name partition of the logs or stats collection.

Testing:

  • Verified this enables the new ops view of the entity details page.
  • Verified that slightly perturbing how build up allowed prefixes causes unauthorized errors to be returned.
  • Verified that a locally-modified flowctl is able to list journals and fragments of task logs, as well as read them, as a regular user of a local stack.

This is a medium-term ... hack, which enables users to read the ops logs
and stats of their own tasks. When reading the especially-enumerated
collections, we bypass traditional prefix authorization and instead
authorize over the `name` partition of the logs or stats collection.

Testing:
 * Verified this enables the new ops view of the entity details page.
 * Verified that slightly perturbing how build up allowed prefixes
   causes unauthorized errors to be returned.
 * Verified that a locally-modified `flowctl` is able to list journals
   and fragments of task logs, as well as read them, as a regular user
   of a local stack.
Copy link
Member

@psFried psFried left a comment

Choose a reason for hiding this comment

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

LGTM

@jgraettinger jgraettinger merged commit a9689fc into main Jan 19, 2024
1 check passed
@jgraettinger jgraettinger deleted the johnny/enable-ops branch January 19, 2024 14:40
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