BSIP 75 - Asset Owner defines MCR and MSSR values#218
Conversation
bsip-0075.md
Outdated
| The `create_asset_operation` shall be modified to include the setting of the parameters above, as well as initial `flags` values for MCR and MSSR. | ||
|
|
||
| The `update_asset_operation` shall be modified permit updating of the MCR and MSSR values, if the asset has been created with the permission for the asset owner to do so. NOTE: much of the logic currently in `asset_publish_feeds_evaluator::do_apply()` (i.e. call order checks and calculations) shall be also called here if MCR or MSSR are adjusted. | ||
| The `update_asset_operation` shall be modified to permit updating of the MCR and MSSR values, if the asset has the appropriate `flags` value set permitting the asset owner to do so. NOTE: much of the logic currently in `asset_publish_feeds_evaluator::do_apply()` (i.e. call order checks and calculations) shall be also called here if MCR or MSSR are adjusted. |
There was a problem hiding this comment.
I think that the prior text, which references the pre-existing permission setting of the asset, is more correct
bsip-0075.md
Outdated
| # Rationale | ||
| Prior to this change, adjustments to MCR and MSSR were handled by feed producers. This often requires the asset owner to contact the feed producer and ask for the change. For some assets, such a step is time consuming for both parties and ill-fitted as part of the feed producer's responsibilities. | ||
|
|
||
| This new change shall permit the asset owner to determine *at asset creation time* who should have the responsibility to maintain those values. |
There was a problem hiding this comment.
How will existing smartcoins be affected?
There was a problem hiding this comment.
Existing smartcoins aren't affected. The owners can change settings after the protocol upgrade though.
|
I wonder why this hasn't been merged. |
|
I got a better idea about specification and implementation lately: just add new optional value fields in bitasset options, which can be updated by the asset owners: If one of the parameter is set, it means the owner has an opinion on the parameter and it will be in effect; if not set, then the value provided by the feed producers will be in effect. We can assign a permission bit on each parameter too, so the owners can decide whether to forfeit the right to update it. |
I proposed a commit to include permissions. I left out references to ICR because I thought that it might be out of scope. If appropriate I have text which to update BSIP77 with a similar discussion about permissions. |
|
Yes I think BSIP77 can be updated similarly. Thanks. |
BSIP75: Permissions and their forfeiture
See Issue #96