From 2051ff38f81dd866a0ebdb883f262866348bc309 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Stefan=20Schie=C3=9Fl?= Date: Sun, 29 Jul 2018 00:28:32 +0200 Subject: [PATCH] Update bsip-0040.md --- bsip-0040.md | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/bsip-0040.md b/bsip-0040.md index b87b5ce..24e2516 100644 --- a/bsip-0040.md +++ b/bsip-0040.md @@ -79,7 +79,6 @@ List of possible asserts are: | `any` | [`list`, `of`, `allowed`, `values`] | stateless | | `none` | [`none`, `of`, `these`, `values`] | stateless | | `lt, le, gt, ge` | `comparative` | stateless | -| `length` | [`min`, `max`] | stateless | | `limit` | [`max_cumsum`, `interval_in_sec`] | [`current_cumsum`, `interval_began`] | | `limit_monthly` | [`max_cumsum`, `interval_in_months`] | [`current_cumsum`, `interval_began`] | | `contains_only` | [`list`, `of`, `allowed`, `fields`] | stateless | @@ -95,7 +94,7 @@ Stateless assert, all argument types. `Argument value` must be equal to one of v Stateless assert, all argument types. `Argument value` must NOT be equal to any of the values in the list. #### `lt, le, gt, ge` -Stateless assert, only `int` type arguments. +Stateless assert, applies to `int` and `string` type arguments. If argument is `string` type, use its length as `argument value`. - `lt`: `Argument value` must be less than `comparative` - `le`: `Argument value` must be less than or equal to `comparative` - `gt`: `Argument value` must be greater than `comparative` @@ -178,7 +177,7 @@ Notes: The implementation must not differentiate on which operation the custom a # Milestones We propose do split the implmentation into two milestones: -1. Implementation of basic functionaliy to allow custom active permissions and authorities, including `any`, `none`, `lt, le, gt, ge` and `length` `asserts`. If deemed necessary by developpers, reduce to only allow one key or one account for every `custom active authority` +1. Implementation of basic functionaliy to allow custom active permissions and authorities, including `any`, `none` and `lt, le, gt, ge` `asserts`. If deemed necessary by developpers, reduce to only allow one key or one account for every `custom active authority` 2. Evaluation of stateful asserts `limit`, `limit_monthly` and more sophisticated assert `contains_only` in terms of performance. If positively evaluated, implement This approach allows as well to add other asserts at a later stage (with a new BSIP).