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

Issue 738233 link

Starred by 1 user
Status: Archived
Owner:
hansberry@chromium.org
Closed: Jul 2017
Cc:
█ jlklein@chromium.org
█ lesliewatkins@chromium.org
khorimoto@chromium.org
jhawkins@chromium.org
hansberry@chromium.org
█ jonmann@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Should not log instant tether wifi password in logs.

Project Member Reported by jlklein@chromium.org, Jun 29 2017 
We're logging the plaintext ssid and password in proximity auth logs. We probably shouldn't.

Comment 1 by khorimoto@chromium.org, Jun 30 2017

Cc: jlklein@chromium.org lesliewatkins@chromium.org jonmann@chromium.org jhawkins@chromium.org hansberry@chromium.org
Labels: -Pri-3 M-61 Pri-1

Comment 2 by khorimoto@chromium.org, Jul 10 2017

Owner: ----
Status: Available (was: Assigned)

Comment 3 by hansberry@chromium.org, Jul 18 2017

Owner: hansberry@chromium.org
Status: Started (was: Available)
Project Member

Comment 4 by bugdroid1@chromium.org, Jul 18 2017 

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

commit a1d23fe4824cfb6c96ccb5c2752ca1bf467b59a6
Author: Ryan Hansberry <hansberry@chromium.org>
Date: Tue Jul 18 01:40:22 2017

Tether: Do not log password of host hotspot.

Bug:  738233 
Change-Id: I7c8f474904a4203ee7c069a674722c3bc60545c4
Reviewed-on: https://chromium-review.googlesource.com/575764
Commit-Queue: Ryan Hansberry <hansberry@chromium.org>
Commit-Queue: Kyle Horimoto <khorimoto@chromium.org>
Reviewed-by: Kyle Horimoto <khorimoto@chromium.org>
Cr-Commit-Position: refs/heads/master@{#487333}
[modify] https://crrev.com/a1d23fe4824cfb6c96ccb5c2752ca1bf467b59a6/chromeos/components/tether/tether_connector.cc

Comment 5 by khorimoto@chromium.org, Jul 18 2017

Status: Fixed (was: Started)

Comment 6 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)
►

Sign in to add a comment