UUPSUpgradeable is not ERC1882 (draft) compatible, but rather only ERC1967 compatible. about openzeppelin-contracts HOT 2 CLOSED

hey, i agree that EIP-1967 has better rational. But after you mentioned it i see that EIP 1822 is allowing to change it. So I am closing this issue. Thanks for the quick response.

from openzeppelin-contracts.

Hi @yaronvel, quick note:

EIP 1882 was never approved,

EIPs are not "approved". It's the author's responsibility to finalize them and gather community consensus. EIPs may become stagnant but can be picked up by anyone in the community if they think it's valuable to move it forward.

Now on the issue:

EIP 1822 never enforces the use of keccak256("PROXIABLE") as storage location. In fact, the proxiable specification suggests to change it:

proxiable
Compatibility check to ensure the new Logic Contract implements the Universal Upgradeable Proxy Standard. Note that in order to support future implementations, the bytes32 comparison could be changed e.g., keccak256("PROXIABLE-ERC1822-v1").

The storage location is thought to not collide with regular storage locations derived by Solidity variables. However, EIP-1967 has shown a better rationale for this, and similarly for EIP-7201, so using any of both is still within the EIP-1822 boundaries.

As EIP1882 is stagnated, maybe it is just best to remove it from your codebase and avoid confusion.

Being stagnant doesn't mean is not still useful and widely adopted.
Would you mind clarifying how you think this is confusing and if it's causing an issue to our users?

from openzeppelin-contracts.