Conversation
itepastra
left a comment
itepastra
left a comment
There was a problem hiding this comment.
I made some guidance comments
There was a problem hiding this comment.
just for me to make reqwest working, don't worry
There was a problem hiding this comment.
okay, this one you may care about. It's close to a copy from grid tho
There was a problem hiding this comment.
this url should probably be changed or be like cli configurable
| x, | ||
| y, | ||
| c, | ||
| #[cfg(feature = "auth")] |
There was a problem hiding this comment.
this is annoying but needed to have the tests work with --features auth as well
|
|
||
| pub async fn process_socket(&mut self) -> io::Result<()> { | ||
| // Handle the auth flow | ||
| #[cfg(feature = "auth")] |
There was a problem hiding this comment.
This should be redone at some point, or at least depend on the other side
There was a problem hiding this comment.
all relatively trivial changes
There was a problem hiding this comment.
save 2 files each loop, the state and the blame map
1 participant
Adds a start for an OAUTH based authentication flow, there is a feature flag to enable it.