Update index.js

[khuramzahid]

@Clement-Jean I have tried running this code on my machine. I was having issues until I moved the client.close() command inside the callback function as shown. Please review.

[Clement-Jean]

Could you please describe the "issues" you were having?

[khuramzahid]

@Clement-Jean I tried running the code as it is, but it was not displaying the message received from the server. Based on my understanding, because it takes some time for the server to execute the RPC method, and the client executes close() function synchronously, it was not giving time to the client to wait for the RPC execution and getting closed before that.

[Clement-Jean]

@khuramzahid So, we actually don't need this client.close(). I was probably being too cautious. Could you remove it in greet and all the other client/index.js?

edit

make sure to git pull before doing the edits

[khuramzahid]

@Clement-Jean Sure. I am currently following the Udemy course. As soon as I am done with the course, I will push another commit to my forked repo and update this Pull Request.