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

Unnecessary logs #4

Open
julianzamt opened this issue Aug 28, 2023 · 0 comments
Open

Unnecessary logs #4

julianzamt opened this issue Aug 28, 2023 · 0 comments
Labels
Backlog Stuff to be done in the future Enhancement This improve the product or Código's source code

Comments

@julianzamt
Copy link

Describe the bug
Not a bug but a recommendation.
In Solana, to log a string message is an expensive action (CPU-intensive).

In the generated processor code, each ix logs its name by default
image
also csl fns
image

This can cause unnecessary expensive execution cost or even worse, a Denial of Service, if the cost surpass limits.

Solution: Remove the logs or implement a way to tlet the user decide if the generated code should have or not debugging info.

@julianzamt julianzamt added the Triage This item needs to be prioritize label Aug 28, 2023
@JazielGuerrero JazielGuerrero added Enhancement This improve the product or Código's source code todo and removed todo labels Aug 30, 2023
@JazielGuerrero JazielGuerrero added Backlog Stuff to be done in the future and removed Triage This item needs to be prioritize labels Dec 5, 2023
@JazielGuerrero JazielGuerrero removed their assignment Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backlog Stuff to be done in the future Enhancement This improve the product or Código's source code
Projects
None yet
Development

No branches or pull requests

2 participants