From a7ff1a41e77bb94465f1d8d7b097ccd410c41528 Mon Sep 17 00:00:00 2001 From: Damian Jarek Date: Sat, 8 Jun 2019 16:21:46 +0200 Subject: [PATCH] Remove redundant instation of `static_string` in websocket Signed-off-by: Damian Jarek --- CHANGELOG.md | 1 + include/boost/beast/websocket/detail/hybi13.ipp | 6 +++--- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/CHANGELOG.md b/CHANGELOG.md index 9b7d14cb..2d1a63aa 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,6 +4,7 @@ Version 259: * Remove the use of `static_string` from `http::fields` * Add gcc-9 to AzP CI test matrix * Enable split compilation in http::basic_fields +* Remove redundant instation of `static_string` in websocket -------------------------------------------------------------------------------- diff --git a/include/boost/beast/websocket/detail/hybi13.ipp b/include/boost/beast/websocket/detail/hybi13.ipp index b0ed5775..8868b540 100644 --- a/include/boost/beast/websocket/detail/hybi13.ipp +++ b/include/boost/beast/websocket/detail/hybi13.ipp @@ -47,11 +47,11 @@ make_sec_ws_accept( string_view key) { BOOST_ASSERT(key.size() <= sec_ws_key_type::max_size_n); - static_string m(key); - m.append("258EAFA5-E914-47DA-95CA-C5AB0DC85B11"); + auto const& guid = "258EAFA5-E914-47DA-95CA-C5AB0DC85B11"; beast::detail::sha1_context ctx; beast::detail::init(ctx); - beast::detail::update(ctx, m.data(), m.size()); + beast::detail::update(ctx, key.data(), key.size()); + beast::detail::update(ctx, guid, sizeof(guid) - 1); char digest[beast::detail::sha1_context::digest_size]; beast::detail::finish(ctx, &digest[0]); accept.resize(accept.max_size());