Skip to content
This repository has been archived by the owner on Apr 25, 2022. It is now read-only.

Add endpoint for generating JWTs #148

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jameswestman
Copy link
Contributor

These tokens are provided to flat-manager to authorize the user to download an app. In the future, this enpoint should first check whether the user has paid for the app if necessary.

Copy link
Contributor

@danielsilverstone-ct danielsilverstone-ct left a comment

Choose a reason for hiding this comment

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

I think this is OK, I take it the secret here is just a test secret you were using locally?

With the request I made resolved, I think this could be merged. But the TODO will be critical for that to be OK.

app/main.py Outdated Show resolved Hide resolved
@jameswestman
Copy link
Contributor Author

Added a TODO.

These tokens are provided to flat-manager to authorize the user to
download an app. In the future, this enpoint should first check whether
the user has paid for the app if necessary.
Add a table, UserOwnedApp, to track which users have bought/otherwise
have access to which apps. Currently there is no way to add entries to
this table; that is part of the payment processing work.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants