invalid cache file location.

[adamgorMSFT]

cpp_client_telemetry/lib/api/LogManagerImpl.cpp

Line 114 in 550cc6e

std::string cacheFilePath = MAT::GetAppLocalTempDirectory();

In a uwp app, MAT::GetAppLocalTempDirectory() returns a path like
"C:\Users\adamgor\AppData\Local\Packages\myApp_8wekyb3d8bbwe\LocalState"

However it then is appended to via cacheFilePath += tenantId;

cpp_client_telemetry/lib/api/LogManagerImpl.cpp

Line 123 in 550cc6e

cacheFilePath += tenantId;

which results in
"C:\Users\adamgor\AppData\Local\Packages\myApp_8wekyb3d8bbwe\LocalStatesomeFilename.db"

As you can see from the above result there needs to be a cacheFilePath += "/" + tenantId;
or equivalent to fix this lack of folder separation issue.
Alternatively if there is an API to do a Path::Combine, that would be even better.

In the interim, I was able to work around this by providing our own config["cacheFilePath"] override before Initialization.

[maxgolov]

Maybe we should just fix the implementation of GetAppLocalTempDirectory for Win 10 Universal to always return a path with trailing slash. It seems like we do that in Win32 Desktop.

[adamgorMSFT]

Ah yep
That'd work too

cpp_client_telemetry/lib/utils/Utils.cpp

Line 84 in 780205d

// - verify that the path ends with a slash

[maxgolov]

Fix submitted. Please take a look.