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

Enable connecting to the AWS IoT MQTT broker #3

Merged
merged 3 commits into from
Jan 11, 2016

Conversation

sbeaulie
Copy link
Contributor

@sbeaulie sbeaulie commented Jan 8, 2016

See README file for changelog

See amazon doc for details of what client configuration they support
http://docs.aws.amazon.com/iot/latest/developerguide//protocols.html

Samuel Beaulieu and others added 3 commits January 8, 2016 15:00
+-Update dependency to <eclipse.paho.mqtt.version>1.0.2</eclipse.paho.mqtt.version>
+-Allow for blank username and password in publisher and subscriber
+-Added clean session checkbox in publisher gui instead of hard coded false
+-Fix a NPE with the Util UUID generator substring method
+-set the Sampler label name in the response sampler for nicer display in View Result Tree Listeners

String testPlanFile = GuiPackage.getInstance().getTestPlanFile();
Copy link
Owner

Choose a reason for hiding this comment

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

If we are running headless, "GuiPackage.getInstance()" will give a nullpointerexception.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@hemikak
Copy link
Owner

hemikak commented Jan 10, 2016

Can you get the latest of the README.md file and commit. I made some changes it recently.

@@ -13,7 +13,7 @@

<properties>
<jmeter-version>2.10</jmeter-version>
<eclipse.paho.mqtt.version>0.4.0</eclipse.paho.mqtt.version>
<eclipse.paho.mqtt.version>1.0.2</eclipse.paho.mqtt.version>
Copy link
Owner

Choose a reason for hiding this comment

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

+1

@sbeaulie
Copy link
Contributor Author

If you merge you can keep your version of the file it was just for me to
write down the changes made to your script. I see you have made changes in
parallel to the master version I used so feel free to keep your changes too
On 10 Jan 2016 1:16 p.m., "Hemika Kodikara" notifications@github.com
wrote:

Can you get the latest of the README.md file and commit. I made some
changes it recently.


Reply to this email directly or view it on GitHub
#3 (comment).

hemikak added a commit that referenced this pull request Jan 11, 2016
Enable connecting to the AWS IoT MQTT broker
@hemikak hemikak merged commit 4b86020 into hemikak:master Jan 11, 2016
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