Return-Path: Received: from hemlock.osuosl.org (smtp2.osuosl.org [140.211.166.133]) by lists.linuxfoundation.org (Postfix) with ESMTP id 006B2C0893 for ; Wed, 23 Dec 2020 07:29:36 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by hemlock.osuosl.org (Postfix) with ESMTP id D716F87200 for ; Wed, 23 Dec 2020 07:29:36 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from hemlock.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id Ks+Fyyp9CDRp for ; Wed, 23 Dec 2020 07:29:35 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.7.6 Received: from mail-oi1-f175.google.com (mail-oi1-f175.google.com [209.85.167.175]) by hemlock.osuosl.org (Postfix) with ESMTPS id 645AB87068 for ; Wed, 23 Dec 2020 07:29:35 +0000 (UTC) Received: by mail-oi1-f175.google.com with SMTP id q25so17410248oij.10 for ; Tue, 22 Dec 2020 23:29:35 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=qtZxCmvokIdA6N9wSmhvdRUJAmyj0aOACfo0WE5CWRo=; b=AeZqzadYJLcNS5NQMGnY7MZCCofmeFGUC+1W8iR0MaZoCSbqAROHqpRC65o7z+Xrjf h6wY/ywbHMXBznlp/6VzsO82cNvstaJYYzd6uqtP3+dGwS9cjV3zTLJ5j8s9Js1PrmjE OKYF3G3OQ7QZ3n+4FjHfsz/MZhux1AOheniLAOvJyom1d8rIUkFcuXP+EuhvMVSqCjQF pL44Id12fiunpvi1guAAQcZky2ZKVMXwBI1U8UlBTwgn7bBGawfXVki/7MJvteeXAE6y FCdsObYNwIGYX060c2dEmyZ8g7NsNKRU3VcOW6RZQf0TP1BTO/0NiJljBZLuej6eLx6l mUHA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=qtZxCmvokIdA6N9wSmhvdRUJAmyj0aOACfo0WE5CWRo=; b=ReOce/4vSQhMJhPHQhJONV458rkbAayp9wjNPc6J/xMrog7hjQspIGWhK+MxyveIyt Ya1Muu9SKMD/sDfagO95tKx9LNTL9drYZ+POO+yU/kyB6NcNbcI+cV6Qo3rnZlAY8wyb XxbgMiehyvqGPVlxPkFVfLBxUFWkeCExDoZTx9ydM2WS6Ugf8ikk2JPyc/BL65+zdlY6 qsCiBkGkQJlnb0kpEVypNfPzr6O9iX+WVBcci+VOWzzHsPKg6bF//dqSvSkZqt1X4an9 mLPpSx4Xj6+Eh2LMjLkbDbRrvMeR5D+9aMNV/rQSdzpvOAc3JVrQMBLkO/4uW3qjsmFb WHYg== X-Gm-Message-State: AOAM533ktJhnwLCPvF2vPsAqaOnslBra/lw33+CILXHQ6ssh/TZf/0TT CpPLLriA1gneEGvJRAB+u0aodYs9N439hid7fAQ= X-Google-Smtp-Source: ABdhPJw03mBEXiKvOuE0lyfHxMc+HjIeubyyfFFX1mgfNKjuiB/ApSPJgazqGkQzp4IaaJPFOjhSJhxA8coiLJr1GZg= X-Received: by 2002:aca:c4cd:: with SMTP id u196mr17288870oif.133.1608708574553; Tue, 22 Dec 2020 23:29:34 -0800 (PST) MIME-Version: 1.0 References: <202012230215.46394.luke@dashjr.org> In-Reply-To: <202012230215.46394.luke@dashjr.org> From: monokh Date: Wed, 23 Dec 2020 07:29:23 +0000 Message-ID: To: Luke Dashjr Content-Type: multipart/alternative; boundary="00000000000022291505b71ca754" X-Mailman-Approved-At: Wed, 23 Dec 2020 08:42:31 +0000 Cc: Bitcoin Protocol Discussion Subject: Re: [bitcoin-dev] BIP Proposal: Wallet Interface X-BeenThere: bitcoin-dev@lists.linuxfoundation.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Bitcoin Protocol Discussion List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 23 Dec 2020 07:29:37 -0000 --00000000000022291505b71ca754 Content-Type: text/plain; charset="UTF-8" Thanks for the input Luke. > 1) People should not be encouraged to write or use web browsers for their wallet. Indeed. Holding keys in the browser can be very insecure, however the spec is not limited to this. I will amend to make this clear. The same interface can be used to communicate from a web context or even desktop application with hardware wallets where keys are segregated safely. The prominent hardware wallets already have such an interface. Unfortunately as there has been no standardisation, an application must specifically provide an implementation for each wallet to be compatible. > 2) You may want to look over earlier work in this area. Please share if you have specifics in mind. What has been considered were mainly hardware wallet apis. The requests have been defined such that they would be compatible. I will make references to such considerations in the text. I welcome any feedback on what may be missing or problematic for these providers - something I will also pursue outwith the thread. -monokh On Wed, Dec 23, 2020 at 2:15 AM Luke Dashjr wrote: > 1) People should not be encouraged to write or use web browsers for their > wallet. > 2) You may want to look over earlier work in this area. > > On Tuesday 22 December 2020 14:43:11 monokh via bitcoin-dev wrote: > > Hi > > > > This is a first draft of a BIP we intend to submit. The main intention is > > to define a simple interface that wallets and applications can agree on > > that would cover the vast majority of use cases. This can enable writing > > bitcoin applications (e.g. time lock, multi sig) on the web that can be > > seamlessly used with any compatible wallets. We have implementations of > > such examples but I don't want to turn this thread into a promotion and > > rather focus on the spec. > > > > Appreciate input from the list. Please share if there are existing > efforts, > > relevant specs or use cases. > > > > ------------------------------ > > > > A wallet interface specification for bitcoin applications > > > > ## Abstract > > > > This BIP describes an API for Bitcoin wallets and applications as a > > standard. > > > > ## Summary > > > > Bitcoin wallets should expose their address derivation and signing > > functions to external applications. The interface would be expressed as > > follows in javascript: > > > > ``` > > { > > // Wallet Metadata > > wallet: { > > name: 'Bitcoin Core' > > }, > > > > // Request access to the wallet for the current host > > async enable: (), > > > > // Request addresses and signatures from wallet > > async request ({ method, params }) > > } > > ``` > > > > In the web context the interface could be exposed at the top level of a > > webpage, for example under `window.bitcoin`. However this spec does not > > intend to define any standards for how and where the interfaces should be > > exposed. > > > > ## Motivation > > > > Due to the seldom available APIs exposed by wallets, applications (web or > > otherwise) are limited in how they are able to interact. Generally only > > simple sends have been available. A more robust API that introduces other > > requests will promote richer Bitcoin applications. > > > > Additionally, wallet APIs have frequently included inconsistencies in > their > > interfaces and behaviour. This has required applications to build and > > maintain a separate client for each wallet, increasing the risk of bugs > and > > unintended behaviour as well as being a limiting factor for the adoption > of > > usable bitcoin applications. > > > > With a standardised wallet API: > > > > - Wallets have a clear API to implement > > - Applications have a clear expectation of wallet interface and behaviour > > - Applications become agnostic to the wallet specifics, increasing choice > > for users > > > > If more wallets implement the specification, applications will be > developed > > more confidently by benefiting from the wallet interoperability. This > > creates a positive feedback loop. > > > > ## Specification > > > > For simplicity, the interface is defined in the context of web > applications > > running in the browser (JS) however, they are simple enough to be easily > > implemented in other contexts. > > > > ### General Rules > > > > - For sensitive functions (e.g. signing), wallet software should always > > prompt the user for confirmation > > > > ### Types > > > > **UserDeniedError** > > An error type indicating that the application's request has been denied > by > > the user > > Type: Error > > > > **Hex** > > Type: String > > Example: > > `"0000000000000000000a24677957d1e50d70e67c513d220dbe8868c4c3aefc08"` > > > > **Address** > > Address details > > Type: Object > > Example: > > > > ``` > > { > > "address": "bc1qn0fqlzamcfuahq6xuujrq08ex7e26agt20gexs", > > "publicKey": > > "02ad58c0dced71a236f4073c3b6f0ee27dde6fe96978e9a9c9500172e3f1886e5a", > > "derivationPath": "84'/1'/0'/0/0" > > } > > ``` > > > > ### API > > > > The wallet must implement the following methods. > > > > **enable** > > > > The enable call prompts the user for access to the wallet. > > > > If successful, it resolves to an address (`**Address**` type) of the > > wallet. Typically the first external address to be used as an identity. > > > > **`UserDeniedError`** will be thrown if the request is rejected. > > > > **request** > > > > The request method must take one parameter in the following format: > > > > ``` > > { > > "method": "wallet_methodName", > > "params": ["foo", "bar", "baz"] > > } > > ``` > > > > For a list of mandatory methods see Table > > > > The wallet should reject request calls unless `enable` has been resolved. > > > > Sensitive requests that involve signing should always prompt the user for > > confirmation > > > > On success the request should resolve to the response as defined in the > > method table. > > > > **`UserDeniedError`** will be thrown if the request is rejected. > > > > **Mandatory methods** > > > > method: `wallet_getAddresses` params: [`index = 0, numAddresses = 1, > change > > = false`] > > return: `[ Address ]` > > error: UserDeniedError > > > > method: `wallet_signMessage` params: `[ message, address ]` > > return: Signature `Hex` > > error: UserDeniedError > > > > method: `wallet_signPSBT` params: `[ [psbtBase64, inputIndex, address] ]` > > return: `psbtBase64` > > error: UserDeniedError > > > > method: `wallet_getConnectedNetwork` params: `[]` > > return: Network object `mainnet` | `testnet` | `regetst` > > error: UserDeniedError > > > > ## Rationale > > > > The purpose of the API is to expose a set of commonly used wallet > > operations. In addition, it should be flexible enough to serve for other > > requests such as node RPC calls. > > > > **Why is there a singular request call instead of named methods?** > > The transport layer for the requests cannot be assumed, therefore it is > > much more flexible to instead define an abstract format. > > > > **Why are the mandatory methods so primitive? Where is getBalance, > > getUtxos, ... ?** > > A wallet need not worry about providing every possible scenario for > usage. > > The primitives of keys and signing can expose enough to applications to > do > > the rest. Applications should have flexibility in how they implement > these > > functions. It is the role of a library rather than the wallet. > > > > ## Security Implications > > > > Great care should be taken when exposing wallet functionality externally > as > > the security and privacy of the user is at risk. > > > > ### Signing > > > > Operations that trigger signing using private keys should be guarded > behind > > confirmation screens where the user is fully aware of the nature of the > > transaction. In the example of a PSBT signature request, the outputs, the > > inputs and which key is being used should be clearly marked. > > > > ### Privacy > > > > Some api methods expose metadata about the user, such as public keys. > > Depending on how privacy focused the wallet intends to be, the wallet > could > > protect these behind a confirmation. Commonly the wallet just needs to > give > > the origin access to all of its public keys, however it could also allow > > the option to expose only selected derivation paths. > > > > -monokh > > --00000000000022291505b71ca754 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Thanks for the input Luke.

> 1) People should no= t be encouraged to write or use web browsers for their wallet.

Indee= d. Holding keys in the browser can be very insecure, however the spec is no= t limited to this. I will amend to make this clear. The same interface can = be used to communicate from a web context or even desktop application with = hardware wallets where keys are segregated safely. The prominent hardware w= allets already have such an interface. Unfortunately as there has been no s= tandardisation, an application must specifically provide an implementation = for each wallet to be compatible.

> 2) You may want to look over = earlier work in this area.

Please share if you have specifics in min= d. What has been considered were mainly hardware wallet apis. The requests = have been defined such that they would be compatible. I will make reference= s to such considerations in the text. I welcome any feedback on what may be= missing or problematic for these providers - something I will also pursue = outwith the thread.

-monokh=C2=A0

On Wed, Dec 23, 2020 at 2:15 AM Luk= e Dashjr <luke@dashjr.org> wro= te:
1) People sh= ould not be encouraged to write or use web browsers for their
wallet.
2) You may want to look over earlier work in this area.

On Tuesday 22 December 2020 14:43:11 monokh via bitcoin-dev wrote:
> Hi
>
> This is a first draft of a BIP we intend to submit. The main intention= is
> to define a simple interface that wallets and applications can agree o= n
> that would cover the vast majority of use cases. This can enable writi= ng
> bitcoin applications (e.g. time lock, multi sig) on the web that can b= e
> seamlessly used with any compatible wallets. We have implementations o= f
> such examples but I don't want to turn this thread into a promotio= n and
> rather focus on the spec.
>
> Appreciate input from the list. Please share if there are existing eff= orts,
> relevant specs or use cases.
>
> ------------------------------
>
> A wallet interface specification for bitcoin applications
>
> ## Abstract
>
> This BIP describes an API for Bitcoin wallets and applications as a > standard.
>
> ## Summary
>
> Bitcoin wallets should expose their address derivation and signing
> functions to external applications. The interface would be expressed a= s
> follows in javascript:
>
> ```
> {
> // Wallet Metadata
> wallet: {
> name: 'Bitcoin Core'
> },
>
> // Request access to the wallet for the current host
> async enable: (),
>
> // Request addresses and signatures from wallet
> async request ({ method, params })
> }
> ```
>
> In the web context the interface could be exposed at the top level of = a
> webpage, for example under `window.bitcoin`. However this spec does no= t
> intend to define any standards for how and where the interfaces should= be
> exposed.
>
> ## Motivation
>
> Due to the seldom available APIs exposed by wallets, applications (web= or
> otherwise) are limited in how they are able to interact. Generally onl= y
> simple sends have been available. A more robust API that introduces ot= her
> requests will promote richer Bitcoin applications.
>
> Additionally, wallet APIs have frequently included inconsistencies in = their
> interfaces and behaviour. This has required applications to build and<= br> > maintain a separate client for each wallet, increasing the risk of bug= s and
> unintended behaviour as well as being a limiting factor for the adopti= on of
> usable bitcoin applications.
>
> With a standardised wallet API:
>
> - Wallets have a clear API to implement
> - Applications have a clear expectation of wallet interface and behavi= our
> - Applications become agnostic to the wallet specifics, increasing cho= ice
> for users
>
> If more wallets implement the specification, applications will be deve= loped
> more confidently by benefiting from the wallet interoperability. This<= br> > creates a positive feedback loop.
>
> ## Specification
>
> For simplicity, the interface is defined in the context of web applica= tions
> running in the browser (JS) however, they are simple enough to be easi= ly
> implemented in other contexts.
>
> ### General Rules
>
> - For sensitive functions (e.g. signing), wallet software should alway= s
> prompt the user for confirmation
>
> ### Types
>
> **UserDeniedError**
> An error type indicating that the application's request has been d= enied by
> the user
> Type: Error
>
> **Hex**
> Type: String
> Example:
> `"0000000000000000000a24677957d1e50d70e67c513d220dbe8868c4c3aefc0= 8"`
>
> **Address**
> Address details
> Type: Object
> Example:
>
> ```
> {
> "address": "bc1qn0fqlzamcfuahq6xuujrq08ex7e26agt20gexs&= quot;,
> "publicKey":
> "02ad58c0dced71a236f4073c3b6f0ee27dde6fe96978e9a9c9500172e3f1886e= 5a",
> "derivationPath": "84'/1'/0'/0/0"
> }
> ```
>
> ### API
>
> The wallet must implement the following methods.
>
> **enable**
>
> The enable call prompts the user for access to the wallet.
>
> If successful, it resolves to an address (`**Address**` type) of the > wallet. Typically the first external address to be used as an identity= .
>
> **`UserDeniedError`** will be thrown if the request is rejected.
>
> **request**
>
> The request method must take one parameter in the following format: >
> ```
> {
> "method": "wallet_methodName",
> "params": ["foo", "bar", "baz"= ]
> }
> ```
>
> For a list of mandatory methods see Table
>
> The wallet should reject request calls unless `enable` has been resolv= ed.
>
> Sensitive requests that involve signing should always prompt the user = for
> confirmation
>
> On success the request should resolve to the response as defined in th= e
> method table.
>
> **`UserDeniedError`** will be thrown if the request is rejected.
>
> **Mandatory methods**
>
> method: `wallet_getAddresses` params: [`index =3D 0, numAddresses =3D = 1, change
> =3D false`]
> return: `[ Address ]`
> error: UserDeniedError
>
> method: `wallet_signMessage` params: `[ message, address ]`
> return: Signature `Hex`
> error: UserDeniedError
>
> method: `wallet_signPSBT` params: `[ [psbtBase64, inputIndex, address]= ]`
> return: `psbtBase64`
> error: UserDeniedError
>
> method: `wallet_getConnectedNetwork` params: `[]`
> return: Network object `mainnet` | `testnet` | `regetst`
> error: UserDeniedError
>
> ## Rationale
>
> The purpose of the API is to expose a set of commonly used wallet
> operations. In addition, it should be flexible enough to serve for oth= er
> requests such as node RPC calls.
>
> **Why is there a singular request call instead of named methods?**
> The transport layer for the requests cannot be assumed, therefore it i= s
> much more flexible to instead define an abstract format.
>
> **Why are the mandatory methods so primitive? Where is getBalance,
> getUtxos, ... ?**
> A wallet need not worry about providing every possible scenario for us= age.
> The primitives of keys and signing can expose enough to applications t= o do
> the rest. Applications should have flexibility in how they implement t= hese
> functions. It is the role of a library rather than the wallet.
>
> ## Security Implications
>
> Great care should be taken when exposing wallet functionality external= ly as
> the security and privacy of the user is at risk.
>
> ### Signing
>
> Operations that trigger signing using private keys should be guarded b= ehind
> confirmation screens where the user is fully aware of the nature of th= e
> transaction. In the example of a PSBT signature request, the outputs, = the
> inputs and which key is being used should be clearly marked.
>
> ### Privacy
>
> Some api methods expose metadata about the user, such as public keys.<= br> > Depending on how privacy focused the wallet intends to be, the wallet = could
> protect these behind a confirmation. Commonly the wallet just needs to= give
> the origin access to all of its public keys, however it could also all= ow
> the option to expose only selected derivation paths.
>
> -monokh

--00000000000022291505b71ca754--