Dynamically Change Credentials

[rburgst]

Dynamically Change Credentials

[rburgst]

I have added a test case for how updating the username/password mid-flight. https://github.com/rburgst/okhttp-digest/blob/master/src/test/java/com/burgstaller/okhttp/basic/BasicAuthenticatorWithMockWebserverTest.java#L111.
Please let me know if I should keep this issue open, otherwise I will close it in the coming days.

[EFCniux]

Hello!
I have been using thus far something like this, since I don't have the credentials on app startup:

public interface CredentialsProvider {
    @Nullable String getIdentity();
    @Nullable String getPassword();
}

public class Credentials extends com.burgstaller.okhttp.digest.Credentials {
    private final @NonNull CredentialsProvider provider;

    public Credentials(
            @NonNull CredentialsProvider provider
    ) {
        super(null, null);
        this.provider = Check.nonNull(provider);
    }

    @Override
    public @Nullable String getUserName() {
        return provider.getIdentity();
    }

    @Override
    public @Nullable String getPassword() {
        return provider.getPassword();
    }

    @Override
    public final void setUserName(String userName) {
        throw new UnsupportedOperationException();
    }

    @Override
    public final void setPassword(String password) {
        throw new UnsupportedOperationException();
    }
}

By using an interface to supply the credentials you can avoid recreating the whole client, but maybe the cache Map<String, CachingAuthenticator> should be cleared when changing them.

[rburgst]

clearing the cache is definitely necessary as pre-existing authentications will probalby no longer work.

sorry for not coming back earlier