New issue
Advanced search Search tips

Issue 803826 link

Starred by 1 user

Issue metadata

Status: Available
Owner: ----
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Windows , Chrome , Mac
Pri: 3
Type: ----


Participants' hotlists:
Hotlist-1


Sign in to add a comment

Add TokenBinding to WebAuthN (desktop)

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

Issue description

Actually add the TokenBindingID to the CollecteClientData to be signed over by the authenticator.
 
Owner: hongjunchoi@chromium.org
Status: Assigned (was: Available)
Summary: Add TokenBinding to WebAuthN (desktop) (was: Add TokenBinding to WebAuthN)
From Anupam: 
"//src/content/browser/webauth/authenticator_impl.cc
needs to do something like

BrowserContext* browser_context =
      render_frame_host->GetProcess()->GetBrowserContext();
 StoragePartition* storage_partition =
      BrowserContext::GetDefaultStoragePartition(browser_context);


and then, like in 
//src/extensions/browser/api/messaging/message_property_provider.cc
void MessagePropertyProvider::GetChannelID()
asynchronously get the appropriate channel id."

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

Labels: M-69 Pri-2
Owner: ----
Status: Available (was: Assigned)
Kim, what's the urgency of this? Can you please update it below?

(Removing OWNER for the moment until priority is defined.)

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

Status: Untriaged (was: Available)

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

Labels: Hotlist-WebAuthnFixit
Labels: -M-69 M-68
It's not urgent, but it'd be nice to have for M68.

Note also that Token Binding is still controlled by a flag, which will need to be taken into account when adding support. https://cs.chromium.org/chromium/src/components/network_session_configurator/common/network_features.cc
Status: Available (was: Untriaged)
Labels: -Hotlist-WebAuthnFixit
Labels: -M-68
Labels: -Pri-2 Hotlist-WebAuthnFixit Pri-3

Sign in to add a comment