Skip to content

ARGO-5587 Support component admins to refresh tokens on component acc…#532

Merged
agelostsal merged 1 commit into
develfrom
component-accounts
May 27, 2026
Merged

ARGO-5587 Support component admins to refresh tokens on component acc…#532
agelostsal merged 1 commit into
develfrom
component-accounts

Conversation

@kaggis

@kaggis kaggis commented May 27, 2026

Copy link
Copy Markdown
Contributor

…ounts

Goal

Allow users with the service-wide role "component_admin" to be able to refresh access for integrated components under specific projects

exampe user story:
As a component admin i want to refresh access for my component monbox which integrates with AMS under project FOO

Implementation

The main request that allows a component admin to refresh access for a specific component e.g. monbox under a specific project FOO is the following:

HTTP POST /integrations/components/monbox/by-project-name/FOO/refresh

and the successful response is:

{
    "status": {
        "message": "component api key succesfully renewed",
        "code": "200"
    },
    "data": {
        "api_key": "..."
    }
}

Implementation Steps

  • Update User models to have two extra optional fields: component which holds the component name attached to this user account and component_project project under which the specific component operates under
  • Update all user management method signatures to support the new optional fields
  • Update interfaces (such as auth and store) to use the optional fields
  • Update old unit tests to handle the new fields
  • Add new handler that implements the HTTP POST /integrations/components/{component}/by-project-name/{project}/refresh API call under which a component_admin can refresh access
  • Update documentation
  • Update openapi definitions

Misc

  • Minor clean up of forgotten debug print statements

@kaggis kaggis force-pushed the component-accounts branch from 6cf8601 to 7e9c805 Compare May 27, 2026 11:33
@kaggis kaggis force-pushed the component-accounts branch from 7e9c805 to cead18f Compare May 27, 2026 11:41
@kaggis kaggis requested a review from agelostsal May 27, 2026 11:49
@agelostsal agelostsal merged commit 96a1d60 into devel May 27, 2026
2 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.

2 participants