(feat/samples): Add AMD token generation sample #27
Conversation
This commit adds a sample that showcases token generation for Adaptive Media Delivery. Signed-off-by: Radu Matei <[email protected]>
| version = "0.1.0" | ||
|
|
||
| [variables] | ||
| encryption_key = { required = true } |
There was a problem hiding this comment.
should this be a secret?
There was a problem hiding this comment.
Note that secret = true doesn't actually do anything today. It was meant to allow partial debug logging of variable values but instead we just never log any. 🤷
There was a problem hiding this comment.
I suppose optimistically some external tooling could take it into account 😅
| const encoder = new TextEncoder(); | ||
| const data = encoder.encode(message); | ||
|
|
||
| // Use crypto.subtle.sign instead of crypto.hmac |
There was a problem hiding this comment.
I think it'd be good to expand on the comment here a bit, and explain why this is a valid thing to do
Signed-off-by: Radu Matei <[email protected]>
Signed-off-by: Radu Matei <[email protected]>
| } | ||
|
|
||
| // Handle token generation | ||
| if path == "/rustapi/token" { |
There was a problem hiding this comment.
I would tend to invert this check and return early with the 404, so the rest of the code can be unindented a level—but certainly not blocking at all.
|
@radu-matei can this merge or are you still working on it? |
This commit adds a sample that showcases token generation for Adaptive Media Delivery.