Skip to content

Commit aa5e843

Browse files
molotkov-andKamil Khamitov
authored and
Kamil Khamitov
committed
Improve error log message in ticket parser (ydb-platform#15020)
1 parent ec8ffac commit aa5e843

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

ydb/core/security/ticket_parser_impl.h

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1784,12 +1784,16 @@ class TTicketParserImpl : public TActorBootstrapped<TDerived> {
17841784
void SetError(const TString& key, TTokenRecord& record, const TEvTicketParser::TError& error) {
17851785
record.Error = error;
17861786
TInstant now = TlsActivationContext->Now();
1787+
TStringBuilder errorLogMessage;
1788+
if (error.HasLogMessage()) {
1789+
errorLogMessage << " (" << error.LogMessage << ")";
1790+
}
17871791
if (record.Error.Retryable) {
17881792
record.ExpireTime = GetExpireTime(record, now);
17891793
record.SetErrorRefreshTime(this, now);
17901794
CounterTicketsErrorsRetryable->Inc();
17911795
BLOG_D("Ticket " << record.GetMaskedTicket() << " ("
1792-
<< record.PeerName << ") has now retryable error message '" << error.Message << "'");
1796+
<< record.PeerName << ") has now retryable error message '" << error.Message << errorLogMessage << "'");
17931797
if (record.RefreshRetryableErrorImmediately) {
17941798
record.RefreshRetryableErrorImmediately = false;
17951799
GetDerived()->CanRefreshTicket(key, record);
@@ -1802,7 +1806,7 @@ class TTicketParserImpl : public TActorBootstrapped<TDerived> {
18021806
record.SetOkRefreshTime(this, now);
18031807
CounterTicketsErrorsPermanent->Inc();
18041808
BLOG_D("Ticket " << record.GetMaskedTicket() << " ("
1805-
<< record.PeerName << ") has now permanent error message '" << error.Message << "'");
1809+
<< record.PeerName << ") has now permanent error message '" << error.Message << errorLogMessage << "'");
18061810
}
18071811
CounterTicketsErrors->Inc();
18081812
record.IsLowAccessServiceRequestPriority = true;

0 commit comments

Comments
 (0)