neo-project / neo / 13671626924

push

github

SmartContract: restrict the number of allowed notifications (#3548)

* add hardofork HF_Echidna

* Add entries to `Designation` event (#3397)

* Add entries to Designation event

* Change to HF_Echidna

* Add UT

* Add count

* [Neo Core StdLib] Add Base64url (#3453)

* add base64url

* active in

* update placehold hf height

* fix hf issue and move methods to proper place.

* fix test

* use identifymodel instead.

* add hardofork HF_Echidna

* Add entries to `Designation` event (#3397)

* Add entries to Designation event

* Change to HF_Echidna

* Add UT

* Add count

* [Neo Core StdLib] Add Base64url (#3453)

* add base64url

* active in

* update placehold hf height

* fix hf issue and move methods to proper place.

* fix test

* use identifymodel instead.

* SmartContract: restrict the number of allowed notifications

Fix the problem described in
https://github.com/nspcc-dev/neo-go/issues/3490. Port the solution from
https://github.com/nspcc-dev/neo-go/pull/3640.

MaxNotificationsCount constraint is chosen based on the Mainnet
statistics of the number of notifications per every transaction, ref.
https://github.com/nspcc-dev/neo-go/issues/3490#issuecomment-2427153253.

Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>

* SmartContract: fix format

Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>

* Update src/Neo/SmartContract/ApplicationEngine.Runtime.cs

* Avoid notification creation

* add hardofork HF_Echidna

* Add entries to `Designation` event (#3397)

* Add entries to Designation event

* Change to HF_Echidna

* Add UT

* Add count

* [Neo Core StdLib] Add Base64url (#3453)

* add base64url

* active in

* update placehold hf height

* fix hf issue and move methods to proper place.

* fix test

* use identifymodel instead.

* format

* Fixed typo

* Added back #3397

* fix format

* Update src/Neo/SmartContract/Native/RoleManagement.cs

* Neo.CLI: revert configuration changes

Signed-off-by: Anna S... (continued)

6049 of 8784 branches covered (68.86%)

Branch coverage included in aggregate %.

2 of 4 new or added lines in 1 file covered. (50.0%)

22160 of 27196 relevant lines covered (81.48%)

128140.96 hits per line