Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Logic] Trading of asset disabled by issuer #183

Open
MarkoPaasila opened this issue Jun 5, 2018 · 4 comments
Open

[Logic] Trading of asset disabled by issuer #183

MarkoPaasila opened this issue Jun 5, 2018 · 4 comments
Labels
component: strategy primitives StrategyBase, BitsharesOrderEngine, BitsharesPriceFeed [3] Type: Enhancement [4] Priority: Medium

Comments

@MarkoPaasila
Copy link
Collaborator

This error message:

 File "/usr/local/lib/python3.5/dist-packages/grapheneapi/graphenewsrpc.py", line 173, in rpcexec
    raise RPCError(ret['error']['message'])
grapheneapi.exceptions.RPCError: Assert Exception: is_authorized_asset( d, *_seller, *_receive_asset ):

means that the issuer of the asset we are trying to trade has disabled trading temporarily or semi-permanently. This doesn't happen so often, but when it happens it will frustrate many users. We should handle this error properly either by

  1. displaying the user a message and stopping the affected worker, or by
  2. slowing down and retrying once every hour until trading is resumed.

Any other ideas?

@litepresence
Copy link

litepresence commented Jun 5, 2018

interesting, in my (non DEXbot but pybitshares using) scripts the error was

RPCError: Assert Exception: _sell_asset->options.whitelist_markets.find(_receive_asset->id) != _sell_asset->options.whitelist_markets.end()

same thoughts though; assertion arguments are very confusing for the user

bitshares/bitshares-core#1018

@thehapax
Copy link
Collaborator

thehapax commented Feb 5, 2019

@MarkoPaasila shall we close or this issue still a concern?

@MarkoPaasila
Copy link
Collaborator Author

It's pretty rare, but also simple. Just one specific exception. Backlog?

@thehapax
Copy link
Collaborator

thehapax commented Feb 7, 2019

@MarkoPaasila changing status to available for any one who wants to take ownership

@bitphage bitphage added the component: strategy primitives StrategyBase, BitsharesOrderEngine, BitsharesPriceFeed label Mar 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: strategy primitives StrategyBase, BitsharesOrderEngine, BitsharesPriceFeed [3] Type: Enhancement [4] Priority: Medium
Projects
None yet
Development

No branches or pull requests

4 participants