Skip to content

Conversation

@rubenmoral
Copy link
Member

The example is located in the "communication/ip" category.

https://jira.digi.com/browse/XBJAPI-360

Signed-off-by: Ruben Moral Ruben.Moral@digi.com

@rubenmoral rubenmoral added this to the wifi milestone Dec 15, 2016
@rubenmoral rubenmoral self-assigned this Dec 15, 2016
@rubenmoral rubenmoral requested a review from diescalo December 15, 2016 12:19
-------------------
First, build and launch the application. When the application starts, it
connects to the web server and sends an empty message to start the
communication. You will receive the following message:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would clarify that you receive the message through the standard output and you have to write the answers there :

 You will receive the following message through the standard output:
 You have to respond typing the following text:

client = serverSocket.accept();
PrintWriter out = new PrintWriter(client.getOutputStream());
BufferedReader in = new BufferedReader(new InputStreamReader(client.getInputStream()));
KnockKnockProtocol kkp = new KnockKnockProtocol();
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

KnockKnockProtocol could be instantiated outside the while loop. It is constant for any socket client.

@rubenmoral rubenmoral force-pushed the rmoral/web_server_example branch from 177ea0d to 90d9ef1 Compare December 16, 2016 09:42
The example is located in the "communication/ip" category.

https://jira.digi.com/browse/XBJAPI-360

Signed-off-by: Ruben Moral <Ruben.Moral@digi.com>
@rubenmoral rubenmoral force-pushed the rmoral/web_server_example branch from 90d9ef1 to 1af4cce Compare December 16, 2016 09:43
@rubenmoral rubenmoral merged commit 1af4cce into wlan_support Dec 16, 2016
@rubenmoral rubenmoral deleted the rmoral/web_server_example branch December 16, 2016 09:43
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.

2 participants