MAINT-7594: add platform name string and address size to login request for crash stats (and add request parameter logging at DEBUG)

#18 Merged at b75591b
Repository
viewer-maint-7594
Branch
default
Repository
viewer-release
Branch
default
Author
  1. Oz Linden
Reviewers
Description

Do Not Pull to viewer-release - for review only

The viewer crash stats currently have a very large percentage of crashes for which no platform version is available. This is because the crash indications are logged via the LogLogin message at login time, but the platform is logged as part of session statistics later. The DW reports have to combine these and sometimes are unsuccessful.

By adding the platform long form name and the build address size to the initial log entry, we will be able to generate much better per-platform crash reports.

Comments (2)

  1. Callum Linden

    The for(..) loop to copy credentials into request params seems overkill vs assign as before and append but looks good to me. I love the idea that we'll have more reliable stats on platform.