simplito / elliptic-php

Fast, general Elliptic Curve Cryptography library. Supports curves used in Bitcoin, Ethereum and other cryptocurrencies (secp256k1, ed25519, ..)
MIT License
220 stars 52 forks source link

Modernize: add `#[\SensitiveParameter]` #46

Closed DubbaThony closed 1 year ago

DubbaThony commented 1 year ago

There is no behavior change except tagging some parameters with #[\SensitiveParameter]. Unit tests pass, with one deprecation.

Solves #45

The formatting may look slightly odd, but new tags should be last thing on line, since that's new feature of php, and older versions can continue to work. For php 7.X the tag looks just like comment, so the tag must be last thing on line to preserve php 7.X compatibility.

Docs to SensitiveParameter