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

response timeout due to asynchronous method used in rpc provide method in spite of setting autoAck to false #534

Closed
CengizPoyraz opened this issue Jan 17, 2017 · 5 comments
Labels

Comments

@CengizPoyraz
Copy link

CengizPoyraz commented Jan 17, 2017

I call once the rpc for registering then it returns two responses. I set the timeout values to 1 min and I am sure that process does not do it more than 1 min. How is that possible?

client.rpc.provide("register", function(data, response){
   client.rpc.make("anorherProcess", data, function(err, result){
if (err){
   response.send({"err": err});
   return;
}
    response.send({"err": err, message:  'user is added successfully!'});
});
});

client.rpc.make("register", data, function(err, result){
   console.log("err:", err, " result:", result);
});

err: RESPONSE_TIMEOUT result: undefined
err: null result: { message: 'user is added successfully!'}

@CengizPoyraz
Copy link
Author

CengizPoyraz commented Jan 18, 2017

I think it is because of asynchronous method used in rpc provide method, Why rpc triggers timeout error before timeout value from config file passed? even I set autoAck property to false, it still gives timeout error.

@CengizPoyraz CengizPoyraz changed the title rpc make returns both timeout and success response timeout due to aynchronous method used in rpc provide method Jan 18, 2017
@CengizPoyraz CengizPoyraz changed the title response timeout due to aynchronous method used in rpc provide method response timeout due to asynchronous method used in rpc provide method in spite of setting autoAck to false Jan 28, 2017
@yasserf yasserf added the bug label Feb 12, 2017
@yasserf
Copy link
Contributor

yasserf commented Feb 12, 2017

Thanks for raising! We will take a look at this as soon as possible.

If possible, can you raise issues against the SDK repo? It makes it easier for sprint planning.

Thank you!

@Sheshagiri
Copy link

This seems to be happening with deepstream.io-client-java as well.

@AlexBHarley
Copy link
Contributor

Hi @Sheshagiri, it looks like the correct config option to be using is rpcResponseTimeout, rather than rpcTimeout as seen here.

Could you please try that and let us know if you're still having trouble?

@Sheshagiri
Copy link

@AlexBHarley thank you for the prompt response. After adding the rpcResponseTimeout flag to the client I don't see the RESPONSE_TIMEOUT issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants