Skip to content
This repository has been archived by the owner on Sep 21, 2024. It is now read-only.

Implement typed header parsing for ucan header #68

Closed
cdata opened this issue Sep 30, 2022 · 2 comments
Closed

Implement typed header parsing for ucan header #68

cdata opened this issue Sep 30, 2022 · 2 comments
Labels
Enhancement New user-facing feature Good First Issue Good for newcomers

Comments

@cdata
Copy link
Collaborator

cdata commented Sep 30, 2022

After we integrate support for changes to rs-ucan landing in ucan-wg/rs-ucan#28, we will have to contend with a new world where HTTP request headers that use UCAN-based JWT for authorization may also include auxilliary ucan headers (see: https://github.com/ucan-wg/ucan-as-bearer-token#2-request-headers).

We already benefit from typed header parsing for the authorization header. We should author an extension to the typed headers that are offered by axum to include something to make parsing ucan headers more convenient and consistent.

@blaine
Copy link

blaine commented Aug 5, 2023

If anyone looks to pick this up, we have [almost] an implementation in fission-server that will be extracted to a generic axum extractor. Will update as we get closer. It probably makes sense to implement this as a separate crate from rs-ucan, but appreciate any thoughts on the matter!

ucan-wg/rs-ucan#122

@jsantell
Copy link
Contributor

@github-project-automation github-project-automation bot moved this from 🏔️ Icebox to 🌱 Done in Subconscious + Noosphere Feb 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Enhancement New user-facing feature Good First Issue Good for newcomers
Projects
Status: 🌱 Done
Development

No branches or pull requests

3 participants