More detailed error response on authentication/registration failure (#356)

jstuczyn · web-flow · commit 9a61c4ae1f52 · 2020-09-28T15:20:19.000+01:00
diff --git a/gateway/src/node/client_handling/websocket/connection_handler.rs b/gateway/src/node/client_handling/websocket/connection_handler.rs
@@ -283,7 +283,7 @@ impl<S> Handle<S> {
             ClientsHandlerResponse::Error(e) => {
                 error!("Authentication unexpectedly failed - {}", e);
                 ServerResponse::Error {
-                    message: "unexpected failure".into(),
+                    message: format!("Authentication failure - {}", e),
                 }
             }
             _ => panic!("received response to wrong query!"), // this should NEVER happen
@@ -349,7 +349,7 @@ impl<S> Handle<S> {
             ClientsHandlerResponse::Error(e) => {
                 error!("Post-handshake registration unexpectedly failed - {}", e);
                 ServerResponse::Error {
-                    message: "unexpected failure".into(),
+                    message: format!("Registration failure - {}", e),
                 }
             }
             _ => panic!("received response to wrong query!"), // this should NEVER happen

Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,7 @@ impl<S> Handle<S> {`
`283`	`283`	`ClientsHandlerResponse::Error(e) => {`
`284`	`284`	`error!("Authentication unexpectedly failed - {}", e);`
`285`	`285`	`ServerResponse::Error {`
`286`		`- message: "unexpected failure".into(),`
	`286`	`+ message: format!("Authentication failure - {}", e),`
`287`	`287`	`}`
`288`	`288`	`}`
`289`	`289`	`_ => panic!("received response to wrong query!"), // this should NEVER happen`
`@@ -349,7 +349,7 @@ impl<S> Handle<S> {`
`349`	`349`	`ClientsHandlerResponse::Error(e) => {`
`350`	`350`	`error!("Post-handshake registration unexpectedly failed - {}", e);`
`351`	`351`	`ServerResponse::Error {`
`352`		`- message: "unexpected failure".into(),`
	`352`	`+ message: format!("Registration failure - {}", e),`
`353`	`353`	`}`
`354`	`354`	`}`
`355`	`355`	`_ => panic!("received response to wrong query!"), // this should NEVER happen`