Skip to content

Feature: Move Callouts from Auth Service to pkg/authz #2804

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

Open
arvindh123 opened this issue Apr 9, 2025 · 0 comments · May be fixed by #2823
Open

Feature: Move Callouts from Auth Service to pkg/authz #2804

arvindh123 opened this issue Apr 9, 2025 · 0 comments · May be fixed by #2823
Assignees

Comments

@arvindh123
Copy link
Contributor

Is your feature request related to a problem? Please describe.

Currently, callouts to external services are made from the auth service, specifically within the Authorize method in auth/service.go.

Problem:

  • These callouts are constrained due to limited context available in the auth service.
  • For example, callout service needs operation-specific context (e.g., create, update, add users, send invitation).
  • In the future, additional callouts might require even more request details.

Describe the feature you are requesting, as well as the possible use case(s) for it.

Proposal:

  • Move the callout logic from the auth service to pkg/authz.
  • pkg/authz has access to richer context about the request, enabling it to send more complete information to external services.

Indicate the importance of this feature to you.

Must-have

Anything else?

Benefits:

  • Centralizes and enriches the context for callouts.
  • Allows flexible and extensible constraint checks.
  • Decouples callout logic from the auth layer, making it easier to maintain and extend.
@github-project-automation github-project-automation bot moved this to ⛏ Backlog in SuperMQ Apr 9, 2025
@arvindh123 arvindh123 moved this from ⛏ Backlog to 🚧 In Progress in SuperMQ Apr 9, 2025
@nyagamunene nyagamunene moved this from 🚧 In Progress to 🩺 Review and testing in SuperMQ Apr 11, 2025
@nyagamunene nyagamunene moved this from 🩺 Review and testing to 🚧 In Progress in SuperMQ Apr 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 🚧 In Progress
Development

Successfully merging a pull request may close this issue.

2 participants