Skip to content

Commit 982180d

Browse files
GazizonokiGazizonoki
committed
Remove grpc dependency leak through iam headers
1 parent 07e05c9 commit 982180d

File tree

12 files changed

+191
-202
lines changed

12 files changed

+191
-202
lines changed

‎include/ydb-cpp-sdk/client/iam/iam.h‎

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,18 @@
11
#pragma once
22

3-
#include "common/iam.h"
3+
#include "types.h"
4+
namespace NYdb {
5+
6+
/// Acquire an IAM token using a local metadata service on a virtual machine.
7+
TCredentialsProviderFactoryPtr CreateIamCredentialsProviderFactory(const TIamHost& params = {});
8+
9+
/// Acquire an IAM token using a JSON Web Token (JWT) file name.
10+
TCredentialsProviderFactoryPtr CreateIamJwtFileCredentialsProviderFactory(const TIamJwtFilename& params);
11+
12+
/// Acquire an IAM token using JSON Web Token (JWT) contents.
13+
TCredentialsProviderFactoryPtr CreateIamJwtParamsCredentialsProviderFactory(const TIamJwtContent& param);
14+
15+
// Acquire an IAM token using a user OAuth token.
16+
TCredentialsProviderFactoryPtr CreateIamOAuthCredentialsProviderFactory(const TIamOAuth& params);
17+
18+
}

‎include/ydb-cpp-sdk/client/iam/types.h‎

Lines changed: 53 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,53 @@
1+
#pragma once
2+
3+
#include <ydb-cpp-sdk/client/types/credentials/credentials.h>
4+
#include <ydb-cpp-sdk/library/jwt/jwt.h>
5+
6+
#include <util/datetime/base.h>
7+
8+
#include <fstream>
9+
#include <string>
10+
11+
namespace NYdb {
12+
13+
namespace NIam {
14+
15+
constexpr std::string_view DEFAULT_ENDPOINT = "iam.api.cloud.yandex.net";
16+
constexpr bool DEFAULT_ENABLE_SSL = true;
17+
18+
constexpr std::string_view DEFAULT_HOST = "169.254.169.254";
19+
constexpr uint32_t DEFAULT_PORT = 80;
20+
21+
constexpr TDuration DEFAULT_REFRESH_PERIOD = TDuration::Hours(1);
22+
constexpr TDuration DEFAULT_REQUEST_TIMEOUT = TDuration::Seconds(10);
23+
24+
}
25+
26+
struct TIamHost {
27+
std::string Host = std::string(NIam::DEFAULT_HOST);
28+
uint32_t Port = NIam::DEFAULT_PORT;
29+
TDuration RefreshPeriod = NIam::DEFAULT_REFRESH_PERIOD;
30+
};
31+
32+
struct TIamEndpoint {
33+
std::string Endpoint = std::string(NIam::DEFAULT_ENDPOINT);
34+
TDuration RefreshPeriod = NIam::DEFAULT_REFRESH_PERIOD;
35+
TDuration RequestTimeout = NIam::DEFAULT_REQUEST_TIMEOUT;
36+
bool EnableSsl = NIam::DEFAULT_ENABLE_SSL;
37+
};
38+
39+
struct TIamJwtFilename : TIamEndpoint { std::string JwtFilename; };
40+
41+
struct TIamJwtContent : TIamEndpoint { std::string JwtContent; };
42+
43+
struct TIamJwtParams : TIamEndpoint { TJwtParams JwtParams; };
44+
45+
struct TIamOAuth : TIamEndpoint { std::string OAuthToken; };
46+
47+
48+
inline TJwtParams ReadJwtKeyFile(const std::string& filename) {
49+
std::ifstream input(filename, std::ios::in);
50+
return ParseJwtParams({std::istreambuf_iterator<char>(input), std::istreambuf_iterator<char>()});
51+
}
52+
53+
}

‎include/ydb-cpp-sdk/client/iam_private/iam.h‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#pragma once
22

3-
#include <ydb-cpp-sdk/client/iam/common/iam.h>
3+
#include <ydb-cpp-sdk/client/iam/types.h>
44

55
namespace NYdb {
66

‎src/client/helpers/CMakeLists.txt‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ target_link_libraries(client-helpers
44
PUBLIC
55
yutil
66
client-ydb_types-credentials-oauth2
7-
client-iam-common
7+
client-iam
88
client-ydb_types-credentials
99
)
1010

‎src/client/helpers/helpers.cpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#include <ydb-cpp-sdk/client/helpers/helpers.h>
22

3-
#include <ydb-cpp-sdk/client/iam/common/iam.h>
3+
#include <ydb-cpp-sdk/client/iam/iam.h>
44
#include <ydb-cpp-sdk/client/resources/ydb_ca.h>
55
#include <ydb-cpp-sdk/client/types/credentials/oauth2_token_exchange/from_file.h>
66

‎src/client/iam/CMakeLists.txt‎

Lines changed: 12 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,20 @@ add_subdirectory(common)
22

33
_ydb_sdk_add_library(client-iam)
44

5-
target_link_libraries(client-iam PUBLIC
6-
yutil
7-
api-client-yc_public
8-
client-iam-common
5+
target_link_libraries(client-iam
6+
PUBLIC
7+
client-ydb_types-credentials
8+
http-simple
9+
json
10+
library-jwt
11+
yutil
12+
PRIVATE
13+
api-client-yc_public
14+
client-iam-common
915
)
1016

11-
_ydb_sdk_install_targets(TARGETS client-iam)
12-
1317
target_sources(client-iam PRIVATE
1418
iam.cpp
1519
)
20+
21+
_ydb_sdk_make_client_component(Iam client-iam)

‎src/client/iam/common/CMakeLists.txt‎

Lines changed: 9 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,12 @@
1-
_ydb_sdk_add_library(client-iam-common)
1+
_ydb_sdk_add_library(client-iam-common INTERFACE)
22

3-
target_link_libraries(client-iam-common PUBLIC
4-
yutil
5-
grpc-client
6-
http-simple
7-
json
8-
library-jwt
9-
client-ydb_types-credentials
10-
api-client-yc_public
3+
target_link_libraries(client-iam-common
4+
INTERFACE
5+
client-ydb_types-credentials
6+
grpc-client
7+
library-jwt
8+
threading-future
9+
yutil
1110
)
1211

13-
target_sources(client-iam-common PRIVATE
14-
iam.cpp
15-
)
16-
17-
_ydb_sdk_make_client_component(Iam client-iam-common)
12+
_ydb_sdk_install_targets(client-iam-common)

‎src/client/iam/common/iam.cpp‎

Lines changed: 0 additions & 114 deletions
This file was deleted.

‎include/ydb-cpp-sdk/client/iam/common/iam.h‎ renamed to ‎src/client/iam/common/iam.h‎

Lines changed: 3 additions & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -1,69 +1,16 @@
11
#pragma once
22

3+
#include <ydb-cpp-sdk/client/iam/types.h>
34
#include <ydb-cpp-sdk/client/types/credentials/credentials.h>
4-
55
#include <ydb-cpp-sdk/library/grpc/client/grpc_client_low.h>
6-
#include <library/cpp/threading/future/future.h>
76

8-
#include <ydb-cpp-sdk/library/jwt/jwt.h>
9-
#include <util/datetime/base.h>
7+
#include <library/cpp/threading/future/future.h>
108

11-
#include <util/system/spinlock.h>
129
#include <util/string/builder.h>
13-
14-
#include <fstream>
10+
#include <util/system/spinlock.h>
1511

1612
namespace NYdb {
1713

18-
namespace NIam {
19-
constexpr std::string_view DEFAULT_ENDPOINT = "iam.api.cloud.yandex.net";
20-
constexpr bool DEFAULT_ENABLE_SSL = true;
21-
22-
constexpr std::string_view DEFAULT_HOST = "169.254.169.254";
23-
constexpr uint32_t DEFAULT_PORT = 80;
24-
25-
constexpr TDuration DEFAULT_REFRESH_PERIOD = TDuration::Hours(1);
26-
constexpr TDuration DEFAULT_REQUEST_TIMEOUT = TDuration::Seconds(10);
27-
}
28-
29-
struct TIamHost {
30-
std::string Host = std::string(NIam::DEFAULT_HOST);
31-
uint32_t Port = NIam::DEFAULT_PORT;
32-
TDuration RefreshPeriod = NIam::DEFAULT_REFRESH_PERIOD;
33-
};
34-
35-
struct TIamEndpoint {
36-
std::string Endpoint = std::string(NIam::DEFAULT_ENDPOINT);
37-
TDuration RefreshPeriod = NIam::DEFAULT_REFRESH_PERIOD;
38-
TDuration RequestTimeout = NIam::DEFAULT_REQUEST_TIMEOUT;
39-
bool EnableSsl = NIam::DEFAULT_ENABLE_SSL;
40-
};
41-
42-
struct TIamJwtFilename : TIamEndpoint { std::string JwtFilename; };
43-
44-
struct TIamJwtContent : TIamEndpoint { std::string JwtContent; };
45-
46-
struct TIamJwtParams : TIamEndpoint { TJwtParams JwtParams; };
47-
48-
inline TJwtParams ReadJwtKeyFile(const std::string& filename) {
49-
std::ifstream input(filename, std::ios::in);
50-
return ParseJwtParams({std::istreambuf_iterator<char>(input), std::istreambuf_iterator<char>()});
51-
}
52-
53-
struct TIamOAuth : TIamEndpoint { std::string OAuthToken; };
54-
55-
/// Acquire an IAM token using a local metadata service on a virtual machine.
56-
TCredentialsProviderFactoryPtr CreateIamCredentialsProviderFactory(const TIamHost& params = {});
57-
58-
/// Acquire an IAM token using a JSON Web Token (JWT) file name.
59-
TCredentialsProviderFactoryPtr CreateIamJwtFileCredentialsProviderFactory(const TIamJwtFilename& params);
60-
61-
/// Acquire an IAM token using JSON Web Token (JWT) contents.
62-
TCredentialsProviderFactoryPtr CreateIamJwtParamsCredentialsProviderFactory(const TIamJwtContent& param);
63-
64-
// Acquire an IAM token using a user OAuth token.
65-
TCredentialsProviderFactoryPtr CreateIamOAuthCredentialsProviderFactory(const TIamOAuth& params);
66-
6714
constexpr TDuration BACKOFF_START = TDuration::MilliSeconds(50);
6815
constexpr TDuration BACKOFF_MAX = TDuration::Seconds(10);
6916

0 commit comments

Comments
 (0)