-
Notifications
You must be signed in to change notification settings - Fork 467
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
Add new url property usage description. #464
Conversation
README.md
Outdated
@@ -73,6 +73,18 @@ Publish a message to a topic: | |||
}]; | |||
``` | |||
|
|||
If you already have a self signed URL from broker like AWS IoT endpoint, use the url property of MQTTWebsocketTransport: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we put MQTTWebsocketTransport
in backquotes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done
README.md
Outdated
MQTTSession *session = [[MQTTSession alloc] init]; | ||
session.transport = transport; | ||
[session connectWithConnectHandler:^(NSError *error) { | ||
// Do some work |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we indent this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oops. Sorry missed this one.
@jcavar took care of comments. |
Nice! Thank you @mendirattanishant! |
In PR #461 we introduced new property
url
inMQTTWebsocketTransport
to accept custom pre signed url for connection.