aboutsummaryrefslogtreecommitdiff
path: root/toolsrc/src
diff options
context:
space:
mode:
authordan-shaw <51385773+dan-shaw@users.noreply.github.com>2019-11-14 13:12:36 -0800
committerdan-shaw <51385773+dan-shaw@users.noreply.github.com>2019-11-14 13:12:36 -0800
commit552296a74113a7f56babe232e7f554f1279484c5 (patch)
tree55d655ef619402bd22b3a4712090291b2794195f /toolsrc/src
parentdf4773c05614eb19084ae4db1fbc1bb3295d3ec6 (diff)
downloadvcpkg-552296a74113a7f56babe232e7f554f1279484c5.tar.gz
vcpkg-552296a74113a7f56babe232e7f554f1279484c5.zip
[vcpkg] update telemetry
Diffstat (limited to 'toolsrc/src')
-rw-r--r--toolsrc/src/vcpkg.cpp4
-rw-r--r--toolsrc/src/vcpkg/build.cpp10
-rw-r--r--toolsrc/src/vcpkg/install.cpp5
-rw-r--r--toolsrc/src/vcpkg/metrics.cpp12
4 files changed, 21 insertions, 10 deletions
diff --git a/toolsrc/src/vcpkg.cpp b/toolsrc/src/vcpkg.cpp
index 9cd0ddf19..c336d2f63 100644
--- a/toolsrc/src/vcpkg.cpp
+++ b/toolsrc/src/vcpkg.cpp
@@ -304,7 +304,6 @@ int main(const int argc, const char* const* const argv)
SetConsoleCP(CP_UTF8);
SetConsoleOutputCP(CP_UTF8);
- const std::string trimmed_command_line = trim_path_from_command_line(Strings::to_utf8(GetCommandLineW()));
#endif
Checks::register_global_shutdown_handler([]() {
@@ -335,9 +334,6 @@ int main(const int argc, const char* const* const argv)
{
auto locked_metrics = Metrics::g_metrics.lock();
locked_metrics->track_property("version", Commands::Version::version());
-#if defined(_WIN32)
- locked_metrics->track_property("cmdline", trimmed_command_line);
-#endif
}
System::register_console_ctrl_handler();
diff --git a/toolsrc/src/vcpkg/build.cpp b/toolsrc/src/vcpkg/build.cpp
index 618e4126b..c606594af 100644
--- a/toolsrc/src/vcpkg/build.cpp
+++ b/toolsrc/src/vcpkg/build.cpp
@@ -580,7 +580,15 @@ namespace vcpkg::Build
{
auto locked_metrics = Metrics::g_metrics.lock();
- locked_metrics->track_buildtime(spec.to_string() + ":[" + Strings::join(",", config.feature_list) + "]",
+
+ locked_metrics->track_buildtime(Hash::get_string_hash(spec.to_string(), Hash::Algorithm::Sha256) + ":[" +
+ Strings::join(",",
+ config.feature_list,
+ [](std::string feature) {
+ return Hash::get_string_hash(feature,
+ Hash::Algorithm::Sha256);
+ }) +
+ "]",
buildtimeus);
if (return_code != 0)
{
diff --git a/toolsrc/src/vcpkg/install.cpp b/toolsrc/src/vcpkg/install.cpp
index 21be2d7b0..d1c4a4b2d 100644
--- a/toolsrc/src/vcpkg/install.cpp
+++ b/toolsrc/src/vcpkg/install.cpp
@@ -1,6 +1,7 @@
#include "pch.h"
#include <vcpkg/base/files.h>
+#include <vcpkg/base/hash.h>
#include <vcpkg/base/system.print.h>
#include <vcpkg/base/util.h>
#include <vcpkg/build.h>
@@ -690,9 +691,9 @@ namespace vcpkg::Install
// log the plan
const std::string specs_string = Strings::join(",", action_plan, [](const AnyAction& action) {
if (auto iaction = action.install_action.get())
- return iaction->spec.to_string();
+ return Hash::get_string_hash(iaction->spec.to_string(), Hash::Algorithm::Sha256);
else if (auto raction = action.remove_action.get())
- return "R$" + raction->spec.to_string();
+ return "R$" + Hash::get_string_hash(raction->spec.to_string(), Hash::Algorithm::Sha256);
Checks::unreachable(VCPKG_LINE_INFO);
});
diff --git a/toolsrc/src/vcpkg/metrics.cpp b/toolsrc/src/vcpkg/metrics.cpp
index b8c55919e..7aaa852c3 100644
--- a/toolsrc/src/vcpkg/metrics.cpp
+++ b/toolsrc/src/vcpkg/metrics.cpp
@@ -184,9 +184,15 @@ namespace vcpkg::Metrics
if (buildtime_names.size() > 0)
{
if (props_plus_buildtimes.size() > 0) props_plus_buildtimes.push_back(',');
- props_plus_buildtimes.append(Strings::format(R"("buildnames": [%s], "buildtimes": [%s])",
- Strings::join(",", buildtime_names, to_json_string),
- Strings::join(",", buildtime_times)));
+ props_plus_buildtimes.append(
+ Strings::format(R"("buildnames": [%s], "buildtimes": [%s])",
+ Strings::join(",",
+ buildtime_names,
+ [](std::string buildname) {
+ return to_json_string(vcpkg::Hash::get_string_hash(
+ buildname, Hash::Algorithm::Sha256));
+ }),
+ Strings::join(",", buildtime_times)));
}
const std::string& session_id = get_session_id();