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

To many secret calls cause a memeory explosion #31

Open
fuerststat opened this issue Jun 6, 2018 · 6 comments
Open

To many secret calls cause a memeory explosion #31

fuerststat opened this issue Jun 6, 2018 · 6 comments
Labels
bug an unexpected problem or unintended behavior reprex needs a minimal reproducible example

Comments

@fuerststat
Copy link

I'm doing ETL tasks using ssecret for DNS hiding in Statements.
The Memory (RAM) which is allocated by this "INSERT" statements cannot be freed by GC until I kill my R session (RAM sums up to more than 24-gig during the proces, not using secrets max is 2gig).
Can anyone explain this behaviour to me?

KR, Leo

@gaborcsardi
Copy link
Owner

Maybe a memory leak. Which backend are you using?

@fuerststat
Copy link
Author

R3.4.3 on R Studio Server running on Ubuntu 16.
I thought, maybe the decrypted secret_raw_string (means object or variable) is marked permanently as "active". So the GC would never be able to collect it.
But it's just guessing!?

KR, Leo

@gaborcsardi
Copy link
Owner

Are you using libsecret or environment variables?

@fuerststat
Copy link
Author

fuerststat commented Jun 6, 2018

i prefer neither and call the valut for each statement.
Storing the value in a env-variable, prevents the mem-explosion.
nevertheless i would prefere the vault solution, as also people "debugging" the code are not able to access the secrets

@gaborcsardi gaborcsardi added bug an unexpected problem or unintended behavior reprex needs a minimal reproducible example labels May 6, 2020
@gaborcsardi
Copy link
Owner

Seems like a memory leak in the openssl package: jeroen/openssl#79

@gaborcsardi
Copy link
Owner

Seems like it is not read_key after all, so it must be something else.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug an unexpected problem or unintended behavior reprex needs a minimal reproducible example
Projects
None yet
Development

No branches or pull requests

2 participants