Skip to content
This repository has been archived by the owner on Dec 3, 2021. It is now read-only.

Update index.js #12

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update index.js #12

wants to merge 1 commit into from

Conversation

erdemunal35
Copy link

Random delay between 100 - 500 seconds would eliminate the risk of getting detected.

Random delay between 100 - 500 seconds would eliminate the risk of getting detected.
@sibbl
Copy link
Owner

sibbl commented May 24, 2021

Hi Erdem, thanks for your PR 😊

While I like the idea behind your change, this currently only affects the default DELAY value.

What do you think about having a solution, which also works when setting a custom DELAY in the environment variables?

I could think of adding a new setting like "DELAY_SPREAD" which represents a maximum percentage, which is added or subtracted from the DELAY value.

By default it could be 0%, so the DELAY value is always used. When setting it to 20 and a DELAY of 1000, then the actually used value would be a random value between 800 and 1200. I hope you get the idea?

Finally, this random value should be updated on each iteration instead of once on startup, shouldn't it? Currently the requests are still done in the same interval the whole time the application runs.

Let me know what you think about these suggestions and whether you think you could possibly find the time to implement it in such kind of a way as part of this PR, or if you need some support 😊

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.

2 participants