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

Override timeout on process_check_run #277

Open
agritheory opened this issue Aug 20, 2024 · 0 comments
Open

Override timeout on process_check_run #277

agritheory opened this issue Aug 20, 2024 · 0 comments
Assignees

Comments

@agritheory
Copy link
Owner

agritheory commented Aug 20, 2024

In cases where a lot of checks need to be processed, the default nginx timeout that frappe provides (120 seconds) may not be enough.

Potential strategies:

  • restructure so that a return value is sent after the process is enqueued
  • Allow the realtime function to continue updating
  • Force update when it completes

As best I can find, Frappe does not provide a place to override the timeout value on a request in the client, like in frappe.call, which is too bad. This likely wouldn't override the nginx setting anyway.

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

No branches or pull requests

2 participants