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.

Sign up for GitHub

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

Jump to bottom

Add Personal Access Token Support #14

Closed
3 tasks done
knolleary opened this issue Aug 9, 2021 · 5 comments · Fixed by #2584
Closed
3 tasks done

Add Personal Access Token Support #14

knolleary opened this issue Aug 9, 2021 · 5 comments · Fixed by #2584
Assignees
@hardillb
Labels
customer request requested by customer epic A significant feature or piece of work that doesn't easily fit into a single release scope:enterprise Enterprise adoption and roll out features
Milestone
1.11

Comments

@knolleary
Copy link
Member

knolleary commented Aug 9, 2021 •
edited by MarianRaphael
Loading

  • Add database model for Personal Access Tokens (PATs)
  • Define REST API interface to allow user to generate/delete tokens
  • Add auth middleware to handle PAT in place of session cookie
  • Create UI view to list user's tokens
  • Create UI view to create a new token

User Stories
Beta Give feedback
  1. Story: Revoke a personal-access-token #42
    scope:enterprise size:M - 3 story
  2. Story: Generate a personal-access-token #41
    scope:enterprise size:M - 3 story
  3. APIs with SSO Enabled #2482
    size:M - 3 story
@knolleary knolleary added the epic A significant feature or piece of work that doesn't easily fit into a single release label Jan 21, 2022
@sammachin
Copy link
Contributor

Call with customer today expressed an interest in this feature.
They would see it as a reason to take an enterprise license.

@ZJvandeWeg ZJvandeWeg added the scope:enterprise Enterprise adoption and roll out features label Jun 3, 2022
@knolleary knolleary mentioned this issue Nov 28, 2022
Open
@knolleary knolleary mentioned this issue Apr 25, 2023
Closed
@MarianRaphael MarianRaphael mentioned this issue Jul 13, 2023
Closed
@MarianRaphael MarianRaphael added the consideration A potential feature or improvement that is under review for possible development and implementation label Jul 13, 2023
@MarianRaphael MarianRaphael added this to the 1.11 milestone Aug 1, 2023
@MarianRaphael MarianRaphael added customer request requested by customer and removed consideration A potential feature or improvement that is under review for possible development and implementation labels Aug 1, 2023
@hardillb
Copy link
Contributor

hardillb commented Aug 7, 2023

Should PAT have expiry dates or will they be eternal (until revoked)?

Pezmc pushed a commit that referenced this issue Aug 8, 2023
@joepavitt
Merge pull request #14 from flowforge/v0.2
a996e28 
v0.2
@hardillb
Copy link
Contributor

hardillb commented Aug 8, 2023

We shouldn't need a PAT table in the database the AcessToken table should work

@hardillb hardillb self-assigned this Aug 8, 2023
@hardillb
Copy link
Contributor

hardillb commented Aug 8, 2023

Tokens should have names so that a user can have multiple with different scopes and be able to tell them apart

@hardillb hardillb mentioned this issue Aug 14, 2023
Merged
11 tasks
@github-actions github-actions bot mentioned this issue Aug 22, 2023
Merged
11 tasks
@hardillb hardillb mentioned this issue Aug 24, 2023
Merged
10 tasks
@github-actions github-actions bot mentioned this issue Aug 24, 2023
Merged
10 tasks
@hardillb
Copy link
Contributor

Verified on Staging

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hardillb hardillb

Labels
customer request requested by customer epic A significant feature or piece of work that doesn't easily fit into a single release scope:enterprise Enterprise adoption and roll out features
Projects
🛠 Development
Archived in project
Milestone
1.11
Development

Successfully merging a pull request may close this issue.

Personal access tokens FlowFuse/flowfuse
5 participants
@knolleary @sammachin @hardillb @ZJvandeWeg @MarianRaphael