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

remote job submission: add an API function for setting RPC timeout #3379

Merged
merged 3 commits into from
Nov 21, 2019

Conversation

AenBleidd
Copy link
Member

boinc_set_timeout($x): set timeout for job submission RPCs to x seconds

@AenBleidd
Copy link
Member Author

Just a clean copy of #3315 with no unrelated commit

@TheAspens
Copy link
Member

@lfield - are you able to review this PR? I believe that you use the remote submission API and would be in a good position to review. Let me know - thanks!

@lfield lfield self-assigned this Nov 20, 2019
@lfield
Copy link
Contributor

lfield commented Nov 20, 2019

A question and a comment before I test tomorrow. Is this also needed for the C++ code? I think the Python global should also be rpc_timeout.

@davidpanderson
Copy link
Contributor

I changed the variable name. If anyone asks for this feature in the C++ API I'll add it.

@lfield
Copy link
Contributor

lfield commented Nov 21, 2019

I tested both the Python and PHP.
Python returns:
ssl.SSLError: ('The read operation timed out',)
PH returns:
Error: HTTP error
As far as I understand both are behaving as designed.

@lfield lfield merged commit 4458b64 into BOINC:master Nov 21, 2019
@AenBleidd AenBleidd deleted the dpa_rpc_timeout_for_pr branch November 21, 2019 13:49
@AenBleidd AenBleidd added this to the Server Release 1.4.1 milestone Aug 14, 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.

4 participants