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

feat: Issue session token for usage with API #1710

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

MoritzWeber0
Copy link
Member

@MoritzWeber0 MoritzWeber0 commented Aug 16, 2024

Some sessions may want to automate tasks against the CCM API. A session token is issued as PAT and is valid for the duration of the session.

The session token should only have access to read-operations, not write-operations. Therefore, it's required to implement fine-grained tokens before this PR can be merged.

Resolves #1774.

Some sessions may want to automate tasks against the CCM API.
A session token is issued as PAT and is valid for the duration of the session.
Copy link

github-actions bot commented Aug 16, 2024

A Storybook preview is available for commit 8d3f4e2.
View Storybook
View Chromatic build

Copy link

sonarcloud bot commented Aug 16, 2024

Copy link

codecov bot commented Aug 16, 2024

Codecov Report

Attention: Patch coverage is 56.52174% with 10 lines in your changes missing coverage. Please review.

Project coverage is 83.85%. Comparing base (4906bab) to head (6528b9e).
Report is 1 commits behind head on main.

Files Patch % Lines
...kend/capellacollab/sessions/hooks/session_token.py 47.36% 10 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1710      +/-   ##
==========================================
- Coverage   83.96%   83.85%   -0.11%     
==========================================
  Files         185      186       +1     
  Lines        6105     6126      +21     
  Branches      674      676       +2     
==========================================
+ Hits         5126     5137      +11     
- Misses        831      841      +10     
  Partials      148      148              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

Issue session token for usage with API
1 participant