create account

RE: Let Zappl Clerify some things. FUD Correction by inertia

View this thread on: hive.blogpeakd.comecency.com

Viewing a response to: @zappl/re-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-reggaemuffin-re-inertia-re-zappl-let-zappl-clerify-some-things-fud-correction-20180213t003249874z

· @inertia ·
First there this ...

![](https://steemitimages.com/DQmcQhgbUYF88j8tA8RSSs1YDZ6fgFruB583wG4vsNJYYqQ/image.png)

Then you said ...

> If you look at the tickets we have closed tickets with out commenting ...
> # but the fix had still been put in.

Ok, so the fix had still been put in.  Then you said ...

> Ah what your referring to is transaction being handled on the server and not in the browser.

So is there a fix currently in GitHub or not?
properties (22)
authorinertia
permlinkre-zappl-re-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-reggaemuffin-re-inertia-re-zappl-let-zappl-clerify-some-things-fud-correction-20180213t004103510z
categoryzappl
json_metadata{"tags":["zappl"],"image":["https://steemitimages.com/DQmcQhgbUYF88j8tA8RSSs1YDZ6fgFruB583wG4vsNJYYqQ/image.png"],"app":"steemit/0.1"}
created2018-02-13 00:41:03
last_update2018-02-13 00:41:03
depth9
children5
last_payout2018-02-20 00:41:03
cashout_time1969-12-31 23:59:59
total_payout_value0.000 HBD
curator_payout_value0.000 HBD
pending_payout_value0.000 HBD
promoted0.000 HBD
body_length446
author_reputation346,568,901,399,561
root_title"Let Zappl Clerify some things. FUD Correction"
beneficiaries[]
max_accepted_payout1,000,000.000 HBD
percent_hbd10,000
post_id37,065,556
net_rshares0
@ridhosafwan ·
When I trying using zappl, it so perfectly app
So I hope this can fast get solution, I will trying to help if you need
👍  
properties (23)
authorridhosafwan
permlinkre-inertia-2018214t22144639z
categoryzappl
json_metadata{"tags":"zappl","app":"esteem/1.5.0","format":"markdown+html","community":"esteem"}
created2018-02-13 19:21:48
last_update2018-02-13 19:21:48
depth10
children0
last_payout2018-02-20 19:21:48
cashout_time1969-12-31 23:59:59
total_payout_value0.000 HBD
curator_payout_value0.000 HBD
pending_payout_value0.000 HBD
promoted0.000 HBD
body_length118
author_reputation113,761,409,575
root_title"Let Zappl Clerify some things. FUD Correction"
beneficiaries
0.
accountesteemapp
weight500
max_accepted_payout1,000,000.000 HBD
percent_hbd10,000
post_id37,284,754
net_rshares586,805,086
author_curate_reward""
vote details (1)
@zappl ·
We don't save keys, that data is sent and signed in the server.

The connection.js file shows how zappl handles data. That we do save, What your referring to is me talking about the how we handle tickets.

We don't capture keys in a database, What we do is sign transaction via the server. Which it appears people don't agree with. We were already aware of this since you brought it up to @thedegensloth in discord before.

There is an update to browserfiy transaction for this is already in the works, since before this issue showed up in this posts. 

Zappl is not capturing your keys and storing them. It is signing transaction via the server though instead of the browser. Which will be corrected since people seem to disagree with the method being used.

Please feel free to see how the server handles things here: https://github.com/Zappl/Zappl/tree/master/server
properties (22)
authorzappl
permlinkre-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-reggaemuffin-re-inertia-re-zappl-let-zappl-clerify-some-things-fud-correction-20180213t005916497z
categoryzappl
json_metadata{"tags":["zappl"],"users":["thedegensloth"],"links":["https://github.com/Zappl/Zappl/tree/master/server"],"app":"steemit/0.1"}
created2018-02-13 00:59:15
last_update2018-02-13 00:59:15
depth10
children3
last_payout2018-02-20 00:59:15
cashout_time1969-12-31 23:59:59
total_payout_value0.000 HBD
curator_payout_value0.000 HBD
pending_payout_value0.000 HBD
promoted0.000 HBD
body_length869
author_reputation41,244,449,218,741
root_title"Let Zappl Clerify some things. FUD Correction"
beneficiaries[]
max_accepted_payout1,000,000.000 HBD
percent_hbd10,000
post_id37,068,784
net_rshares0
@inertia ·
[![](https://steemitimages.com/DQmddcw3uwsLPf7Lpxe5UvKtUvfeQ7ttWpjv3Yof39uEMmx/image.png)](https://github.com/Zappl/Zappl/blob/master/server/api/index.js#L3039)

So, the way I read it, if there's an error broadcasting the transfer, you're writing everything in the POST body to the log files.  That would include the wif active key.
properties (22)
authorinertia
permlinkre-zappl-re-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-reggaemuffin-re-inertia-re-zappl-let-zappl-clerify-some-things-fud-correction-20180213t011107600z
categoryzappl
json_metadata{"tags":["zappl"],"image":["https://steemitimages.com/DQmddcw3uwsLPf7Lpxe5UvKtUvfeQ7ttWpjv3Yof39uEMmx/image.png"],"links":["https://github.com/Zappl/Zappl/blob/master/server/api/index.js#L3039"],"app":"steemit/0.1"}
created2018-02-13 01:11:06
last_update2018-02-13 01:11:06
depth11
children2
last_payout2018-02-20 01:11:06
cashout_time1969-12-31 23:59:59
total_payout_value0.000 HBD
curator_payout_value0.000 HBD
pending_payout_value0.000 HBD
promoted0.000 HBD
body_length332
author_reputation346,568,901,399,561
root_title"Let Zappl Clerify some things. FUD Correction"
beneficiaries[]
max_accepted_payout1,000,000.000 HBD
percent_hbd10,000
post_id37,070,960
net_rshares0
@zappl · (edited)
We will exclude the log of keys thank you for letting us know about that. This will be corrected and all log files have been dumped. And the files have been shredded.

Just to be public about the log files we dumped there were no keys in the files. But as inertia let us know as the site scales it was possible. Currently zappl load balances so its less likely and error would occur that keys would be caused to be dumped in logs.

But zappl will make sure keys even on error don't save in logs. This will be a temp fix until browserfy methods and updated in.
👍  
properties (23)
authorzappl
permlinkre-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-zappl-re-inertia-re-reggaemuffin-re-inertia-re-zappl-let-zappl-clerify-some-things-fud-correction-20180213t012330713z
categoryzappl
json_metadata{"tags":["zappl"],"app":"steemit/0.1"}
created2018-02-13 01:23:30
last_update2018-02-13 01:25:09
depth12
children1
last_payout2018-02-20 01:23:30
cashout_time1969-12-31 23:59:59
total_payout_value0.000 HBD
curator_payout_value0.000 HBD
pending_payout_value0.000 HBD
promoted0.000 HBD
body_length559
author_reputation41,244,449,218,741
root_title"Let Zappl Clerify some things. FUD Correction"
beneficiaries[]
max_accepted_payout1,000,000.000 HBD
percent_hbd10,000
post_id37,073,325
net_rshares621,948,802
author_curate_reward""
vote details (1)