Skip to content

Commit 62843c1

Browse files
EXT-1557 audit asan fix (#26156)
1 parent f4d5afd commit 62843c1

File tree

1 file changed

+10
-13
lines changed

1 file changed

+10
-13
lines changed

ydb/core/mon/audit/audit_ut.cpp

Lines changed: 10 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -7,19 +7,16 @@ using namespace NMonitoring::NAudit;
77

88
namespace {
99

10-
struct TRequestHolder : public NHttp::THttpIncomingRequest {
11-
TStringBuf Store(TString value) {
12-
return Storage.emplace_back(std::move(value));
13-
}
14-
private:
15-
TVector<TString> Storage;
16-
};
17-
1810
NHttp::THttpIncomingRequestPtr MakeRequest(TString method, TString url) {
19-
auto request = MakeIntrusive<TRequestHolder>();
20-
request->Method = request->Store(std::move(method));
21-
request->URL = request->Store(std::move(url));
22-
return std::move(request);
11+
static TVector<TString> Storage;
12+
13+
auto request = MakeIntrusive<NHttp::THttpIncomingRequest>();
14+
Storage.emplace_back(std::move(method));
15+
request->Method = Storage.back();
16+
Storage.emplace_back(std::move(url));
17+
request->URL = Storage.back();
18+
19+
return request;
2320
}
2421

2522
} // namespace
@@ -45,7 +42,7 @@ Y_UNIT_TEST_SUITE(TAuditTest) {
4542
UNIT_ASSERT(!ctx.AuditableRequest(MakeRequest("OPTIONS", "/path")));
4643
}
4744

48-
Y_UNIT_TEST(BlacklistedPathsAreNotAudited) {
45+
Y_UNIT_TEST(DeniedPathsAreNotAudited) {
4946
TAuditCtx ctx;
5047
UNIT_ASSERT(!ctx.AuditableRequest(MakeRequest("GET", "/counters")));
5148
UNIT_ASSERT(!ctx.AuditableRequest(MakeRequest("GET", "/viewer/subpage")));

0 commit comments

Comments
 (0)