Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ZIP 321] Update spec to cover Unified Addresses #451

Closed
daira opened this issue Mar 2, 2021 · 1 comment · Fixed by #749
Closed

[ZIP 321] Update spec to cover Unified Addresses #451

daira opened this issue Mar 2, 2021 · 1 comment · Fixed by #749
Assignees

Comments

@daira
Copy link
Collaborator

daira commented Mar 2, 2021

Original description:

The idea is that a ZIP 321 URI can include multiple address types, and can therefore act as a substitute for an address that can more smoothly accommodate address upgrades.

Now that we have unified addresses which achieve the same effect as the above, we should make sure that they are explicitly allowed by ZIP 321.

Adding example URIs that use Unified Addresses is #753.

@daira daira added the ZIP idea label Mar 2, 2021
@r3ld3v r3ld3v modified the milestone: Release 5.0.0 Mar 29, 2022
@daira daira changed the title Update ZIP 321 to cover address bundles Update ZIP 321 to cover unified addresses Dec 25, 2023
@daira daira changed the title Update ZIP 321 to cover unified addresses Update ZIP 321 examples to cover unified addresses Dec 25, 2023
@daira daira changed the title Update ZIP 321 examples to cover unified addresses Update ZIP 321 spec and examples to cover unified addresses Dec 25, 2023
@daira
Copy link
Collaborator Author

daira commented Dec 25, 2023

The current specification says:

Implementations SHOULD check that each instance of zcashaddress is a valid string encoding of either:

  • a Zcash transparent address, using Base58Check [7] as defined in [12]; or
  • a Zcash Sapling address, using Bech32 [8] as defined in [13].

New address formats may be added in future. If the context of whether the payment URI is intended for Testnet or Mainnet is available, then each address SHOULD be checked to be for the correct network.

There isn't really any reason to restrict what new address types can be used, so we should also generalize the wording so that anything described as an address in section 5.6 of the protocol spec is allowed, unless (as for Sprout) it is explicitly prohibited. This should reduce the need to update the ZIP in future.

@daira daira assigned daira and unassigned pacu Dec 25, 2023
@daira daira changed the title Update ZIP 321 spec and examples to cover unified addresses [ZIP 321] Update spec and examples to cover unified addresses Dec 25, 2023
@daira daira changed the title [ZIP 321] Update spec and examples to cover unified addresses [ZIP 321] Update spec and examples to cover Unified Addresses Dec 25, 2023
@daira daira changed the title [ZIP 321] Update spec and examples to cover Unified Addresses [ZIP 321] Update spec to cover Unified Addresses Jan 3, 2024
@daira daira closed this as completed in #749 Jan 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants