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

Improve http runner protectoin #7

Merged
merged 4 commits into from
May 11, 2024
Merged

Improve http runner protectoin #7

merged 4 commits into from
May 11, 2024

Conversation

farzadghanei
Copy link
Owner

  • limit concurrent runs
  • protect http runner with list of required HTTP headers (can also be used to restrict to http auth, but also other headers). since no SSL is used, protection is not secure, but SSL can be provided by a different service like a web server

This is useful for limiting the number of simultaneous connections the
web runner handles at any given time, preventing overloading the server
These changes allow to configure a list of required HTTP headers
to limit clients accessing the http runner.
a list of headers can be specified using a map
Split merging function into two separate functions for timeouts and responses
- Update the Chkok application version from 0.2.0 to 0.3.0
@farzadghanei farzadghanei changed the title Improve http runner security Improve http runner protectoin May 11, 2024
@farzadghanei farzadghanei merged commit 60acf6e into main May 11, 2024
2 checks passed
@farzadghanei farzadghanei deleted the limit-concurrent-runs branch May 11, 2024 07:57
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.

1 participant