monero-project / monero-site

https://getmonero.org
BSD 3-Clause "New" or "Revised" License
275 stars 384 forks source link

blog: 10block old decoy vuln. disclosure #2170

Closed plowsof closed 1 year ago

plowsof commented 1 year ago

https://github.com/monero-project/monero/issues/8872

netlify[bot] commented 1 year ago

Deploy Preview for barolo-time-757cf9 ready!

Built without sensitive environment variables

Name Link
Latest commit 5955217ff51bde512f44f04c972fba0821afa441
Latest deploy log https://app.netlify.com/sites/barolo-time-757cf9/deploys/647f0ce01d2c3c0007574380
Deploy Preview https://deploy-preview-2170--barolo-time-757cf9.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

jeffro256 commented 1 year ago

BTW, I edited the graph image in the original post to make the terminology line up better

jeffro256 commented 1 year ago

Do you have a specific question about the credits?

plowsof commented 1 year ago

@jeffro256 credits: just double checking that everyone involved was mentioned (i think its fine as all the PR's are listed?) and the author is "jeffro256" - do you want another handle? Thanks for editing the image, will add. my thinking was the legend on the graph could be above or below instead of to the side (graph would get squished less)

erciccione commented 1 year ago

please squash

plowsof commented 1 year ago

squashed - todo: confirm the advice given by jberman to not instruct old wallet users to avoid spending 10 block old outputs (or simply delete the sentence entirely) , and fix the author of the commit to jeffro256

jeffro256 commented 1 year ago

I confirm that advice @plowsof

jeffro256 commented 1 year ago

I'll fix that graph image tonight edit: and done

plowsof commented 1 year ago

thanks! added new graph, jeffro as author, and removed the old wallet suggestion.