From 49c1928dd8000db78dee82ab203d4259400a73d1 Mon Sep 17 00:00:00 2001 From: ermolovd Date: Fri, 11 Apr 2025 10:09:05 +0300 Subject: TClientOptions::FromUser: do not set usertag equal to username By default missing usertag equals username commit_hash:7296fbc797f3877c6f1460adea4759299a6758bd --- yt/yt/client/api/options.cpp | 4 ++-- yt/yt/client/api/options.h | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/yt/yt/client/api/options.cpp b/yt/yt/client/api/options.cpp index 03c7683160c..14784589c27 100644 --- a/yt/yt/client/api/options.cpp +++ b/yt/yt/client/api/options.cpp @@ -25,11 +25,11 @@ TClientOptions TClientOptions::FromUserAndToken(std::string user, std::string to return options; } -TClientOptions TClientOptions::FromUser(std::string user, const std::optional& userTag) +TClientOptions TClientOptions::FromUser(std::string user, std::optional userTag) { TClientOptions options; options.User = std::move(user); - options.UserTag = userTag.value_or(user); + options.UserTag = std::move(userTag); return options; } diff --git a/yt/yt/client/api/options.h b/yt/yt/client/api/options.h index 2cca2a093c8..b22de69f256 100644 --- a/yt/yt/client/api/options.h +++ b/yt/yt/client/api/options.h @@ -28,7 +28,7 @@ struct TClientOptions * Such options works in internal YT code and can work in tests. * Production clusters reject such options. */ - static TClientOptions FromUser(std::string user, const std::optional& userTag = {}); + static TClientOptions FromUser(std::string user, std::optional userTag = {}); //! Creates options for authenticating as root. /*! -- cgit v1.3