New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 758359 link

Starred by 2 users

Issue metadata

Status: Duplicate
Merged: issue 781363
Owner:
Closed: Mar 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug-Regression



Sign in to add a comment

chrome://net-internals#chromeos regression: debug log file is missing "tar.gz" extension

Project Member Reported by kirtika@google.com, Aug 23 2017

Issue description

chrome://net-internals#chromeos used a generate a log file with the
name "debug-logs-<timestamp>.tar.gz". Recently the tar.gz extension is missing, meaning its harder to extract/view logs on a chromebook / non-Linux-desktop. 

 
Components: Internals>Network
Components: -Internals>Network Internals>Network>Logging

Comment 3 by mmenke@chromium.org, Oct 30 2017

Components: -Internals>Network>Logging OS>Systems>Network

Comment 4 by kirtika@google.com, Feb 3 2018

Cc: r...@chromium.org st...@chromium.org snanda@chromium.org
Labels: -Type-Bug -Pri-2 Pri-1 Type-Bug-Regression
This is a regression and has been in the field for 2-3 releases now.
Customers/OEMs get confused and zip up the resulting filename and send to us. 
Can we please get some cycles on this? 

Labels: RegressedIn-62
Owner: semenzato@chromium.org
Status: Assigned (was: Untriaged)
Ha, I've been wondering why partners send me really stupid-looking zipped-up tarballs with no extension...now I know!

And unfortunately, it's our own fault :(

https://chromium-review.googlesource.com/599026

+  base::FilePath new_path = base_path;
+  // Assume that the base_path is "chrome.LATEST", and remove the extension.
+  // Ideally we would also check the value of base_path, but we cannot reliably
+  // log anything here, and aborting seems too harsh a choice.
+  new_path = new_path.ReplaceExtension("");

base::FilePath GenerateTimestampedName() is also used by StartLogRetrieval(), for debug_log_writer.cc...

https://cs.chromium.org/chromium/src/chrome/browser/chromeos/system_logs/debug_log_writer.cc?type=cs&q=GenerateTimestampedName&sq=package:chromium&l=222

That's Chrome 62.0.3182.0~114
Mergedinto: 781363
Status: Duplicate (was: Assigned)
Ah, it's fixed on ToT. Duplicated at  bug 781363 

Sign in to add a comment