Re: [webauthn] Device-bound key extension (#1658)

A nitpick observation wrt:
```
  ; whether this key is scoped to the whole device, or a loosely-defined,
  ; smaller scope called "app". [ ... ]

  context: "device" / "app",
```
...I'm inclined to declare `context` as a boolean, both to future-proof it against mission-creep and reduce bytes-on-the-wire:
```
  ; Whether this key is scoped to the entire device, or a loosely-defined,
  ; smaller scope called "app". [ ... ]

  context: bool,    ; true means "entire-device" context, and 
                    ; false means "app"-scoped context
```
...(I'm also inclined to rename it, say, "cntx").

Though, one could also argue that "scope" better reflects the connotations here:
```
  ; Whether this key is scoped to the entire device, or a loosely-defined,
  ; smaller scope called "app". [ ... ]

  scope: bool,    ; true means "entire device", i.e., "all apps" scope, and 
                  ; false means per-"app" scope
```





-- 
GitHub Notification of comment by equalsJeffH
Please view or discuss this issue at https://github.com/w3c/webauthn/issues/1658#issuecomment-896621083 using your GitHub account


-- 
Sent via github-notify-ml as configured in https://github.com/w3c/github-notify-ml-config

Received on Wednesday, 11 August 2021 08:33:23 UTC