From f8ab117c9f537a06f832a2d244ca1cababc6776d Mon Sep 17 00:00:00 2001
From: Jack Grigg <jack@electriccoin.co>
Date: Tue, 16 Jul 2024 04:31:07 +0000
Subject: [PATCH] Render manually, while the CI-based rendering is broken

---
 README.rst    |   2 +-
 index.html    |   2 +-
 zip-0173.html |   5 +
 zip-0315.html | 407 +++++++++++++++++++++++++++++++++++++++++++++++++-
 4 files changed, 410 insertions(+), 6 deletions(-)

diff --git a/README.rst b/README.rst
index 313b78be..920e58e6 100644
--- a/README.rst
+++ b/README.rst
@@ -115,7 +115,7 @@ Index of ZIPs
     <tr> <td><span class="reserved">312</span></td> <td class="left"><a class="reserved" href="zip-0312.rst">Shielded Multisignatures using FROST</a></td> <td>Reserved</td>
     <tr> <td><strike>313</strike></td> <td class="left"><strike><a href="zip-0313.rst">Reduce Conventional Transaction Fee to 1000 zatoshis</a></strike></td> <td>Obsolete</td>
     <tr> <td><span class="reserved">314</span></td> <td class="left"><a class="reserved" href="zip-0314.rst">Privacy upgrades to the Zcash light client protocol</a></td> <td>Reserved</td>
-    <tr> <td><span class="reserved">315</span></td> <td class="left"><a class="reserved" href="zip-0315.rst">Best Practices for Wallet Handling of Multiple Pools</a></td> <td>Reserved</td>
+    <tr> <td>315</td> <td class="left"><a href="zip-0315.rst">Best Practices for Wallet Implementations</a></td> <td>Draft</td>
     <tr> <td>316</td> <td class="left"><a href="zip-0316.rst">Unified Addresses and Unified Viewing Keys</a></td> <td>Revision 0: Final, Revision 1: Proposed</td>
     <tr> <td>317</td> <td class="left"><a href="zip-0317.rst">Proportional Transfer Fee Mechanism</a></td> <td>Active</td>
     <tr> <td><span class="reserved">318</span></td> <td class="left"><a class="reserved" href="zip-0318.rst">Associated Payload Encryption</a></td> <td>Reserved</td>
diff --git a/index.html b/index.html
index c86c21f9..fb400c4e 100644
--- a/index.html
+++ b/index.html
@@ -90,7 +90,7 @@
   <tr> <td><span class="reserved">312</span></td> <td class="left"><a class="reserved" href="zip-0312">Shielded Multisignatures using FROST</a></td> <td>Reserved</td>
   <tr> <td><strike>313</strike></td> <td class="left"><strike><a href="zip-0313">Reduce Conventional Transaction Fee to 1000 zatoshis</a></strike></td> <td>Obsolete</td>
   <tr> <td><span class="reserved">314</span></td> <td class="left"><a class="reserved" href="zip-0314">Privacy upgrades to the Zcash light client protocol</a></td> <td>Reserved</td>
-  <tr> <td><span class="reserved">315</span></td> <td class="left"><a class="reserved" href="zip-0315">Best Practices for Wallet Handling of Multiple Pools</a></td> <td>Reserved</td>
+  <tr> <td>315</td> <td class="left"><a href="zip-0315">Best Practices for Wallet Implementations</a></td> <td>Draft</td>
   <tr> <td>316</td> <td class="left"><a href="zip-0316">Unified Addresses and Unified Viewing Keys</a></td> <td>Revision 0: Final, Revision 1: Proposed</td>
   <tr> <td>317</td> <td class="left"><a href="zip-0317">Proportional Transfer Fee Mechanism</a></td> <td>Active</td>
   <tr> <td><span class="reserved">318</span></td> <td class="left"><a class="reserved" href="zip-0318">Associated Payload Encryption</a></td> <td>Reserved</td>
diff --git a/zip-0173.html b/zip-0173.html
index 78829305..52e120c7 100644
--- a/zip-0173.html
+++ b/zip-0173.html
@@ -193,6 +193,11 @@ def bech32_verify_checksum(hrp, data):
                         <li><a href="https://github.com/zcash/zcash/blob/master/src/bech32.cpp">In C++</a></li>
                     </ul>
                 </li>
+                <li>Encoders and decoders written specifically for Zcash:
+                    <ul>
+                        <li><a href="https://github.com/nerdcash/Nerdbank.Cryptocurrencies/blob/main/src/Nerdbank.Cryptocurrencies/Bech32.cs">For C#</a> (Bech32 and Bech32m)</li>
+                    </ul>
+                </li>
                 <li>Encoders and decoders written for Bitcoin:
                     <ul>
                         <li><a href="https://github.com/sipa/bech32/tree/master/ref/c">For C</a></li>
diff --git a/zip-0315.html b/zip-0315.html
index 810c1900..380f111d 100644
--- a/zip-0315.html
+++ b/zip-0315.html
@@ -1,18 +1,417 @@
 <!DOCTYPE html>
 <html>
 <head>
-    <title>ZIP 315: Best Practices for Wallet Handling of Multiple Pools</title>
+    <title>ZIP 315: Best Practices for Wallet Implementations</title>
     <meta charset="utf-8" />
 <meta name="viewport" content="width=device-width, initial-scale=1"><link rel="stylesheet" href="css/style.css"></head>
 <body>
     <section>
         <pre>ZIP: 315
-Title: Best Practices for Wallet Handling of Multiple Pools
+Title: Best Practices for Wallet Implementations
 Owners: Daira-Emma Hopwood &lt;daira-emma@electriccoin.co&gt;
         Jack Grigg &lt;jack@electriccoin.co&gt;
-Status: Reserved
+        Kris Nuttycombe &lt;kris@electriccoin.co&gt;
+Credits: Francisco Gindre
+Status: Draft
 Category: Wallet
-Discussions-To: &lt;<a href="https://github.com/zcash/zips/issues/447">https://github.com/zcash/zips/issues/447</a>&gt;</pre>
+Discussions-To: &lt;<a href="https://github.com/zcash/zips/issues/447">https://github.com/zcash/zips/issues/447</a>&gt;
+Pull-Request: &lt;<a href="https://github.com/zcash/zips/pull/607">https://github.com/zcash/zips/pull/607</a>&gt;</pre>
+        <section id="terminology"><h2><span class="section-heading">Terminology</span><span class="section-anchor"> <a rel="bookmark" href="#terminology"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h2>
+            <p>The key words "MUST", "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", and "MAY" in this document are to be interpreted as described in BCP 14 <a id="footnote-reference-1" class="footnote_reference" href="#bcp14">1</a> when, and only when, they appear in all capitals.</p>
+            <p>The terms below are to be interpreted as follows:</p>
+            <dl>
+                <dt>Auto-shielding</dt>
+                <dd>The process of automatically transferring transparent funds to the most recent shielded pool on receipt.</dd>
+                <dt>Auto-migration</dt>
+                <dd>The process of automatically transferring shielded funds from older pools to the most preferred (usually the most recent) pool on receipt.</dd>
+                <dt>Opportunistic migration</dt>
+                <dd>The process of transferring previously received shielded funds from older pools to the most preferred (usually the most recent) pool as part of a user-initiated transaction.</dd>
+                <dt>Transaction output (TXO)</dt>
+                <dd>An output (transparent coin or shielded note) of a transaction on the consensus block chain or in the mempool visible to a wallet.</dd>
+            </dl>
+            <p>TODO: Add informal definitions of: * known-spendable and confirmed-spendable TXOs; * trusted and untrusted TXOs.</p>
+            <p>These should forward-reference the specification section with the formal definitions.</p>
+        </section>
+        <section id="motivation"><h2><span class="section-heading">Motivation</span><span class="section-anchor"> <a rel="bookmark" href="#motivation"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h2>
+            <p>Zcash wallets have to serve two purposes as a user agent:</p>
+            <ul>
+                <li>to manage the economic actions dictated by the user;</li>
+                <li>to manage the privacy implications of these actions.</li>
+            </ul>
+            <p>The goal of this ZIP is to document security and privacy best practices when handling funds and transactions as the user's agent. These best practices are intended to provide as much privacy as is feasible by default, while still enabling the user's desired transactions to occur, and exposing any privacy implications to the user so that they have enough information to assess the consequences of their economic actions.</p>
+            <p>This ZIP covers best practices for:</p>
+            <ul>
+                <li>what information to display to the user about transactions and balances;</li>
+                <li>how to handle interactions between the ZIP 32 <a id="footnote-reference-2" class="problematic" href="#system-message-2">[#zip-0032]_</a> key tree and Unified Addresses <a id="footnote-reference-3" class="problematic" href="#system-message-3">[#zip-0316]_</a>;</li>
+                <li>when to use external or internal keys/addresses;</li>
+                <li>sharing addresses and viewing keys;</li>
+                <li>sending and receiving funds (including fee selection);</li>
+                <li>migrating funds between pools.</li>
+            </ul>
+        </section>
+        <section id="requirements"><h2><span class="section-heading">Requirements</span><span class="section-anchor"> <a rel="bookmark" href="#requirements"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h2>
+            <section id="user-consent"><h3><span class="section-heading">User consent</span><span class="section-anchor"> <a rel="bookmark" href="#user-consent"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>A wallet typically reveals some information in the process of creating a transaction. Which information is revealed depends on the configured wallet privacy policy. The guiding principle of these requirements is that users must explicitly consent to each instance of information being revealed by the wallet in a transaction.</p>
+                <p>A user may give blanket consent to reveal a particular kind of information, and must also be able to change the configured wallet privacy policy to avoid the wallet creating new information leaks of a given type.</p>
+                <p>The specifications below describe some situations in which blanket consent may be inappropriate.</p>
+                <p>Some varieties of consent may not be revocable, for example if a user chooses to share some of their keys.</p>
+            </section>
+            <section id="prompt-accessibility-of-funds"><h3><span class="section-heading">Prompt accessibility of funds</span><span class="section-anchor"> <a rel="bookmark" href="#prompt-accessibility-of-funds"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>Wallets need to take account of two main concerns with respect to accessibility of funds:</p>
+                <ul>
+                    <li>enabling funds to be spent as quickly as possible to reduce latency;</li>
+                    <li>ensuring that the confirmed-spendable balance is not overestimated, and so can be trusted by the user.</li>
+                </ul>
+                <p>These concerns affect the way that balances should be computed, which notes are selected for spending, and how the wallet should ensure that sufficient notes are available to cover multiple spends within a short period.</p>
+                <p>To support this we define two kinds of TXOs:</p>
+                <ul>
+                    <li>A trusted TXO is a TXO received from a party where the wallet trusts that it will remain mined in its original transaction, such as TXOs created by the wallet's internal TXO handling.</li>
+                    <li>An untrusted TXO is a TXO received by the wallet that is not trusted.</li>
+                </ul>
+                <p>Wallets can then require that untrusted TXOs need more confirmations before they become confirmed-spendable than trusted TXOs. This provides an improved trade-off between latency on the one hand, and reliability and safety on the other.</p>
+            </section>
+        </section>
+        <section id="specification"><h2><span class="section-heading">Specification</span><span class="section-anchor"> <a rel="bookmark" href="#specification"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h2>
+            <section id="long-term-storage-of-funds"><h3><span class="section-heading">Long-term storage of funds</span><span class="section-anchor"> <a rel="bookmark" href="#long-term-storage-of-funds"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>It is RECOMMENDED that wallets only hold funds as shielded in the long term; that is, if a wallet supports receiving transparent funds (or supports importing a seed from another wallet that might have done so), then it SHOULD auto-shield such funds by default.</p>
+                <p>A shielding transaction is always linked to the transparent addresses it spends from. This can cause undesirable information leaks: a) if there are multiple transparent addresses, they will be linked to each</p>
+                <div>
+                    <h1>System Message: ERROR/3 (zip-0315.rst line 136)</h1>
+                    <p>Unexpected indentation.</p>
+                </div>
+                <blockquote>
+                    <p>other;</p>
+                </blockquote>
+                <div>
+                    <h1>System Message: WARNING/2 (zip-0315.rst line 137)</h1>
+                    <p>Block quote ends without a blank line; unexpected unindent.</p>
+                </div>
+                <ol suffix=")" start="2" type="a">
+                    <li>a link between the input transparent address(es) and the payment will be visible to the recipient(s), or to any other holder of an Incoming Viewing Key for the destination address(es).</li>
+                </ol>
+                <div>
+                    <h1>System Message: INFO/1 (zip-0315.rst line 123)</h1>
+                    <p>Enumerated list start value not ordinal-1: "b" (ordinal 2)</p>
+                </div>
+                <p>Despite the fact that it is not possible to achieve strong privacy guarantees from <em>any</em> use of transparent addresses, it is undesirable to reveal this additional information. In particular, issue b) motivates ruling out the use of "opportunistic shielding", i.e. shielding previously received transparent funds as part of a user-initiated transaction.</p>
+                <p>Wallets SHOULD NOT auto-shield from multiple transparent addresses in the same transaction, and SHOULD NOT use opportunistic shielding.</p>
+                <p>Following all of the recommendations in this section improves both collective privacy and the user's individual privacy, by maximizing the size of the note anonymity set over time and minimizing the potential linkage of shielding transactions to other transactions.</p>
+                <p>The remainder of this specification assumes a wallet that follows all of these recommendations, except where explicitly noted.</p>
+                <p>A wallet MAY allow users to disable auto-shielding, auto-migration, and/or opportunistic migration. If it does so, this need not be via three independent settings.</p>
+                <p>Automatic shielding and automatic/opportunistic migration SHOULD NOT be applied to inputs where the cost of shielding or migrating them will exceed their economic value. If these transactions are paying the ZIP 317 conventional fee <a id="footnote-reference-4" class="footnote_reference" href="#zip-0317">3</a>, that will be the case if the amount of the UTXO to be shielded/migrated exceeds the marginal fee, and cannot be accommodated by an input that would be present in any case due to padding of the number of inputs from a given shielded pool.</p>
+            </section>
+            <section id="trusted-and-untrusted-txos"><h3><span class="section-heading">Trusted and untrusted TXOs</span><span class="section-anchor"> <a rel="bookmark" href="#trusted-and-untrusted-txos"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>A wallet SHOULD treat received TXOs that are outputs of transactions created by the same wallet, as trusted TXOs. Wallets MAY enable users to mark specific external transactions as trusted, allowing their received TXOs also to be classified as trusted TXOs.</p>
+                <p>A wallet SHOULD have a policy that is clearly communicated to the user for the number of confirmations needed to spend untrusted and trusted TXOs respectively. The following confirmation policy is RECOMMENDED:</p>
+                <ul>
+                    <li>10 confirmations, for untrusted TXOs;</li>
+                    <li>3 confirmations, for trusted TXOs.</li>
+                </ul>
+                <section id="rationale-for-the-given-numbers-of-confirmations"><h4><span class="section-heading">Rationale for the given numbers of confirmations</span><span class="section-anchor"> <a rel="bookmark" href="#rationale-for-the-given-numbers-of-confirmations"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <p>The rationale for choosing three confirmations for trusted TXOs is that anecdotally, reorgs are usually less than three blocks.</p>
+                    <p>The consequences of attempting to spend a trusted TXO may be less severe in the case of a rollback than the consequences of attempting to spend an untrusted TXO. The value received from a trusted TXO should always be recoverable, whereas recovering value received from an untrusted TXO may require the user to request that funds are re-sent.</p>
+                </section>
+            </section>
+            <section id="categories-of-txos-according-to-spendability"><h3><span class="section-heading">Categories of TXOs according to spendability</span><span class="section-anchor"> <a rel="bookmark" href="#categories-of-txos-according-to-spendability"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>A TXO is <em>known-spendable</em>, relative to a given block chain and wallet state, if and only if all of the following are true in that state:</p>
+                <ul>
+                    <li>the TXO is unspent at the wallet's view of the chain tip, and that view is reasonably up-to-date;
+                        <p>TODO: consider undoing the up-to-date part, as when combined with the definition of balance below, it causes wallet balance to drop to zero in the short window between opening and updating the wallet's chain tip view.</p>
+                    </li>
+                    <li>the TXO is not committed to be spent in another transaction created by this wallet; and</li>
+                    <li>the wallet has the TXO's spending key (for whatever protocol the TXO uses).</li>
+                </ul>
+                <p>A TXO is <em>confirmed-spendable</em>, relative to a given block chain and wallet state, if and only if all of the following are true in that state:</p>
+                <ul>
+                    <li>the wallet is synchronized; and</li>
+                    <li>the TXO is known-spendable; and</li>
+                    <li>either auto-shielding is not active or the TXO is shielded; and</li>
+                    <li>auto-migration <em>from</em> whatever protocol the TXO uses is not active; and</li>
+                    <li>the TXO is trusted and has at least the required confirmations for trusted TXOs, or it is untrusted and has at least the required confirmations for untrusted TXOs.</li>
+                </ul>
+                <p>A TXO is <em>unconfirmed-spendable</em>, relative to a given block chain and wallet state, if and only if the TXO is known-spendable but is not confirmed-spendable in that state.</p>
+                <p>A TXO is <em>watch-only</em> if and only if the wallet has its full viewing key (or address in the case of a transparent TXO) but not its spending key.</p>
+                <p>A wallet MUST NOT attempt to spend a TXO in a user-initiated transaction that is not confirmed-spendable.</p>
+                <p>Open question: what should be specified about which TXOs can be spent in non-user-initiated transactions?</p>
+                <p>Note: the definition of a TXO includes outputs in mempool transactions that are unconflicted from the perspective of the wallet.</p>
+            </section>
+            <section id="reporting-of-balances"><h3><span class="section-heading">Reporting of balances</span><span class="section-anchor"> <a rel="bookmark" href="#reporting-of-balances"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>Wallets SHOULD report:</p>
+                <ul>
+                    <li>Confirmed-spendable balance.</li>
+                    <li>Pending balance, <em>or</em> total balance.</li>
+                </ul>
+                <p>These are calculated as follows:</p>
+                <ul>
+                    <li>The confirmed-spendable balance is the sum of values of confirmed-spendable TXOs.</li>
+                    <li>The pending balance is the sum of values of unconfirmed-spendable TXOs.</li>
+                    <li>The total balance is the confirmed-spendable balance plus the pending balance.</li>
+                </ul>
+                <p>Note: the definition of "confirmed-spendable" above ensures that:</p>
+                <ul>
+                    <li>if auto-shielding is enabled, transparent funds will be reported in the pending or total balance, but not in the confirmed-spendable balance;</li>
+                    <li>if the wallet has not synchronized at least the nullifier set to the chain tip, the confirmed-spendable balance will be zero.</li>
+                </ul>
+                <p>If auto-shielding is disabled, the wallet MAY report shielded and transparent balances separately. If it does so, it MUST make clear whether each reported balance corresponds to a confirmed-spendable, pending, or total subset of funds.</p>
+                <section id="rationale-for-reporting-of-balances"><h4><span class="section-heading">Rationale for reporting of balances</span><span class="section-anchor"> <a rel="bookmark" href="#rationale-for-reporting-of-balances"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <p>If auto-shielding is disabled, then separate shielded and transparent balances can constitute useful information. If auto-shielding is enabled then the wallet can and will automatically spend transparent TXOs in order to shield them, and so transparent TXOs need to be presented as pending, not as part of the balance spendable by the user.</p>
+                    <p>Potentially, for expert users, separate shielded balances per pool could also be useful.</p>
+                    <p>Open question: Does the specification of balance reporting give the user sufficient visibility into the operation of auto-shielding and pool migration/usage?</p>
+                </section>
+            </section>
+            <section id="reporting-of-transactions"><h3><span class="section-heading">Reporting of transactions</span><span class="section-anchor"> <a rel="bookmark" href="#reporting-of-transactions"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>If a transaction includes watch-only received TXOs, its watch-only incoming balance MUST be reported separately to any potentially known-spendable balance.</p>
+                <section id="incoming-transactions"><h4><span class="section-heading">Incoming transactions</span><span class="section-anchor"> <a rel="bookmark" href="#incoming-transactions"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <p>A transaction is incoming if it contains unconfirmed-spendable TXOs. Incoming transactions SHOULD be reported with their number of confirmations and their balances as described in <a href="#reporting-of-balances">Reporting of balances</a>.</p>
+                </section>
+                <section id="sent-transactions"><h4><span class="section-heading">Sent transactions</span><span class="section-anchor"> <a rel="bookmark" href="#sent-transactions"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <p>A transaction is sent if it was either: * created by the wallet, or * detected by using the wallet's outgoing viewing keys to decrypt</p>
+                    <div>
+                        <h1>System Message: ERROR/3 (zip-0315.rst line 308)</h1>
+                        <p>Unexpected indentation.</p>
+                    </div>
+                    <blockquote>
+                        <p>Sapling or Orchard outputs, or</p>
+                    </blockquote>
+                    <div>
+                        <h1>System Message: WARNING/2 (zip-0315.rst line 309)</h1>
+                        <p>Block quote ends without a blank line; unexpected unindent.</p>
+                    </div>
+                    <ul>
+                        <li>detected as spending a note that was at some time held by the wallet by recognizing that note's nullifier, or</li>
+                        <li>detected as spending a transparent TXO associated with one of the wallet's addresses (including watch-only addresses).</li>
+                    </ul>
+                    <p>Sent transactions SHOULD be reported with their number of confirmations, an estimate of how long until they expire (if they are unmined and have an expiry height), and their balances as described in <a href="#reporting-of-balances">Reporting of balances</a>.</p>
+                </section>
+            </section>
+            <section id="transaction-creation"><h3><span class="section-heading">Transaction creation</span><span class="section-anchor"> <a rel="bookmark" href="#transaction-creation"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <section id="obtaining-user-consent-for-information-leakage"><h4><span class="section-heading">Obtaining user consent for information leakage</span><span class="section-anchor"> <a rel="bookmark" href="#obtaining-user-consent-for-information-leakage"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <section id="information-leakage-analysis"><h5><span class="section-heading">Information leakage analysis</span><span class="section-anchor"> <a rel="bookmark" href="#information-leakage-analysis"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h5>
+                        <p>The privacy provided by a Zcash transaction depends on the information leaked in the creation of that transaction and the process of it being broadcast for inclusion in the block chain.</p>
+                        <p>The requirements in this section are intended to minimize the leakage of such information where possible, and to ensure that the user is informed of any remaining information that would be leaked, and consents to such leakage.</p>
+                        <p>The list below describes the kinds of information that might be leaked. After a candidate transaction has been created and prior to it being revealed outside a trusted path to the user, the wallet user interface SHOULD obtain the user's consent for all of the leaked information.</p>
+                        <p>Assumption: There is always a transaction confirmation step for transactions that send funds out of the wallet.</p>
+                    </section>
+                    <section id="kinds-of-information-leakage"><h5><span class="section-heading">Kinds of information leakage</span><span class="section-anchor"> <a rel="bookmark" href="#kinds-of-information-leakage"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h5>
+                        <ul>
+                            <li>Transaction version (v4 or v5, as of NU5)</li>
+                        </ul>
+                        <p>It is RECOMMENDED to use v5 transactions unless Sprout funds are being spent.</p>
+                        <ul>
+                            <li>Lock time (rarely used; may be a distinguisher if it is)</li>
+                        </ul>
+                        <p>SHOULD be zero.</p>
+                        <ul>
+                            <li>Expiry height and anchor position</li>
+                        </ul>
+                        <p>See <a href="#anchor-selection">Anchor selection</a> below.</p>
+                        <p>These give information about what block height the creator was synced to, and some policy information.</p>
+                        <ul>
+                            <li>Transparent inputs and outputs</li>
+                        </ul>
+                        <p>See <a href="#linkability-of-transactions-or-addresses">Linkability of transactions or addresses</a>.</p>
+                        <ul>
+                            <li>Public value balances
+                                <ul>
+                                    <li>Together with the transparent inputs and outputs, these determine the fee and the amount being transferred between pools.</li>
+                                    <li>The fee is dependent on policy, but for a given policy we attempt to make it only depend on other already-leaked metadata.</li>
+                                </ul>
+                            </li>
+                        </ul>
+                        <p>TODO: we SHOULD try to create fully shielded transactions where possible.</p>
+                        <ul>
+                            <li>Numbers of JoinSplits, Spends, Outputs, and Actions
+                                <ul>
+                                    <li>This could correlate with other information under certain circumstances. For example in a “dusting attack”, the adversary sends a victim lots of small-valued notes (or notes with strategically chosen values), increasing the probability that the victim’s transactions will have a larger number of spends than other transactions (or an identifiable number of spends). There are note management strategies that can mitigate this, but they have not been implemented yet.</li>
+                                </ul>
+                            </li>
+                            <li>Which spends of given TXOs are repeated across transactions
+                                <ul>
+                                    <li>This may happen because a previous transaction expired and the user is trying to spend some of the same TXOs.</li>
+                                </ul>
+                            </li>
+                            <li>Whether the transaction is coinbase</li>
+                            <li>For coinbase transactions, the amounts / destination addresses / memos of shielded outputs</li>
+                            <li>Orchard flags (enableSpends/enableOutputs)
+                                <ul>
+                                    <li>Under normal circumstances these only depend on whether the transaction is coinbase.</li>
+                                </ul>
+                            </li>
+                        </ul>
+                    </section>
+                </section>
+                <section id="linkability-of-transactions-or-addresses"><h4><span class="section-heading">Linkability of transactions or addresses</span><span class="section-anchor"> <a rel="bookmark" href="#linkability-of-transactions-or-addresses"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <section id="motivation-for-choices-reducing-linkability"><h5><span class="section-heading">Motivation for choices reducing linkability</span><span class="section-anchor"> <a rel="bookmark" href="#motivation-for-choices-reducing-linkability"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h5>
+                        <p>We want to support creating unlinkable addresses, in order that a user can give different addresses to different counterparties, in such a way that the counterparties (even if they collude) cannot tell that the addresses were provided by the same or distinct users.</p>
+                        <p>If multiple UTXOs are received at the same transparent address, it is safe to shield them all in the same transaction, because that is not leaking additional information.</p>
+                        <p>UTXOs received on different transparent receivers SHOULD NOT be shielded in the same transaction. Ideally, when they are shielded in separate transactions, this should be done in such a way that the timing of those transactions is not linkable.</p>
+                        <p>TODO: move this. Daira-Emma thinks that if we only document leakage rather than explicitly say in the specification of how to construct transactions how to mitigate it, then implementors will ignore it.</p>
+                        <p>When spending transparent UTXOs, they SHOULD only be sent to an internal shielded receiver belonging to the wallet, except when they are generated and spent ephemerally as part of a ZIP 320 transfer <a id="footnote-reference-5" class="footnote_reference" href="#zip-0320">4</a>.</p>
+                        <p>A wallet MUST NOT send funds to a transparent address unless all of the source funds come from shielded pools, and this SHOULD be a single shielded pool.</p>
+                        <p>We want to minimize pool crossing.</p>
+                    </section>
+                </section>
+                <section id="anchor-selection"><h4><span class="section-heading">Anchor selection</span><span class="section-anchor"> <a rel="bookmark" href="#anchor-selection"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <p>A wallet SHOULD choose an anchor a number of blocks back from the head of the chain equal to the trusted confirmation depth. That is, if the current block is at height H, the anchor SHOULD reflect the final treestate of the block at height H-3.</p>
+                    <p>TODO: Define a parameter for this depth, and then RECOMMEND a value of 3.</p>
+                </section>
+                <section id="rationale-for-anchor-selection"><h4><span class="section-heading">Rationale for anchor selection</span><span class="section-anchor"> <a rel="bookmark" href="#rationale-for-anchor-selection"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <ul>
+                        <li>If the chain rolls back past the block at which the anchor is chosen, then the anchor and the transaction will be invalidated. This is undesirable both for reliability, and because the nullifiers of spent shielded notes will have been revealed, linking this transaction to any future transactions that spend those notes.
+                            <p>TODO: Reword this given the proposed note management mitigation below.</p>
+                        </li>
+                        <li>On the other hand, it is undesirable to choose an anchor too many blocks back, because that prevents more recently received shielded notes from being spent.</li>
+                        <li>Using a fixed anchor depth (as opposed to a different depth depending on whether or not we are spending trusted shielded notes) avoids leaking information about whether or not the shielded notes we spent were trusted.</li>
+                    </ul>
+                </section>
+                <section id="note-selection"><h4><span class="section-heading">Note selection</span><span class="section-anchor"> <a rel="bookmark" href="#note-selection"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <p>TODO: recommend that when nullifiers are revealed in a transaction that is then invalidated (or revealed in some other detectable way), they SHOULD be used in a note management tx to avoid linking the invalidated tx with some arbitrary future tx. Provided that there are no transparent outputs leaving this wallet's control, the same arities and transparent outputs SHOULD be preserved, which also avoids revealing whether the user changed their mind about whether to send the original semantic transaction.</p>
+                </section>
+                <section id="expiration-height"><h4><span class="section-heading">Expiration height</span><span class="section-anchor"> <a rel="bookmark" href="#expiration-height"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <p>A wallet SHOULD create transactions using the default expiration height of 40 blocks from the current height, as specified in <a id="footnote-reference-6" class="footnote_reference" href="#zip-0203">2</a>.</p>
+                </section>
+                <section id="open-question"><h4><span class="section-heading">Open question</span><span class="section-anchor"> <a rel="bookmark" href="#open-question"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h4>
+                    <p>How should wallet developers time transactions to avoid linkability?</p>
+                    <ul>
+                        <li>when we roll addresses with transparent components, we have to consider how that could allow linking of shielded components</li>
+                    </ul>
+                    <p>TODO: dusting attack mitigation</p>
+                </section>
+            </section>
+            <section id="network-layer-privacy"><h3><span class="section-heading">Network-layer privacy</span><span class="section-anchor"> <a rel="bookmark" href="#network-layer-privacy"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+            </section>
+            <section id="viewing-keys"><h3><span class="section-heading">Viewing keys</span><span class="section-anchor"> <a rel="bookmark" href="#viewing-keys"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>What they are supposed to reveal; see ZIP 310 for Sapling (needs updating for Orchard). <a href="https://github.com/zcash/zips/issues/606">https://github.com/zcash/zips/issues/606</a></p>
+            </section>
+            <section id="allowed-transfers"><h3><span class="section-heading">Allowed transfers</span><span class="section-anchor"> <a rel="bookmark" href="#allowed-transfers"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <ul>
+                    <li>Sprout -&gt; transparent or Sapling</li>
+                    <li>Sapling -&gt; transparent or Sapling or Orchard</li>
+                    <li>Orchard -&gt; transparent or Sapling or Orchard</li>
+                    <li>if auto-shielding is off: * transparent -&gt; transparent or Sapling or Orchard</li>
+                    <li>if auto-shielding is on: * transparent -&gt; internal Orchard or Sapling</li>
+                </ul>
+                <p>Note: wallets MAY further restrict the set of transfers they perform.</p>
+            </section>
+            <section id="auto-shielding"><h3><span class="section-heading">Auto-shielding</span><span class="section-anchor"> <a rel="bookmark" href="#auto-shielding"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>Wallets SHOULD NOT spend funds from a transparent address to an external address, unless the user gives explicit consent for this on a per-transaction basis.</p>
+                <p>TODO: Reword this to define the source transparent address as any transparent address (external, internal, or ephemeral), except in the case of ephemeral as allowed for implementing ZIP 320.</p>
+                <p>In order to support this policy, wallets SHOULD implement a system of auto-shielding with the following characteristics:</p>
+                <p>If auto-shielding functionality is available in a wallet, then users MUST be able to explicitly consent to one of the following possibilities:</p>
+                <ul>
+                    <li>auto-shielding is always on;</li>
+                    <li>auto-shielding is always off;</li>
+                    <li>the user specifies a policy...</li>
+                </ul>
+                <p>Auto-shielding MUST be one of:</p>
+                <ul>
+                    <li>"must opt in or out" (zcashd will do this -- i.e. refuse to start unless the option is configured), or</li>
+                    <li>always on.</li>
+                </ul>
+            </section>
+            <section id="auto-migration"><h3><span class="section-heading">Auto-migration</span><span class="section-anchor"> <a rel="bookmark" href="#auto-migration"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+            </section>
+            <section id="information-leakage-for-transfers-between-pools"><h3><span class="section-heading">Information leakage for transfers between pools</span><span class="section-anchor"> <a rel="bookmark" href="#information-leakage-for-transfers-between-pools"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>If no auto-migration, if you can satisfy a transfer request to Sapling from your Sapling funds, do so.</p>
+                <p>The user's consent is needed to reveal amounts publically (as opposed to revealing them to the holder of a viewing key authorized to see that amount). Therefore, there should be per-transaction opt-in for any transfer that publically reveals amounts on chain.</p>
+                <ul>
+                    <li>there may be a compatibility issue for amount-revealing cross-pool txns that were previously allowed without opt-in</li>
+                </ul>
+                <p>Wallets MUST NOT automatically combine funds across pools to satisfy a transfer (since that could reveal the total funds the user holds in some pool).</p>
+                <p>In order to maintain the integrity of IVK guarantees, wallets should not generate unified addresses that contain internal receivers, nor expose internal receivers (such as those used for auto-shielding and change outputs) in any way.</p>
+                <p>Open questions:</p>
+                <ul>
+                    <li>should there be an auto-migration option from Sapling to Orchard?</li>
+                </ul>
+                <p># str4d notes</p>
+                <p>If we want to have both automatic and opportunistic shielding, and keep the two indistinguishable, then we can't auto-shield when the transparent balance reaches some threshold (otherwise opportunistic would either never be used, or would be identifiable when it uses the balance below the threshold).</p>
+                <p>Instead, a proposition: we define a distribution of "time since last payment to the address" from which we sample the time at which the auto-shielding transaction will be created. This distribution is weighted by the balance in the address, so as more funds accrue, the auto-shielding transaction is more likely to be created.</p>
+                <ul>
+                    <li>It ensures that all funds will eventually be auto-shielded, while preventing fee-dusting attacks (where dust is sent in order to repeatedly consume fees from the wallet), as the auto-shielding transaction is not directly triggered by payment receipt.</li>
+                    <li>If the user makes a shielding transaction in the meantime, we opportunistically shield, without it being clearly not an auto-shielding transaction.</li>
+                    <li>If a wallet is offline for a long time, then it would likely auto-shield as soon as it finishes syncing. This maybe isn't enough to reveal that the wallet came online, except that it _might_ result in auto-shielding transactions for multiple transparent addresses being created at the same time. So we might want to special-case this?</li>
+                </ul>
+                <p>Properties we want from auto-shielding:</p>
+                <ul>
+                    <li>Auto-shielding transactions MUST NOT shield from multiple transparent receivers in the same transaction. - Doing so would trivially link diversified UAs containing transparent receivers.</li>
+                </ul>
+                <p>Properties we want from auto-migration:</p>
+                <ul>
+                    <li>Receipt of a shielded payment MUST NOT trigger any on-chain behaviour (as that reveals transaction linkability).</li>
+                </ul>
+                <p>Both auto-shielding and auto-migration are time-triggered actions, not receipt-triggered actions. An auto-shielding or auto-migration transaction MUST NOT be created as a direct result of a payment being received.</p>
+                <p>Both of these are opportunistic: if the user's wallet is making a transaction in which one of these actions would occur anyway, then the wallet takes the opportunity to migrate as much as it would do if it were generating an autoshielding transaction. This both saves on a transaction, and removes the need for any kind of transparent change address within UAs.</p>
+                <p>TODO: what pool should change go to?</p>
+                <ul>
+                    <li>Proposal: the most recent pool already involved in the transaction.</li>
+                </ul>
+            </section>
+            <section id="wallet-recovery"><h3><span class="section-heading">Wallet Recovery</span><span class="section-anchor"> <a rel="bookmark" href="#wallet-recovery"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h3>
+                <p>In the case where we are recovering a wallet from a backed-up mnemonic phrase, and not from a wallet.dat, we don't have enough information to figure out what receiver types the user originally used when deriving each UA under an account. We have a similar issue if someone exports a UFVK, derives an address from it, and has a payment sent to the address: zcashd will detect the payment, but has no way to figure out what address it should display in the UI. A wallet could store this information in the memo field of change outputs, but that adds a bunch of coordination to the problem, and assumes ongoing on-chain state storage.</p>
+                <ul>
+                    <li>If the receiver matches an address that the wallet knows was derived via <code>z_getaddressforaccount</code>, show that UA as expected (matching the receiver types the user selected).</li>
+                    <li>If the receiver matches a UFVK in the wallet, and we are looking it up because we detected a received note in some block, show the UA with the default receiver types that zcashd was using as of that block height (ideally the earliest block height we detect), and cache this for future usage.</li>
+                    <li>For zcashd's current policy of "best and second-best shielded pools, plus transparent pool", that would mean Orchard, Sapling, and transparent for current block heights.</li>
+                    <li>For each release of a wallet, the wallet should specify a set of receiver types and an associated range of block heights during which the wallet will, by default, generate unified addresses using that set of receiver types.</li>
+                    <li>For zcashd we know how the policy evolves because each zcashd release has an approximate release height and End-of-Service height that defines the window.</li>
+                    <li>Subsequent releases of a wallet SHOULD NOT retroactively change their policies for previously defined block height ranges.</li>
+                    <li>If the receiver type for a note received at a given time is not a member of the set of receiver types expected for the range of block heights, the policy corresponding to the nearest block height range that includes that receiver type SHOULD be used.</li>
+                    <li>If the receiver matches a UFVK in the wallet, and we have no information about when this receiver may have been first used, show the UA corresponding to the most recent receiver types policy that includes the receiver's type.</li>
+                    <li>As part of this, we're also going to change the "Sapling receiver to UfvkId" logic to trial-decrypt after trying internal map, so that we will detect all receivers linked to UFVKs in the wallet, not just receivers in addresses generated via z_getaddressforaccount. The internal map lookup is then just an optimisation (and a future refactor to have Orchard do the same is possible, but for now we will only trial-decrypt so we don't need to refactor to access the Rust wallet). TODO: express this in a less zcashd-specific way.</li>
+                </ul>
+                <p>TODO: Mention recommendations (not requirements) of receiver types based on settled ('accepted') network upgrades, as defined in §3.3 of the Zcash Protocol Specification, at the time of the release of the wallet.</p>
+                <p>TODO: Rationale subsection explaining why earliest block height at detection and the rules/recommendations in place at that block height are preferred over showing different UAs at different heights</p>
+            </section>
+        </section>
+        <section id="references"><h2><span class="section-heading">References</span><span class="section-anchor"> <a rel="bookmark" href="#references"><img width="24" height="24" class="section-anchor" src="assets/images/section-anchor.png" alt=""></a></span></h2>
+            <table id="bcp14" class="footnote">
+                <tbody>
+                    <tr>
+                        <th>1</th>
+                        <td><a href="https://www.rfc-editor.org/info/bcp14">Information on BCP 14 — "RFC 2119: Key words for use in RFCs to Indicate Requirement Levels" and "RFC 8174: Ambiguity of Uppercase vs Lowercase in RFC 2119 Key Words"</a></td>
+                    </tr>
+                </tbody>
+            </table>
+            <table id="zip-0203" class="footnote">
+                <tbody>
+                    <tr>
+                        <th>2</th>
+                        <td><a href="zip-0203">ZIP 203: Transaction Expiry</a></td>
+                    </tr>
+                </tbody>
+            </table>
+            <table id="zip-0317" class="footnote">
+                <tbody>
+                    <tr>
+                        <th>3</th>
+                        <td><a href="zip-0317">ZIP 317: Proportional Transfer Fee Mechanism</a></td>
+                    </tr>
+                </tbody>
+            </table>
+            <table id="zip-0320" class="footnote">
+                <tbody>
+                    <tr>
+                        <th>4</th>
+                        <td><a href="zip-0320">ZIP 320: Defining an Address Type to which funds can only be sent from Transparent Addresses</a></td>
+                    </tr>
+                </tbody>
+            </table>
+        </section>
+    </section>
+    <section class="system-messages">
+        <h1>Docutils System Messages</h1>
+        <div id="system-message-1">
+            <h1>System Message: ERROR/3 (zip-0315.rst line 65)</h1>
+            <p>Too many autonumbered footnote references: only 0 corresponding footnotes available.</p>
+        </div>
+        <div id="system-message-2">
+            <h1>System Message: ERROR/3 (zip-0315.rst line 65) <a href="#footnote-reference-2">footnote-reference-2</a></h1>
+            <p>Unknown target name: "zip-0032".</p>
+        </div>
+        <div id="system-message-3">
+            <h1>System Message: ERROR/3 (zip-0315.rst line 65) <a href="#footnote-reference-3">footnote-reference-3</a></h1>
+            <p>Unknown target name: "zip-0316".</p>
+        </div>
     </section>
 </body>
 </html>
\ No newline at end of file