Impact
Since ydb-go-sdk/v3.48.6 if you use a custom credentials object (implementation of interface Credentials) it may leak into logs. This happens because this object could be serialized into an error message using fmt.Errorf("something went wrong (credentials: %q)", credentials) during connection to the YDB server. Printf func use placeholder %q for string representation of argument with quotes. If an argument implements interface fmt.Stringer, it will used through String() func. In other cases used fallback - serialization with reflection.
If such logging occurred, a malicious user with access to logs could read sensitive information (i.e. credentials) information and use it to get access to the database.
Who is impacted: applications with custom credentials object with an explicit token field.
A leak could have occurred if all of these conditions were met simultaneously:
- The credentials object does not implement the
fmt.Stringer interface (does not have a String() method) - potentially these are custom credentials. Official credentials have a String() method.
- There was an error connecting to YDB during driver creation via
ydb.Open(...).
- Some logging system was configured (
ydb-go-sdk does not log such errors by default).
- The connection error was logged into a system that a malicious user had access to.
Patches
ydb-go-sdk contains this problem in versions from v3.48.6 to v3.53.2. The fix for this problem has been released in version v3.53.3 (PR).
Workarounds
Implement the fmt.Stringer interface in your custom credentials type with explicit stringify of object state.
References
sumerki2020 Reporter
se-foster Reporter
blinkov Other
Impact
Since ydb-go-sdk/v3.48.6 if you use a custom credentials object (implementation of interface Credentials) it may leak into logs. This happens because this object could be serialized into an error message using
fmt.Errorf("something went wrong (credentials: %q)", credentials)during connection to the YDB server. Printf func use placeholder%qfor string representation of argument with quotes. If an argument implements interfacefmt.Stringer, it will used throughString()func. In other cases used fallback - serialization with reflection.If such logging occurred, a malicious user with access to logs could read sensitive information (i.e. credentials) information and use it to get access to the database.
Who is impacted: applications with custom credentials object with an explicit token field.
A leak could have occurred if all of these conditions were met simultaneously:
fmt.Stringerinterface (does not have aString()method) - potentially these are custom credentials. Official credentials have aString()method.ydb.Open(...).ydb-go-sdkdoes not log such errors by default).Patches
ydb-go-sdkcontains this problem in versions from v3.48.6 to v3.53.2. The fix for this problem has been released in version v3.53.3 (PR).Workarounds
Implement the
fmt.Stringerinterface in your custom credentials type with explicit stringify of object state.References