Skip to content
This repository has been archived by the owner on Apr 6, 2021. It is now read-only.

deepstream.io-client-java v2+ not able to login API 21- #124

Open
GBR-422777 opened this issue Dec 20, 2017 · 0 comments
Open

deepstream.io-client-java v2+ not able to login API 21- #124

GBR-422777 opened this issue Dec 20, 2017 · 0 comments

Comments

@GBR-422777
Copy link

GBR-422777 commented Dec 20, 2017

deepStreamUtils.getClient().login() is not working on API 21, the same code runs perfectly along with subscribing to records on devices with newer APIs

After checking the code, weird thing is on API 21- devices

/**
* Authenticate the user connection
* @param authParameters The authentication parameters to send to deepstream
* @param loginCallback The callback for a successful / unsuccessful login attempt
* connection
*/
@ObjectiveCName("authenticate:loginCallback:")
void authenticate(JsonElement authParameters, DeepstreamClient.LoginCallback loginCallback) {
this.loginCallback = loginCallback;

    if(authParameters != null) {
        this.authParameters = authParameters;
    } else {
        this.authParameters = new JsonObject();
        this.connectionState=ConnectionState.AWAITING_AUTHENTICATION;
    }

    if( this.tooManyAuthAttempts || this.challengeDenied ) {
        this.client.onError( Topic.ERROR, Event.IS_CLOSED, "The client\'s connection was closed" );
        this.loginCallback.loginFailed(Event.IS_CLOSED, "The client\'s connection was closed");
        return;
    }

    if( this.connectionState == ConnectionState.AWAITING_AUTHENTICATION ) { **//This Always Resolves to solved, with this.connectionState being in ConnectionState.CLOSED**
        this.sendAuthMessage();
    }
}

Whereas devices with API 22+:
f( this.connectionState == ConnectionState.AWAITING_AUTHENTICATION ) { **//This resolves to being True. **

I hope this helps

@GBR-422777 GBR-422777 changed the title Deepstream v2.0.4 able to login API 21- Deepstream v2.0.4 not able to login API 21- Dec 20, 2017
@GBR-422777 GBR-422777 changed the title Deepstream v2.0.4 not able to login API 21- Deepstream v2+ not able to login API 21- Dec 20, 2017
@GBR-422777 GBR-422777 changed the title Deepstream v2+ not able to login API 21- deepstream.io-client-java v2+ not able to login API 21- Dec 20, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant