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

Short circuit UCAN validation #184

Open
expede opened this issue Sep 28, 2023 · 1 comment
Open

Short circuit UCAN validation #184

expede opened this issue Sep 28, 2023 · 1 comment

Comments

@expede
Copy link
Member

expede commented Sep 28, 2023

ETA: 2023-10-31

Use the Capability store to skip validation of UCANs that the server already knows about. This likely requires some integration with rs-ucan (e.g. exposing new traits, etc), but not sure at time of writing.

cc @QuinnWilton thoughts? Here's an example schema of how we'd cache this in a DB.

@matheus23
Copy link
Member

We don't do this yet.
So far I'm avoiding DB requests in the auth checking code. The clients have to come with all of the UCANs needed to verify the request.

So essentially I'm postponing this until it becomes an issue.

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

No branches or pull requests

2 participants