Thank you for your contribution. The code is a good example to teach how to use Steem Python.
- You may want to define mysql access details somewhere outside the constructor of Database class, which is easier to configure and maintain.
- There are some broad execeptions, which is in general a bad practice.
- What are the risks of the database compromised and access tokens leaked?
verify_postmay be better named to indicate its purpose e.g.is_new_post- You may want to check if a post has been upvoted already before voting it.
Your contribution has been evaluated according to Utopian policies and guidelines, as well as a predefined set of questions pertaining to the category.
To view those questions and the relevant answers related to your post, click here.
Need help? Write a ticket on https://support.utopian.io/.
Chat with us on Discord.
[utopian-moderator]
Hi @justyy, thanks for your feedback.
I will look to apply it for the next update. About the database security, currently root access is turned of and each database has a separate account. After your tip I changed the default url for phpmyadmin. I will also add SSL and password protect.
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit
Hey @justyy
Here's a tip for your valuable feedback! @Utopian-io loves and incentivises informative comments.
Contributing on Utopian
Learn how to contribute on our website.
Want to chat? Join us on Discord https://discord.gg/h52nFrV.
Vote for Utopian Witness!
Downvoting a post can decrease pending rewards and make it less visible. Common reasons:
Submit