-
Notifications
You must be signed in to change notification settings - Fork 14
Conversation
Simplify resource access on Kepler by reducing the endpoints to `/invoke` and `/delegate` only, and parsing the request from the invocation.
9972680
to
40c9ea7
Compare
3ad3081
to
d6160d2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+523 -993
mmm that's good
looks about fine I think, we can reorganise the create stuff soon. I will make an issue somewhere for as "authorised http gateway" for recreating a restful interface "frontend" as a separate program
impl_fromreq!(DelAuthWrapper, "del"); | ||
impl_fromreq!(ListAuthWrapper, "list"); | ||
pub enum InvokeAuthWrapper { | ||
Create(OrbitId), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO this should be handled by DelegateAuthWrapper
, but I understand this maintains the current behaviour so fair enough
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah agreed. I had a go at converting it to a delegation, but I couldn't get it working. I'll put an issue in for this.
Simplify resource access on Kepler by reducing the endpoints to
/invoke
and/delegate
only, and parsing the request from the invocation.This does not support invocation by only referencing a pre-submitted delegation, because that is hard and it looks like we will require the delegation going forwards anyway.