-
Notifications
You must be signed in to change notification settings - Fork 44
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: support verify query session info before submit query session #162
Conversation
Ok(id) => { | ||
println!("Close Session {}", id); | ||
Ok((sess_info_node, sess_info_client)) => { | ||
println!("Close Session Successfully!\nNode session {:?}\nClient session: {:?}", |
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.
Do not show the debug information to a normal user, you can add a flag or verbose to show it or not
@@ -184,8 +185,9 @@ pub async fn process_cmd( | |||
.close_session(session.as_ref().unwrap().session_id) | |||
.await | |||
{ | |||
Ok(id) => { | |||
println!("Close Session {}", id); | |||
Ok((sess_info_node, sess_info_client)) => { |
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.
mark
Stop = 2; | ||
} | ||
message QuerySessionInfo { | ||
// the hex encoded string |
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.
a wrong comment?
@@ -97,21 +109,11 @@ message CloseSessionRequest { | |||
bytes signature = 2; | |||
} | |||
message CloseSessionResponse { | |||
int32 session_id = 1; | |||
QuerySessionInfo query_session_info = 1; |
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.
The type of QuerySessionInfo
should be bytes
close since the pr was merged by #163 |
This PR resolve #127