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.

Sign up for GitHub

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

Jump to bottom

Limit the custom backend by time limit #843

Closed
Zlopez opened this issue Sep 25, 2019 · 0 comments · Fixed by #851
Closed

Limit the custom backend by time limit #843

Zlopez opened this issue Sep 25, 2019 · 0 comments · Fixed by #851
Labels
High priority This ticket has a high priority type.feature New feature
Milestone
0.18.0

Comments

@Zlopez
Copy link
Contributor

Zlopez commented Sep 25, 2019

Recently I encountered a project that caused OOM error in Anitya by using custom backend with default regex. It was sourceforge project scribus that used custom backend instead sourceforge backend.

To prevent this in the future we should introduce some kind of timeout if regex parsing takes too long.
@Zlopez Zlopez added type.feature New feature High priority This ticket has a high priority labels Sep 25, 2019
@Zlopez Zlopez added this to the 0.18.0 milestone Oct 15, 2019
Zlopez added a commit to Zlopez/anitya that referenced this issue Nov 12, 2019
@Zlopez
Add timeout to check service
60a7bf7 
This will prevent thread from getting stuck when check for new release
takes too long.

This new value is configurable using CHECK_SERVICE parameter in
configuration.

Fixes fedora-infra#843

Signed-off-by: Michal Konečný <[email protected]>
Zlopez added a commit to Zlopez/anitya that referenced this issue Nov 13, 2019
@Zlopez
Add timeout to check service
cb7808c 
This will prevent thread from getting stuck when check for new release
takes too long.

This new value is configurable using CHECK_TIMEOUT parameter in
configuration.

Fixes fedora-infra#843

Signed-off-by: Michal Konečný <[email protected]>
Zlopez added a commit to Zlopez/anitya that referenced this issue Nov 13, 2019
@Zlopez
Add timeout to check service
314439c 
This will prevent thread from getting stuck when check for new release
takes too long.

This new value is configurable using CHECK_TIMEOUT parameter in
configuration.

Fixes fedora-infra#843

Signed-off-by: Michal Konečný <[email protected]>
@Zlopez Zlopez mentioned this issue Nov 13, 2019
Merged
Zlopez added a commit to Zlopez/anitya that referenced this issue Nov 13, 2019
@Zlopez
Add timeout to check service
a4b96f1 
This will prevent thread from getting stuck when check for new release
takes too long.

This new value is configurable using CHECK_TIMEOUT parameter in
configuration.

Fixes fedora-infra#843

Signed-off-by: Michal Konečný <[email protected]>
Zlopez added a commit that referenced this issue Nov 13, 2019
@Zlopez
Add timeout to check service
9f501b3 
This will prevent thread from getting stuck when check for new release
takes too long.

This new value is configurable using CHECK_TIMEOUT parameter in
configuration.

Fixes #843

Signed-off-by: Michal Konečný <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
High priority This ticket has a high priority type.feature New feature
Projects
None yet
Milestone
0.18.0
Development

Successfully merging a pull request may close this issue.

Add timeout to check service Zlopez/anitya
1 participant
@Zlopez