Skip to content
This repository has been archived by the owner on Nov 22, 2023. It is now read-only.

fix dos attack #458

Closed
wants to merge 3 commits into from
Closed

fix dos attack #458

wants to merge 3 commits into from

Conversation

Jim8y
Copy link
Contributor

@Jim8y Jim8y commented May 7, 2022

@shargon
Copy link
Member

shargon commented May 7, 2022

Why not increase the price?

@Jim8y
Copy link
Contributor Author

Jim8y commented May 7, 2022

Why not increase the price?

hardfork or not hardfork, that is the question..... i dont like hardfork, but if you change the price, there will be a hardfork for sure.

@roman-khimov
Copy link
Contributor

Well, 3.2.2 is a hardfork anyway. So the way to do it is aggregate all the breaking changes into this one hardfork, give it some fancy name (not HF_2712_FixSyscallFees, smth like DevonRex or Chihuahua is much better) and release a new version enabling the whole set of changes needed at some predetermined mainnet/testnet height. Less hardforks, more useful changes.

@Jim8y
Copy link
Contributor Author

Jim8y commented May 7, 2022

Well, 3.2.2 is a hardfork anyway. So the way to do it is aggregate all the breaking changes into this one hardfork,

Still, hardfork update will influence the experience of ordinary users and developers, i respect and would support you if you all agree to do so.

give it some fancy name (not HF_2712_FixSyscallFees, smth like DevonRex or Chihuahua is much better)

I agree, I even suggested erik about naming forks by the name of cities like London or Tokyo~~~

@Jim8y
Copy link
Contributor Author

Jim8y commented May 7, 2022

newbuffer 1:477 s

cat 0:98 s

left 0:95 s

@roman-khimov
Copy link
Contributor

Still, hardfork update will influence the experience of ordinary users and developers

I doubt changes we discuss here (and changes needed to fix the set of problems we have now) will be noticeable for ordinary users/developers. In normal use that'd be like "now this transaction costs 0.000001 GAS more, [shrug] OK".

Speaking of the change itself, I'd rather just change the price. We should at least try to keep the "GAS costs reflects resource use" principle.

@Jim8y
Copy link
Contributor Author

Jim8y commented May 7, 2022

@roman-khimov I understand your concern, i just personally dont like to change existing system settings. But i would stand beside you if you wanna make the price system more reasonable.

@dusmart
Copy link

dusmart commented May 7, 2022

Why not increase the price?

hardfork or not hardfork, that is the question..... i dont like hardfork, but if you change the price, there will be a hardfork for sure.

Actually, every change in neo-vm should be a hardfork. Who knows whether there exists an attack or not before we update?

@dusmart
Copy link

dusmart commented May 7, 2022

IMO, this commit can prevent lots of DoS attack whether block or charge expensively those frequent opcodes.

fundamentally solved the problem !!!

@dusmart
Copy link

dusmart commented May 7, 2022

Why not increase the price?

This may be good for users since some hash implementation on chain will use lots of opcodes frequently. Charge them instead of block them is good.

But it will be hard to implement.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GAS price for Bytes operation is too cheap: cost 37 mins with 20GAS
4 participants