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

Add env variables for throttling #1140

Merged
merged 3 commits into from
Mar 21, 2023

Conversation

TG1999
Copy link
Contributor

@TG1999 TG1999 commented Mar 1, 2023

No description provided.

@TG1999 TG1999 force-pushed the use_en_variables_for_throttling branch from 853a12a to 499d34f Compare March 1, 2023 14:56
@TG1999 TG1999 force-pushed the use_en_variables_for_throttling branch 2 times, most recently from 236f8ae to c06bc50 Compare March 16, 2023 16:08
To override the default settings, add env variables in ``.env`` file
define the settings there. For example::

ALL_VULNERABLE_PACKAGES_THROTTLING_RATE = '1000/hour'
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add prefix VULNERABLECODE_

Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM... make sure we have a prefix to the variables such as VCIO_ or VULNERABLECODE

@TG1999 TG1999 force-pushed the use_en_variables_for_throttling branch from c06bc50 to c0deac6 Compare March 21, 2023 11:02
TG1999 added 3 commits March 21, 2023 16:44
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
Signed-off-by: Tushar Goel <tushar.goel.dav@gmail.com>
@TG1999 TG1999 force-pushed the use_en_variables_for_throttling branch from c0deac6 to 8560853 Compare March 21, 2023 11:15
@TG1999 TG1999 merged commit 3aff154 into aboutcode-org:main Mar 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants