From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from smtp1.osuosl.org (smtp1.osuosl.org [IPv6:2605:bc80:3010::138]) by lists.linuxfoundation.org (Postfix) with ESMTP id 4164FC002D for ; Mon, 29 Aug 2022 11:25:31 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id 18A91818CA for ; Mon, 29 Aug 2022 11:25:31 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org 18A91818CA Authentication-Results: smtp1.osuosl.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20210112 header.b=qUHyki1C X-Virus-Scanned: amavisd-new at osuosl.org X-Spam-Flag: NO X-Spam-Score: -1.599 X-Spam-Level: X-Spam-Status: No, score=-1.599 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, PDS_BTC_ID=0.499, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=no autolearn_force=no Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id dkdECRu2LSl2 for ; Mon, 29 Aug 2022 11:25:28 +0000 (UTC) X-Greylist: whitelisted by SQLgrey-1.8.0 DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org 978A58176D Received: from mail-lf1-x132.google.com (mail-lf1-x132.google.com [IPv6:2a00:1450:4864:20::132]) by smtp1.osuosl.org (Postfix) with ESMTPS id 978A58176D for ; Mon, 29 Aug 2022 11:25:27 +0000 (UTC) Received: by mail-lf1-x132.google.com with SMTP id m7so1769161lfq.8 for ; Mon, 29 Aug 2022 04:25:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc; bh=baqxdQNUvhGqlpe39JvvzjUttTz+r1kCy24QCZm0tWQ=; b=qUHyki1CNBoyuaXxhtvCu0xZMdN9CWOTime7QN0ViXOpO/PubJTInS7fGVOLuaqISl tu3xp79PqICjpWHJy/8Ov/xsZr6zazCJxdLzEDiUDhDva/CT0q5TQuwJ204FZcqGvaC8 B88LCIGVYh0sqnaRBTaWYuc4K1QIFeUOkqttPkEehMmNpU8MvRiOgvGPaSBEPx9REpAc +zl/aczacfY8YWPor1prvXS3sr90DwagJJDonvUa8hAE/4PDuazQKJ6ta+eiqJQ5NIHo qcyc5t4sV0snjpv2cFXUz+QMu/thqXfnLr0sf1T/8CZNB6AIorSKTGz4lhiLam37VNkk I+5Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc; bh=baqxdQNUvhGqlpe39JvvzjUttTz+r1kCy24QCZm0tWQ=; b=go/je1nDZefD0WO+PSSQHFo74AcXc1VT0yidzCRrXwcQLQ68Qzy/Ds4IY/kzrjhRPk YsRk9g2+YXKPkKZ0fTpxx7u04Mo+Nzb94hWn259PP53H9zetTwbDkw8sFbZsnoNR3XQG LxQ5sgwknJnA6C0WA33clzqoJ8ztWjM0BZIUepecai80Kr3oJaFPhRJOvpGFeSYUT32C E0HVY9DKdkZ2dCYp5ZrJbV+ZPQM4tsmJo01VlgELLseSbmpDjgCWmMiePYBextFhyU8W SJaYFXiJT7DPK4r6a9Pqw6H53p1zr1vRdcaSofErz8lr1mpe6DZ+XLESK1JVgFYXOB3C VrMw== X-Gm-Message-State: ACgBeo07UZTH7GYPT0dboGShEF5DiWe5a6HDyLvT1fQebgzkncRvheN7 sHjOpm8smDWiMm09cvo/RrzWvIUBmYLtqyQsPGWzDwQy/Js= X-Google-Smtp-Source: AA6agR5nADsGJ7FemsObVPQIcqzpyUaiMRNkWBk9U7KxG77Du36MHCU08dPFLbzrh3nF85Xrzo6j7uz1SU/KCIKRcxU= X-Received: by 2002:a05:6512:3a8e:b0:492:c760:b7ad with SMTP id q14-20020a0565123a8e00b00492c760b7admr6914554lfu.473.1661772325308; Mon, 29 Aug 2022 04:25:25 -0700 (PDT) MIME-Version: 1.0 References: <20220824190958.gklg3riadci3ttgm@artanis> In-Reply-To: <20220824190958.gklg3riadci3ttgm@artanis> From: Craig Raw Date: Mon, 29 Aug 2022 13:25:13 +0200 Message-ID: To: Ali Sherief Content-Type: multipart/alternative; boundary="00000000000025ce9f05e75f859e" X-Mailman-Approved-At: Mon, 29 Aug 2022 11:26:31 +0000 Cc: bitcoin-dev@lists.linuxfoundation.org Subject: Re: [bitcoin-dev] BIP Proposal: Wallet Labels Export Format 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: Mon, 29 Aug 2022 11:25:31 -0000 --00000000000025ce9f05e75f859e Content-Type: text/plain; charset="UTF-8" Thanks for your feedback @Ali. I am attempting to achieve two goals with this proposal, primarily for the benefit of wallet users: Goal #1. Transfer labels between different wallet implementations Goal #2. Manage labels in applications outside of Bitcoin wallets (such as Excel) Much of the feedback so far has indicated the tension between these two goals - it may be that it is too difficult to achieve both, in which case Goal #1 is the most important. That said, I think further exploration is still necessary before abandoning Goal #2, because removing it would significantly reduce the value of this proposal and mean users need to rely on application-specific workarounds. > it is important that a version byte is defined If Goal #2 is to be achieved it's difficult to mandate this, particularly if one requires bit flags to be set. Should an importing wallet fail to import if the version byte is not present, even if all the data is otherwise correct? Although it is difficult to know in advance how a format may be extended, it is certainly possible to extend this format with additional types where the nature of hashes serve as unique identifiers (more on this below). > Don't mandate the file extension... There is no way to enforce this on a BIP level. I'm not quite sure what you mean here - for example BIP174, which is widely used, states "Binary PSBT files should use the .psbt file extension." Also, this contradicts Goal #2 - Excel and Numbers register as handlers for .csv, and so make it clear that the file is editable outside of a wallet. > ZIP does not have good performance or compression ratio Indeed, but it is very widely available. That said, gzip is supported widely too these days. Unfortunately, gzip does not offer encryption (see next answer). > ZIP is an archiving format, that happens to have its own compression format. I agree this is not ideal. My main reason for choosing ZIP was that it supports encryption. It seems to me that without considering encryption, an application must create label export files that allow privacy-sensitive wallet information to be readable in plain text. Being able to transfer labels without risking privacy is IMO valuable. I considered other encryption formats such as PGP, but they are much more niche and so again contradict Goal #2. > I don't see the benefit of encrypting addresses and labels together... additionally, the password you propose is insecure - anybody with access to the wallet can unlock it I'm not sure I understand your question, but both wallet addresses and wallet labels contain privacy-sensitive information that should be protected. Wrt to the password, there is actually a more fundamental problem with using the wallet xpub - there is no equivalent for multisig wallets. For this reason I'll remove that requirement in future iterations. > Why the need for input and output formats? There is no difference between them on the wallet level, because they are always identified with a txid and output index. The input refers to the txid and the input index (in the set of vin), so the difference is the context in which they are displayed. A wallet will not necessarily store the spent outputs for a funding transaction containing a UTXO coming into the wallet, but it will contain references to the inputs as part of that transaction. > Another important point is that practically nobody labels inputs or outputs To the contrary, UTXOs are very frequently labelled, as they link and reveal information when spent. Inputs are much less frequently labelled, but there is no particular reason to exclude them. > there is a net benefit for the addresses to be exported in ascending order Indeed, but it makes achieving Goal #2 much more difficult for marginal benefit. > It's better to mandate that they should always be double-quoted, since only wallets will generate label exports anyway. Rather I think it's better to mandate RFC4180 is followed, as per recommendations in other feedback. > The importing code is too naive... it should utilize a dedicate item type field that unambiguously identifies the item It's unclear to me what you mean here. As I've indicated it is currently possible to disambiguate between addresses/transactions/etc without the need for a 3rd column, but in any case the hash functions used ensure that labels will not be associated incorrectly. Even in the unlikely event of some future address type being indistinguishable from a txid, it will simply not match any txids in the wallet. Craig On Wed, Aug 24, 2022 at 9:10 PM Ali Sherief wrote: > Hi Craig, > > This a really good proposal. I studied your BIP and I have some feedback > on some parts of it. > > > The first line in the file is a header, and should be ignored on import. > > From past experience and lessons, most notably BIP39, it is important that > a version byte is defined somewhere in case someone wants to extend it in > the future, currently there is no version byte which someone can increment > if somebody wants to extend it. In the unique case of CSV files, you should > make the header line mandatory (I see you have already implied this, but > you should make it explicit in the BIP), but instead of a line with columns > in it, I suggest instead of Reference,Label, you make the format like this: > > BIP-wallet-labels, > > Since there are two columns per record, this works out nicely. The first > column can be the name of the BIP - BIPxxxx where the x's are numbers, and > the second column can be an unsigned 32-bit integer (most significant 8 > bits reserved for version, the remaining for flags, or perhaps the entirety > for version - but I recommend leaving at least some bits for flags, even if > they all end up being just "reserved"). > > You should make importing fail if the header line is not exactly as > specified - or appropriate, should you decide a different format for the > header. > > > Files exported should use the .csv file extension. > Don't mandate the file extension (read below for why): > > > In order to reduce file size while retaining wide accessibility, the CSV > > file may be compressed using the ZIP file format, using the .zip > > file extension. > I see three problems with this. The first is more important than the later > two because it makes them moot points, but I'll mention them anyway so you > get a background of the situation: > - The BIP is trying to specify in what file format the export format can > be written in onto the filesystem. There is no way to enforce this on a BIP > level (besides, Unix operating systems don't even consider the file > extension, they use its mimetype). Also specifying this in the BIP will > prevent modular "Layer 2" protocols and schemes from encoding the Export > labels into another format - for example Base64 or with their own > compression algorithm. > > Now for the two "moot problems": > - ZIP does not have good performance or compression ratio, there are > better algorithms out there like gzip (which also happens to be more > ubiquitous; nearly all websites are serving HTML compressed with gzip > compression). > - ZIP is an archiving format, that happens to have its own compression > format. Archiving format parsers can have serious vulnerabilities in their > implementation that can allow malware to swipe private keys and passwords, > since the primary target for this BIP is wallets. For example, there was > Zip Slip[1] in 2018, which allows for remote code execution. So the malware > can even hide in memory until private keys or passwords are written to > memory, then send them accros the network. Assuming it's targeting a > specific wallet software it's not hard to carry out at all. > > There's two solutions for all this: > 1. The duck-tape solution: Use some compression algorithm like gzip > instead of ZIP archive format. > 2. The "throw it out and buy a new one" solution: Get rid of the optional > compression specs altogether, because users are responsible for supplying > the export labels in the first place, so all the compression stuff is > redundant and should be left up to the user use if they desire to. > > I prefer the second solution because it hits the nail at the problem > directly instead of putting duck tape on it like the first one. > > > This .zip file may optionally be encrypted using either AES-128 > or > > AES-256 encryption, which is supported by numerous applications including > > Winzip and 7-zip. > > The textual representation of the wallet's extended public key (as > defined > > by BIP32, with an xpub header) should be used as the password. > Not specific to AES, but I don't see the benefit of encrypting addresses > and labels together. Can you please elaborate why this would be desireable? > > Like I said though, it's better to leave it up to users to decide how to > store their exports, since BIPs can't enforce that anyway (additionally, > the password you propose is insecure - anybody with access to the wallet > can unlock it, which is not desireable to some users who want their own > security). > > > * Transaction ID (txid) > > * Address > > * Input (rendered as txid) > > * Output (rendered as txid>index or txid:index) > Why the need for input and output formats? There is no difference between > them on the wallet level, because they are always identified with a txid > and output index. To distinguish between them and hence write them with the > correct format would require a UTXO set and thus access to a full node, > otherwise the CSV cannot be verified to be completely well-formed. > > Another important point is that practically nobody labels inputs or > outputs because most people do not know that those things even exist, and > the rest don't bother to label them. > > But the biggest downside to including them is related to the problem of > information leaking which you make reference to here: > > In both cases, care must be taken when spending to avoid undesirable > leaks > > of private information. > A CSV dump that has inputs/outputs and addresses mixed together can infer > the owner of all those items. In fact, A CVS label dump is basically a > personal information store so everything in it can be correlated as coming > from the same wallet, so it's important that unnecessary types are kept out > of the format. People are known to leave files lying around on their > computer that they don't need anymore, so these files can find their way > via telemetry to surveillence entities. While we can't specify what users > can do with their exports, we can control the information leak by > preventing certain types of items that we know most users will never use > from being exported in the first place. > > > The order in which these records appear is not defined. > Again, since the primary use case for this BIP is wallets, which likely > use heirarchical derivation schemes like BIP44, there is a net benefit for > the addresses to be exported in ascending order of their `address_type`. It > means that wallets can import them in O(n) time as opposed to O(n^2) time > spent serially checking in which index the address appears at. Of course, > this implies that all addresses up to a certain index have to be exported > into the CSV as well, but most wallets I know of like Core, Electrum > already store addresses like that. > > Also if you do this, you will need to group all the transaction records > before the address records or vice versa - you can use lexigraphical > sorting if you want (ie. Addresses before Transactions). The benefit of > this separation of parts is that wallets can split the imported address > records from the transaction records internally, and feed them to separate > functions which set these labels internally. > > If you decide on doing it this way, then you need a 3rd column to identify > the item type, and also you should quote the label (see below). I strongly > recommend using numbers for identification as opposed to character strings, > so you don't have to worry about localization or character case issues. > There is always one unique number, but there could be multiple strings that > reference the same type. This will complicate importing functions. > > If you insist on include Input and Output types then they can both be > specified as : if you do this change. They won't be used to > determine the type anyway. > > > The fields may be quoted, but this is unnecessary, as the first comma in > > the line will always be the delimiter. > Don't implement it like that, because that will break CSV parsers which > expect a fixed amount of rows in each record (2 in the header, and some > rows have >2 rows). It's better to mandate that they should always be > double-quoted, since only wallets will generate label exports anyway. If > you plan to use headers then the 3rd column can be blank for it (or you can > split the version and flags from each other). > > > ==Importing== > > > > When importing, a naive algorithm may simply match against any reference, > > but it is possible to disambiguate between transactions, addresses, > inputs > > and outputs. > > For example in the following pseudocode: > >
> >   if reference length < 64
> >     Set address label
> >   else if reference length == 64
> >     Set transaction label
> >   else if reference contains '<'
> >     Set input label
> >   else
> >     Set output label
> > 
> The importing code is too naive and in its current form will prevent the > BIP from getting a number. It is perhaps the single most important part of > a BIP. When implementing an importer, it should utilize a dedicate item > type field that unambiguously identifies the item. So the naive importer is > not good, you need use a 3rd column for that like I explained above, so > that the importer becomes robust. > > In summary (exclamation marks indicate severity - one means low, two means > medium, and three means high): > > 1. Convert the header into a version line with optional flags, otherwise > nobody can extend this format without compatibility issues (!) > 2. Get rid of the specs related to file compression (!!!) > 3. Add a 3rd column for item type (address, transaction etc.) preferably > as numeric constants and grouping items of one type after items of another > type, or if you insist on strings, then only recognize their Titlecase > ASCII versions the words> (!!) > 4. Require double quotes around the label (or single quotes if you prefer, > as long as spreadsheet software doesn't choke on them) (!!) > 5. Require sorting the records according to the order they are stored in > the wallet implementation. (!) > 6. Consider getting rid of Input and Output item types. (!) > 7. And last and most importantly, please write a more robust importer > algorithm in the example given by the BIP, because code in BIPs are > frequently used as references for software. (!!!) > > I hope you will consider these points in future revisions of your BIP. > > - Ali > > [1] https://github.com/snyk/zip-slip-vulnerability > > On Wed, 24 Aug 2022 11:18:43 +0200, craigraw@gmail.com wrote: > > Hi all, > > > > I would like to propose a BIP that specifies a format for the export and > > import of labels from a wallet. While transferring access to funds across > > wallet applications has been made simple through standards such as BIP39, > > wallet labels remain siloed and difficult to extract despite their value, > > particularly in a privacy context. > > > > The proposed format is a simple two column CSV file, with the reference > to > > a transaction, address, input or output in the first column, and the > label > > in the second column. CSV was chosen for its wide accessibility, > especially > > to users without specific technical expertise. Similarly, the CSV file > may > > be compressed using the ZIP format, and optionally encrypted using AES. > > > > The full text of the BIP can be found at > > https://github.com/craigraw/bips/blob/master/bip-wallet-labels.mediawiki > > and also copied below. > > > > Feedback is appreciated. > > > > Thanks, > > Craig Raw > > > > --- > > > >
> >   BIP: wallet-labels
> >   Layer: Applications
> >   Title: Wallet Labels Export Format
> >   Author: Craig Raw 
> >   Comments-Summary: No comments yet.
> >   Comments-URI:
> > https://github.com/bitcoin/bips/wiki/Comments:BIP-wallet-labels
> >   Status: Draft
> >   Type: Informational
> >   Created: 2022-08-23
> >   License: BSD-2-Clause
> > 
> > > > ==Abstract== > > > > This document specifies a format for the export of labels that may be > > attached to the transactions, addresses, input and outputs in a wallet. > > > > ==Copyright== > > > > This BIP is licensed under the BSD 2-clause license. > > > > ==Motivation== > > > > The export and import of funds across different Bitcoin wallet > applications > > is well defined through standards such as BIP39, BIP32, BIP44 etc. > > These standards are well supported and allow users to move easily between > > different wallets. > > There is, however, no defined standard to transfer any labels the user > may > > have applied to the transactions, addresses, inputs or outputs in their > > wallet. > > The UTXO model that Bitcoin uses makes these labels particularly valuable > > as they may indicate the source of funds, whether received externally or > as > > a result of change from a prior transaction. > > In both cases, care must be taken when spending to avoid undesirable > leaks > > of private information. > > Labels provide valuable guidance in this regard, and have even become > > mandatory when spending in several Bitcoin wallets. > > Allowing users to export their labels in a standardized way ensures that > > they do not experience lock-in to a particular wallet application. > > In addition, by using common formats, this BIP seeks to make manual or > bulk > > management of labels accessible to users without specific technical > > expertise. > > > > ==Specification== > > > > In order to make the import and export of labels as widely accessible as > > possible, this BIP uses the comma separated values (CSV) format, which is > > widely supported by consumer, business, and scientific applications. > > Although the technical specification of CSV in RFC4180 is not always > > followed, the application of the format in this BIP is simple enough that > > compatibility should not present a problem. > > Moreover, the simplicity and forgiving nature of CSV (over for example > > JSON) lends itself well to bulk label editing using spreadsheet and text > > editing tools. > > > > A CSV export of labels from a wallet must be a UTF-8 encoded text file, > > containing one record per line, with records containing two fields > > delimited by a comma. > > The fields may be quoted, but this is unnecessary, as the first comma in > > the line will always be the delimiter. > > The first line in the file is a header, and should be ignored on import. > > Thereafter, each line represents a record that refers to a label applied > in > > the wallet. > > The order in which these records appear is not defined. > > > > The first field in the record contains a reference to the transaction, > > address, input or output in the wallet. > > This is specified as one of the following: > > * Transaction ID (txid) > > * Address > > * Input (rendered as txid) > > * Output (rendered as txid>index or txid:index) > > > > The second field contains the label applied to the reference. > > Exporting applications may omit records with no labels or labels of zero > > length. > > Files exported should use the .csv file extension. > > > > In order to reduce file size while retaining wide accessibility, the CSV > > file may be compressed using the ZIP file format, using the .zip > > file extension. > > This .zip file may optionally be encrypted using either AES-128 > or > > AES-256 encryption, which is supported by numerous applications including > > Winzip and 7-zip. > > In order to ensure that weak encryption does not proliferate, importers > > following this standard must refuse to import .zip files > encrypted > > with the weaker Zip 2.0 standard. > > The textual representation of the wallet's extended public key (as > defined > > by BIP32, with an xpub header) should be used as the password. > > > > ==Importing== > > > > When importing, a naive algorithm may simply match against any reference, > > but it is possible to disambiguate between transactions, addresses, > inputs > > and outputs. > > For example in the following pseudocode: > >
> >   if reference length < 64
> >     Set address label
> >   else if reference length == 64
> >     Set transaction label
> >   else if reference contains '<'
> >     Set input label
> >   else
> >     Set output label
> > 
> > > > Importing applications may truncate labels if necessary. > > > > ==Test Vectors== > > > > The following fragment represents a wallet label export: > >
> > Reference,Label
> >
> c3bdad6e7dcd7997e16a5b7b7cf4d8f6079820ff2eedd5fcbb2ad088f767b37b?,Transaction
> > 1A69TXnEM2ms9fMaY9UuiJ7415X7xZaUSg,Address
> > c3bdad6e7dcd7997e16a5b7b7cf4d8f6079820ff2eedd5fcbb2ad088f767b37b?<0,Input
> >
> c3bdad6e7dcd7997e16a5b7b7cf4d8f6079820ff2eedd5fcbb2ad088f767b37b?>0,Output
> >
> c3bdad6e7dcd7997e16a5b7b7cf4d8f6079820ff2eedd5fcbb2ad088f767b37b?:0,Output
> > (alternative)
> > 
> > > > ==Reference Implementation== > > > > TBD > > --00000000000025ce9f05e75f859e Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
Thanks for your feedback @Ali.

I am att= empting to achieve=C2=A0two goals with this proposal, primarily for the ben= efit of wallet users:

Goal #1. Transfer labels bet= ween different wallet implementations
Goal #2. Manage labels in a= pplications outside=C2=A0of Bitcoin wallets (such as Excel)

<= /div>
Much of the feedback so far has indicated the tension between the= se two goals - it may be that it is too difficult to achieve both, in which= case Goal #1 is the most important. That said, I think further exploration= is still necessary before abandoning Goal #2, because removing it would si= gnificantly reduce the value of this proposal and mean users need to rely o= n application-specific workarounds.

> it is imp= ortant that a version byte is defined
If Goal #2 is to be achieve= d it's difficult to mandate this, particularly if one requires bit flag= s to be set. Should an importing wallet fail to import if the version byte = is not present, even if all the data is otherwise correct? Although it is d= ifficult to know in advance how a format may be extended, it is certainly p= ossible to extend this format with additional types where the nature of has= hes serve as unique identifiers (more on this below).

<= div>=C2=A0> Don't mandate the file extension... There is no way to e= nforce this on a BIP level.
I'm not quite sure what you mean = here - for example BIP174, which is widely used, states "Binary PSBT f= iles should use the .psbt file extension." Also, this contradicts Goal= #2 - Excel and Numbers register as handlers for .csv, and so make it clear= that the file is editable outside of a wallet.

&g= t; ZIP does not have good performance or compression ratio
Indeed= , but it is very widely available. That said, gzip is supported widely too = these days. Unfortunately, gzip does not offer encryption (see next answer)= .

> ZIP is an archiving format, that happens to= have its own compression format.
I agree this is not ideal. My m= ain reason for choosing ZIP was that it supports encryption. It seems to me= that without considering encryption, an application must create label expo= rt files that allow privacy-sensitive wallet information to be readable in = plain text. Being able to transfer labels without risking privacy is IMO va= luable. I considered other encryption formats such as PGP, but they are muc= h more niche and so again contradict Goal #2.

>= I don't see the benefit of encrypting addresses and labels together...= additionally, the password you propose is insecure - anybody with access t= o the wallet can unlock it
I'm not sure I understand your que= stion, but both wallet addresses and wallet labels contain privacy-sensitiv= e information that should be protected. Wrt to the password, there is actua= lly a more fundamental problem with using the wallet xpub - there is no=C2= =A0equivalent for multisig wallets. For this reason I'll remove that re= quirement in future iterations.

> Why the need = for input and output formats? There is no difference between them on the wa= llet level, because they are always identified with a txid and output index= .
The input refers to the txid and the input index (in the set of= vin), so the difference is the context in which they are displayed. A wall= et will not necessarily store the spent outputs for a funding transaction c= ontaining a UTXO coming into the wallet, but it will contain references to = the inputs as part of that transaction.=C2=A0

>= Another important point is that practically nobody labels inputs or output= s
To the contrary, UTXOs are very frequently labelled, as they li= nk and reveal information when spent. Inputs are much less frequently label= led, but there is no=C2=A0particular reason to exclude them.

=
> there is a net benefit for the addresses to be exported in = ascending order
Indeed, but it makes achieving=C2=A0Goal #2 much = more difficult for marginal benefit.

> It's= better to mandate that they should always be double-quoted, since only wal= lets will generate label exports anyway.
Rather I think it's = better to mandate RFC4180 is followed, as per recommendations in other feed= back.

> The importing code is too naive... it s= hould utilize a dedicate item type field that unambiguously identifies the = item
It's unclear to me what you mean here. As I've indic= ated it is currently possible to disambiguate between addresses/transaction= s/etc without the need for a 3rd column, but in any case the hash functions= used ensure that labels will not be associated incorrectly. Even in the un= likely event of some future address type being indistinguishable from a txi= d, it will simply not match any txids in the wallet.

Craig



On Wed, Aug 24, 2022 at 9:10 PM= Ali Sherief <ali@notatether.com> wrote:
Hi= Craig,

This a really good proposal. I studied your BIP and I have some feedback on= some parts of it.

> The first line in the file is a header, and should be ignored on impor= t.

>From past experience and lessons, most notably BIP39, it is important that = a version byte is defined somewhere in case someone wants to extend it in t= he future, currently there is no version byte which someone can increment i= f somebody wants to extend it. In the unique case of CSV files, you should = make the header line mandatory (I see you have already implied this, but yo= u should make it explicit in the BIP), but instead of a line with columns i= n it, I suggest instead of Reference,Label, you make the format like this:<= br>
BIP-wallet-labels,<version>

Since there are two columns per record, this works out nicely. The first co= lumn can be the name of the BIP - BIPxxxx where the x's are numbers, an= d the second column can be an unsigned 32-bit integer (most significant 8 b= its reserved for version, the remaining for flags, or perhaps the entirety = for version - but I recommend leaving at least some bits for flags, even if= they all end up being just "reserved").

You should make importing fail if the header line is not exactly as specifi= ed - or appropriate, should you decide a different format for the header.
> Files exported should use the <tt>.csv</tt> file extension= .
Don't mandate the file extension (read below for why):

> In order to reduce file size while retaining wide accessibility, the C= SV
> file may be compressed using the ZIP file format, using the <tt>= .zip</tt>
> file extension.
I see three problems with this. The first is more important than the later = two because it makes them moot points, but I'll mention them anyway so = you get a background of the situation:
- The BIP is trying to specify in what file format the export format can be= written in onto the filesystem. There is no way to enforce this on a BIP l= evel (besides, Unix operating systems don't even consider the file exte= nsion, they use its mimetype). Also specifying this in the BIP will prevent= modular "Layer 2" protocols and schemes from encoding the Export= labels into another format - for example Base64 or with their own compress= ion algorithm.

Now for the two "moot problems":
- ZIP does not have good performance or compression ratio, there are better= algorithms out there like gzip (which also happens to be more ubiquitous; = nearly all websites are serving HTML compressed with gzip compression).
- ZIP is an archiving format, that happens to have its own compression form= at. Archiving format parsers can have serious vulnerabilities in their impl= ementation that can allow malware to swipe private keys and passwords, sinc= e the primary target for this BIP is wallets. For example, there was Zip Sl= ip[1] in 2018, which allows for remote code execution. So the malware can e= ven hide in memory until private keys or passwords are written to memory, t= hen send them accros the network. Assuming it's targeting a specific wa= llet software it's not hard to carry out at all.

There's two solutions for all this:
1. The duck-tape solution: Use some compression algorithm like gzip instead= of ZIP archive format.
2. The "throw it out and buy a new one" solution: Get rid of the = optional compression specs altogether, because users are responsible for su= pplying the export labels in the first place, so all the compression stuff = is redundant and should be left up to the user use if they desire to.

I prefer the second solution because it hits the nail at the problem direct= ly instead of putting duck tape on it like the first one.

> This <tt>.zip</tt> file may optionally be encrypted using = either AES-128 or
> AES-256 encryption, which is supported by numerous applications includ= ing
> Winzip and 7-zip.
> The textual representation of the wallet's extended public key (as= defined
> by BIP32, with an <tt>xpub</tt> header) should be used as = the password.
Not specific to AES, but I don't see the benefit of encrypting addresse= s and labels together. Can you please elaborate why this would be desireabl= e?

Like I said though, it's better to leave it up to users to decide how t= o store their exports, since BIPs can't enforce that anyway (additional= ly, the password you propose is insecure - anybody with access to the walle= t can unlock it, which is not desireable to some users who want their own s= ecurity).

> * Transaction ID (<tt>txid</tt>)
> * Address
> * Input (rendered as <tt>txid<index</tt>)
> * Output (rendered as <tt>txid>index</tt> or <tt>= txid:index</tt>)
Why the need for input and output formats? There is no difference between t= hem on the wallet level, because they are always identified with a txid and= output index. To distinguish between them and hence write them with the co= rrect format would require a UTXO set and thus access to a full node, other= wise the CSV cannot be verified to be completely well-formed.

Another important point is that practically nobody labels inputs or outputs= because most people do not know that those things even exist, and the rest= don't bother to label them.

But the biggest downside to including them is related to the problem of inf= ormation leaking which you make reference to here:
> In both cases, care must be taken when spending to avoid undesirable l= eaks
> of private information.
A CSV dump that has inputs/outputs and addresses mixed together can infer t= he owner of all those items. In fact, A CVS label dump is basically a perso= nal information store so everything in it can be correlated as coming from = the same wallet, so it's important that unnecessary types are kept out = of the format. People are known to leave files lying around on their comput= er that they don't need anymore, so these files can find their way via = telemetry to surveillence entities. While we can't specify what users c= an do with their exports, we can control the information leak by preventing= certain types of items that we know most users will never use from being e= xported in the first place.

> The order in which these records appear is not defined.
Again, since the primary use case for this BIP is wallets, which likely use= heirarchical derivation schemes like BIP44, there is a net benefit for the= addresses to be exported in ascending order of their `address_type`. It me= ans that wallets can import them in O(n) time as opposed to O(n^2) time spe= nt serially checking in which index the address appears at. Of course, this= implies that all addresses up to a certain index have to be exported into = the CSV as well, but most wallets I know of like Core, Electrum already sto= re addresses like that.

Also if you do this, you will need to group all the transaction records bef= ore the address records or vice versa - you can use lexigraphical sorting i= f you want (ie. Addresses before Transactions). The benefit of this separat= ion of parts is that wallets can split the imported address records from th= e transaction records internally, and feed them to separate functions which= set these labels internally.

If you decide on doing it this way, then you need a 3rd column to identify = the item type, and also you should quote the label (see below). I strongly = recommend using numbers for identification as opposed to character strings,= so you don't have to worry about localization or character case issues= . There is always one unique number, but there could be multiple strings th= at reference the same type. This will complicate importing functions.

If you insist on include Input and Output types then they can both be speci= fied as <txid>:<index> if you do this change. They won't be= used to determine the type anyway.

> The fields may be quoted, but this is unnecessary, as the first comma = in
> the line will always be the delimiter.
Don't implement it like that, because that will break CSV parsers which= expect a fixed amount of rows in each record (2 in the header, and some ro= ws have >2 rows). It's better to mandate that they should always be = double-quoted, since only wallets will generate label exports anyway. If yo= u plan to use headers then the 3rd column can be blank for it (or you can s= plit the version and flags from each other).

> =3D=3DImporting=3D=3D
>
> When importing, a naive algorithm may simply match against any referen= ce,
> but it is possible to disambiguate between transactions, addresses, in= puts
> and outputs.
> For example in the following pseudocode:
> <pre>
>=C2=A0 =C2=A0if reference length < 64
>=C2=A0 =C2=A0 =C2=A0Set address label
>=C2=A0 =C2=A0else if reference length =3D=3D 64
>=C2=A0 =C2=A0 =C2=A0Set transaction label
>=C2=A0 =C2=A0else if reference contains '<'
>=C2=A0 =C2=A0 =C2=A0Set input label
>=C2=A0 =C2=A0else
>=C2=A0 =C2=A0 =C2=A0Set output label
> </pre>
The importing code is too naive and in its current form will prevent the BI= P from getting a number. It is perhaps the single most important part of a = BIP. When implementing an importer, it should utilize a dedicate item type = field that unambiguously identifies the item. So the naive importer is not = good, you need use a 3rd column for that like I explained above, so that th= e importer becomes robust.

In summary (exclamation marks indicate severity - one means low, two means = medium, and three means high):

1. Convert the header into a version line with optional flags, otherwise no= body can extend this format without compatibility issues (!)
2. Get rid of the specs related to file compression (!!!)
3. Add a 3rd column for item type (address, transaction etc.) preferably as= numeric constants and grouping items of one type after items of another ty= pe, or if you insist on strings, then only recognize their Titlecase ASCII = versions <spreadsheet software like Excel always tries to titlecase the = words> (!!)
4. Require double quotes around the label (or single quotes if you prefer, = as long as spreadsheet software doesn't choke on them) (!!)
5. Require sorting the records according to the order they are stored in th= e wallet implementation. (!)
6. Consider getting rid of Input and Output item types. (!)
7. And last and most importantly, please write a more robust importer algor= ithm in the example given by the BIP, because code in BIPs are frequently u= sed as references for software. (!!!)

I hope you will consider these points in future revisions of your BIP.

- Ali

[1]
https://github.com/snyk/zip-slip-vulnerability=

On Wed, 24 Aug 2022 11:18:43 +0200, craigraw@gmail.com wrote:
> Hi all,
>
> I would like to propose a BIP that specifies a format for the export a= nd
> import of labels from a wallet. While transferring access to funds acr= oss
> wallet applications has been made simple through standards such as BIP= 39,
> wallet labels remain siloed and difficult to extract despite their val= ue,
> particularly in a privacy context.
>
> The proposed format is a simple two column CSV file, with the referenc= e to
> a transaction, address, input or output in the first column, and the l= abel
> in the second column. CSV was chosen for its wide accessibility, espec= ially
> to users without specific technical expertise. Similarly, the CSV file= may
> be compressed using the ZIP format, and optionally encrypted using AES= .
>
> The full text of the BIP can be found at
> https://github.com/crai= graw/bips/blob/master/bip-wallet-labels.mediawiki
> and also copied below.
>
> Feedback is appreciated.
>
> Thanks,
> Craig Raw
>
> ---
>
> <pre>
>=C2=A0 =C2=A0BIP: wallet-labels
>=C2=A0 =C2=A0Layer: Applications
>=C2=A0 =C2=A0Title: Wallet Labels Export Format
>=C2=A0 =C2=A0Author: Craig Raw <craig@sparrowwallet.com>
>=C2=A0 =C2=A0Comments-Summary: No comments yet.
>=C2=A0 =C2=A0Comments-URI:
> https://github.com/bitcoin/bips/= wiki/Comments:BIP-wallet-labels
>=C2=A0 =C2=A0Status: Draft
>=C2=A0 =C2=A0Type: Informational
>=C2=A0 =C2=A0Created: 2022-08-23
>=C2=A0 =C2=A0License: BSD-2-Clause
> </pre>
>
> =3D=3DAbstract=3D=3D
>
> This document specifies a format for the export of labels that may be<= br> > attached to the transactions, addresses, input and outputs in a wallet= .
>
> =3D=3DCopyright=3D=3D
>
> This BIP is licensed under the BSD 2-clause license.
>
> =3D=3DMotivation=3D=3D
>
> The export and import of funds across different Bitcoin wallet applica= tions
> is well defined through standards such as BIP39, BIP32, BIP44 etc.
> These standards are well supported and allow users to move easily betw= een
> different wallets.
> There is, however, no defined standard to transfer any labels the user= may
> have applied to the transactions, addresses, inputs or outputs in thei= r
> wallet.
> The UTXO model that Bitcoin uses makes these labels particularly valua= ble
> as they may indicate the source of funds, whether received externally = or as
> a result of change from a prior transaction.
> In both cases, care must be taken when spending to avoid undesirable l= eaks
> of private information.
> Labels provide valuable guidance in this regard, and have even become<= br> > mandatory when spending in several Bitcoin wallets.
> Allowing users to export their labels in a standardized way ensures th= at
> they do not experience lock-in to a particular wallet application.
> In addition, by using common formats, this BIP seeks to make manual or= bulk
> management of labels accessible to users without specific technical > expertise.
>
> =3D=3DSpecification=3D=3D
>
> In order to make the import and export of labels as widely accessible = as
> possible, this BIP uses the comma separated values (CSV) format, which= is
> widely supported by consumer, business, and scientific applications. > Although the technical specification of CSV in RFC4180 is not always > followed, the application of the format in this BIP is simple enough t= hat
> compatibility should not present a problem.
> Moreover, the simplicity and forgiving nature of CSV (over for example=
> JSON) lends itself well to bulk label editing using spreadsheet and te= xt
> editing tools.
>
> A CSV export of labels from a wallet must be a UTF-8 encoded text file= ,
> containing one record per line, with records containing two fields
> delimited by a comma.
> The fields may be quoted, but this is unnecessary, as the first comma = in
> the line will always be the delimiter.
> The first line in the file is a header, and should be ignored on impor= t.
> Thereafter, each line represents a record that refers to a label appli= ed in
> the wallet.
> The order in which these records appear is not defined.
>
> The first field in the record contains a reference to the transaction,=
> address, input or output in the wallet.
> This is specified as one of the following:
> * Transaction ID (<tt>txid</tt>)
> * Address
> * Input (rendered as <tt>txid<index</tt>)
> * Output (rendered as <tt>txid>index</tt> or <tt>= txid:index</tt>)
>
> The second field contains the label applied to the reference.
> Exporting applications may omit records with no labels or labels of ze= ro
> length.
> Files exported should use the <tt>.csv</tt> file extension= .
>
> In order to reduce file size while retaining wide accessibility, the C= SV
> file may be compressed using the ZIP file format, using the <tt>= .zip</tt>
> file extension.
> This <tt>.zip</tt> file may optionally be encrypted using = either AES-128 or
> AES-256 encryption, which is supported by numerous applications includ= ing
> Winzip and 7-zip.
> In order to ensure that weak encryption does not proliferate, importer= s
> following this standard must refuse to import <tt>.zip</tt>= ; files encrypted
> with the weaker Zip 2.0 standard.
> The textual representation of the wallet's extended public key (as= defined
> by BIP32, with an <tt>xpub</tt> header) should be used as = the password.
>
> =3D=3DImporting=3D=3D
>
> When importing, a naive algorithm may simply match against any referen= ce,
> but it is possible to disambiguate between transactions, addresses, in= puts
> and outputs.
> For example in the following pseudocode:
> <pre>
>=C2=A0 =C2=A0if reference length < 64
>=C2=A0 =C2=A0 =C2=A0Set address label
>=C2=A0 =C2=A0else if reference length =3D=3D 64
>=C2=A0 =C2=A0 =C2=A0Set transaction label
>=C2=A0 =C2=A0else if reference contains '<'
>=C2=A0 =C2=A0 =C2=A0Set input label
>=C2=A0 =C2=A0else
>=C2=A0 =C2=A0 =C2=A0Set output label
> </pre>
>
> Importing applications may truncate labels if necessary.
>
> =3D=3DTest Vectors=3D=3D
>
> The following fragment represents a wallet label export:
> <pre>
> Reference,Label
> c3bdad6e7dcd7997e16a5b7b7cf4d8f6079820ff2eedd5fcbb2ad088f767b37b?,Tran= saction
> 1A69TXnEM2ms9fMaY9UuiJ7415X7xZaUSg,Address
> c3bdad6e7dcd7997e16a5b7b7cf4d8f6079820ff2eedd5fcbb2ad088f767b37b?<0= ,Input
> c3bdad6e7dcd7997e16a5b7b7cf4d8f6079820ff2eedd5fcbb2ad088f767b37b?>0= ,Output
> c3bdad6e7dcd7997e16a5b7b7cf4d8f6079820ff2eedd5fcbb2ad088f767b37b?:0,Ou= tput
> (alternative)
> </pre>
>
> =3D=3DReference Implementation=3D=3D
>
> TBD

--00000000000025ce9f05e75f859e--