New issue
Advanced search Search tips

Issue 803825 link

Starred by 0 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 1
Type: ----


Participants' hotlists:
Hotlist-1


Sign in to add a comment

WebAuthN API logging / metrics

Project Member Reported by kpaulhamus@chromium.org, Jan 19 2018

Issue description

We need to add robust metrics/logging to the WebAuthN API, including U2F/CTAP flows.
 
Owner: hongjunchoi@chromium.org
Status: Assigned (was: Available)

Comment 2 by engedy@chromium.org, Mar 31 2018

Labels: M-67 Pri-2
Owner: ----
Status: Available (was: Assigned)

Comment 3 by engedy@chromium.org, Mar 31 2018

Labels: -Pri-2 Pri-1

Comment 4 by engedy@chromium.org, Mar 31 2018

Labels: Hotlist-WebAuthnFixit
Project Member

Comment 5 by bugdroid1@chromium.org, Apr 24 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/69ee0e23e38fb6b260a8ce19ba435b15463d74e1

commit 69ee0e23e38fb6b260a8ce19ba435b15463d74e1
Author: Kim Paulhamus <kpaulhamus@chromium.org>
Date: Tue Apr 24 00:10:38 2018

Add counters for public key credential operations

Bug: 803825
Change-Id: I9387a91cdc2a86520fcf9b09cf75c0950f11078d
Reviewed-on: https://chromium-review.googlesource.com/1019734
Reviewed-by: Jochen Eisinger <jochen@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#552916}
[modify] https://crrev.com/69ee0e23e38fb6b260a8ce19ba435b15463d74e1/third_party/blink/public/platform/web_feature.mojom
[modify] https://crrev.com/69ee0e23e38fb6b260a8ce19ba435b15463d74e1/third_party/blink/renderer/modules/credentialmanager/credentials_container.cc
[modify] https://crrev.com/69ee0e23e38fb6b260a8ce19ba435b15463d74e1/tools/metrics/histograms/enums.xml

Comment 6 by engedy@chromium.org, May 23 2018

Labels: -M-67 M-68
Project Member

Comment 7 by bugdroid1@chromium.org, May 29 2018

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/b5f7b9b7d3ba842f43f76a90ebb191ff71594131

commit b5f7b9b7d3ba842f43f76a90ebb191ff71594131
Author: Kim Paulhamus <kpaulhamus@chromium.org>
Date: Tue May 29 15:42:44 2018

Add UseCounter metrics for WebAuthn to UKM

Bug: 803825
Change-Id: I7267668d906fc3335d28f4d71ef35d60dc0d8611
Reviewed-on: https://chromium-review.googlesource.com/1070307
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/heads/master@{#562438}
[modify] https://crrev.com/b5f7b9b7d3ba842f43f76a90ebb191ff71594131/chrome/browser/page_load_metrics/observers/use_counter/ukm_features.cc

Labels: Merge-Request-68
Requesting merge for commit in comment 7.
Owner: kpaulhamus@chromium.org
Status: Started (was: Available)
Project Member

Comment 10 by sheriffbot@chromium.org, May 30 2018

Labels: -Merge-Request-68 Hotlist-Merge-Approved Merge-Approved-68
Your change meets the bar and is auto-approved for M68. Please go ahead and merge the CL to branch 3440 manually. Please contact milestone owner if you have questions.
Owners: cmasso@(Android), kariahda@(iOS), bhthompson@(ChromeOS), abdulsyed@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Project Member

Comment 11 by bugdroid1@chromium.org, May 30 2018

Labels: -merge-approved-68 merge-merged-3440
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/6fec4cdf2cfc351fd96d52f9f6c37d90045d8149

commit 6fec4cdf2cfc351fd96d52f9f6c37d90045d8149
Author: Kim Paulhamus <kpaulhamus@chromium.org>
Date: Wed May 30 15:55:47 2018

Add UseCounter metrics for WebAuthn to UKM

Bug: 803825
Change-Id: I7267668d906fc3335d28f4d71ef35d60dc0d8611
Reviewed-on: https://chromium-review.googlesource.com/1070307
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Reviewed-by: Bryan McQuade <bmcquade@chromium.org>
Commit-Queue: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#562438}(cherry picked from commit b5f7b9b7d3ba842f43f76a90ebb191ff71594131)
Reviewed-on: https://chromium-review.googlesource.com/1079307
Reviewed-by: Kim Paulhamus <kpaulhamus@chromium.org>
Cr-Commit-Position: refs/branch-heads/3440@{#43}
Cr-Branched-From: 010ddcfda246975d194964ccf20038ebbdec6084-refs/heads/master@{#561733}
[modify] https://crrev.com/6fec4cdf2cfc351fd96d52f9f6c37d90045d8149/chrome/browser/page_load_metrics/observers/use_counter/ukm_features.cc

Labels: -M-68 M-70
Owner: ----
Status: Available (was: Started)
Removing self while not actively working on this.
Labels: -M-70 M-73
Owner: kenrb@chromium.org
Status: Assigned (was: Available)

Sign in to add a comment