Skip to content
This repository has been archived by the owner on Dec 7, 2020. It is now read-only.

Allow setting of host and port in PusherOptions #2

Closed
mirkancal opened this issue Aug 5, 2019 · 6 comments
Closed

Allow setting of host and port in PusherOptions #2

mirkancal opened this issue Aug 5, 2019 · 6 comments
Labels
enhancement New feature or request

Comments

@mirkancal
Copy link

mirkancal commented Aug 5, 2019

Hey guys, first of all I'm glad for this package, using it for a month now.

PusherOptions only takes cluster as a parameter but I need to set host and port.

Can you guys help me to achieve that or add it to the package if you have time to do that?

I've also do some work on it but It gives me connection exception, I'd appreciate some help.
https://github.com/mirkancal/pusher-websocket-flutter

@MisterJimson
Copy link
Contributor

Hey. Yes this would need to be added to this package to support that. I took a quick look at your fork and you seem to be going in the right direction. Not sure why you are getting an exception, I have not needed to configure those things for my usage.

We may have some time in the future to add this support, but can't guarantee any timeline.

@MisterJimson MisterJimson added the enhancement New feature or request label Aug 6, 2019
@MisterJimson MisterJimson changed the title How to set host and port in PusherOptions? Allow setting of host and port in PusherOptions Aug 6, 2019
@mirkancal
Copy link
Author

Thank you for quick response. I will try to solve it but helps needed.
My frontend guy also uses these options:

enabledTransports: ['ws'],
disabledTransports: ['flash'],
forceTLS: true

I couldn't figure how to set those options. Do you know anything about them?

@MisterJimson
Copy link
Contributor

I know forceTLS can be done with PusherOptions.setEncrypted in the Java SDK. Not familiar with the rest.

@gem85247
Copy link
Contributor

I'm writing support for AuthenticationMethod, Host, Port, Encrypted & Presence Channels in my fork.
Should be working/tested in a few days

@gem85247
Copy link
Contributor

Alright, tested it on both android & iOS, should be working now @mirkancal 🙂
Made a pr @MisterJimson please review, thx!

@MisterJimson
Copy link
Contributor

Should be good to go in 0.1.0!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants