tursom/memgraph
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix CPU 100% usage by websocket error handling improvement (#1327)

Browse Source
This commit is contained in:
Marko Budiselić 2023-10-16 15:41:12 +02:00 committed by GitHub
parent 22d8ef75e0
commit fd10d1c9f8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/communication/websocket/session.cpp
View File

@ -1,4 +1,4 @@
// Copyright 2022 Memgraph Ltd. // Copyright 2023 Memgraph Ltd.
// //
// Use of this software is governed by the Business Source License // Use of this software is governed by the Business Source License
// included in the file licenses/BSL.txt; by using this file, you agree to be bound by the terms of the Business Source // included in the file licenses/BSL.txt; by using this file, you agree to be bound by the terms of the Business Source
@ -175,6 +175,11 @@ void Session::OnRead(const boost::beast::error_code ec, const size_t /*bytes_tra
return; return;
} }
if (ec) {
LogError(ec, "read");
return;
}
if (!IsAuthenticated()) { if (!IsAuthenticated()) {
auto response = nlohmann::json(); auto response = nlohmann::json();
auto auth_failed = [this, &response](const std::string &message) { auto auth_failed = [this, &response](const std::string &message) {

2
src/communication/websocket/session.hpp
View File

@ -1,4 +1,4 @@
// Copyright 2022 Memgraph Ltd. // Copyright 2023 Memgraph Ltd.
// //
// Use of this software is governed by the Business Source License // Use of this software is governed by the Business Source License
// included in the file licenses/BSL.txt; by using this file, you agree to be bound by the terms of the Business Source // included in the file licenses/BSL.txt; by using this file, you agree to be bound by the terms of the Business Source