-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
Let's NOT merge this untill we are sure all other SDKs are sending |
Also - this supports redis cache, but will add terraform and env vars in separate PR |
pub enum Addr<'a> { | ||
Combined(&'a str), | ||
Separate { read: &'a str, write: &'a str }, | ||
} |
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.
This feels like an unnecessary abstraction. Why do we need to differentiate between Combined and Separate? Why not pass-through auth_redis_addr_read|write
to Redis::new()
?
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.
Tbh this is just copied from RPC proxy, but if you feel it's unnecessary we can just get rid of it
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.
It's fine to leave as-is for now
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.
Looks good, although I can't vouch for the logic being correct
Co-authored-by: Chris Smith <1979423+chris13524@users.noreply.github.com>
Description
Adding project secret validaiton to all endpoints that should be only usable by authenticated projects.
Resolves #98
How Has This Been Tested?
Due Diligence