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

Support the option to choose whether to add a random delay #40

Merged
merged 4 commits into from
May 23, 2023

Conversation

xffxff
Copy link
Contributor

@xffxff xffxff commented May 22, 2023

What does this PR do?

This pull request adds a new feature that allows users to choose whether or not to use websurfx in a production environment. With this update, you can set the production_use flag to true to introduce a random delay before sending requests to the search engine. Conversely, if you prefer to disable this functionality, you can simply change the option to false, like so:

production_use = false

Why is this change important?

This new feature provides users with greater flexibility and control over how they use the tool.

@xffxff xffxff requested a review from neon-mmd May 22, 2023 01:36
@neon-mmd
Copy link
Owner

Also, one more thing, when you open up a PR fill the mandatory fields and all other fields that can be filled while others that are not necessary should be removed. This will allow us to better distinguish between spam PRs and legitimate PRs because most of the spam PRs never fill all the fields, I hope you got my point 😄 .

@xffxff
Copy link
Contributor Author

xffxff commented May 23, 2023

Also, one more thing, when you open up a PR fill the mandatory fields and all other fields that can be filled while others that are not necessary should be removed. This will allow us to better distinguish between spam PRs and legitimate PRs because most of the spam PRs never fill all the fields, I hope you got my point 😄 .

I see :)

@xffxff xffxff requested a review from neon-mmd May 23, 2023 09:35
neon-mmd
neon-mmd previously approved these changes May 23, 2023
Copy link
Owner

@neon-mmd neon-mmd left a comment

Choose a reason for hiding this comment

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

Ok looks great!! 😄 , let's merge it.

Copy link
Owner

@neon-mmd neon-mmd left a comment

Choose a reason for hiding this comment

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

Ok looks great!! 😄 , let's merge it.

@neon-mmd neon-mmd merged commit 8904f34 into neon-mmd:rolling May 23, 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