one more fix

zvonand · zvonand · commit 9bdb2ccf2763 · 2025-08-03T23:47:05.000Z
diff --git a/src/Access/TokenAccessStorage.cpp b/src/Access/TokenAccessStorage.cpp
@@ -301,7 +301,7 @@ void TokenAccessStorage::assignRolesNoLock(User & user, const std::set<String> &
     if (external_roles.empty())
         roles_per_users.erase(user_name);
     else
-        roles_per_users[user_name] = std::move(external_roles);
+        roles_per_users[user_name] = external_roles;
 
     external_role_hashes[user_name] = external_roles_hash;
 }
diff --git a/src/Access/TokenProcessorsParse.cpp b/src/Access/TokenProcessorsParse.cpp
@@ -46,9 +46,6 @@ std::unique_ptr<DB::ITokenProcessor> ITokenProcessor::parseTokenProcessor(
             }
             else
             {
-                auto userinfo_endpoint = config.getString(prefix + ".userinfo_endpoint");
-                auto token_introspection_endpoint = config.getString(prefix + ".token_introspection_endpoint");
-                auto jwks_uri = config.getString(prefix + ".jwks_uri", "");
                 return std::make_unique<OpenIdTokenProcessor>(processor_name, token_cache_lifetime, username_claim,
                                                               groups_claim,
                                                               config.getString(prefix + ".userinfo_endpoint"),

Original file line number	Diff line number	Diff line change
`@@ -301,7 +301,7 @@ void TokenAccessStorage::assignRolesNoLock(User & user, const std::set<String> &`
`301`	`301`	`if (external_roles.empty())`
`302`	`302`	`roles_per_users.erase(user_name);`
`303`	`303`	`else`
`304`		`- roles_per_users[user_name] = std::move(external_roles);`
	`304`	`+ roles_per_users[user_name] = external_roles;`
`305`	`305`
`306`	`306`	`external_role_hashes[user_name] = external_roles_hash;`
`307`	`307`	`}`
Original file line number	Diff line number	Diff line change
`@@ -46,9 +46,6 @@ std::unique_ptr<DB::ITokenProcessor> ITokenProcessor::parseTokenProcessor(`
`46`	`46`	`}`
`47`	`47`	`else`
`48`	`48`	`{`
`49`		`- auto userinfo_endpoint = config.getString(prefix + ".userinfo_endpoint");`
`50`		`- auto token_introspection_endpoint = config.getString(prefix + ".token_introspection_endpoint");`
`51`		`- auto jwks_uri = config.getString(prefix + ".jwks_uri", "");`
`52`	`49`	`return std::make_unique<OpenIdTokenProcessor>(processor_name, token_cache_lifetime, username_claim,`
`53`	`50`	`groups_claim,`
`54`	`51`	`config.getString(prefix + ".userinfo_endpoint"),`