allow to specify audience on the AuthenticationRequest

auth0 · Nov 24, 2016 · eca69f4 · eca69f4
1 parent 239a4d0
commit eca69f4
Show file tree

Hide file tree

Showing 9 changed files with 102 additions and 0 deletions.
diff --git a/auth0/src/main/java/com/auth0/android/authentication/ParameterBuilder.java b/auth0/src/main/java/com/auth0/android/authentication/ParameterBuilder.java
@@ -63,6 +63,7 @@ public class ParameterBuilder {
     public static final String CLIENT_ID_KEY = "client_id";
     public static final String GRANT_TYPE_KEY = "grant_type";
     public static final String DEVICE_KEY = "device";
+    public static final String AUDIENCE_KEY = "audience";
 
     private Map<String, Object> parameters;
 
@@ -111,6 +112,16 @@ public ParameterBuilder setScope(String scope) {
         return set(SCOPE_KEY, scope);
     }
 
+    /**
+     * Sets the 'audience' parameter.
+     *
+     * @param audience an audience value
+     * @return itself
+     */
+    public ParameterBuilder setAudience(String audience) {
+        return set(AUDIENCE_KEY, audience);
+    }
+
     /**
      * Sets the 'device' parameter
      *

diff --git a/auth0/src/main/java/com/auth0/android/authentication/request/SignUpRequest.java b/auth0/src/main/java/com/auth0/android/authentication/request/SignUpRequest.java
@@ -88,6 +88,12 @@ public AuthenticationRequest setDevice(String device) {
         return this;
     }
 
+    @Override
+    public AuthenticationRequest setAudience(String audience) {
+        authenticationRequest.setAudience(audience);
+        return this;
+    }
+
     @Override
     public AuthenticationRequest setAccessToken(String accessToken) {
         authenticationRequest.setAccessToken(accessToken);

diff --git a/auth0/src/main/java/com/auth0/android/request/AuthenticationRequest.java b/auth0/src/main/java/com/auth0/android/request/AuthenticationRequest.java
@@ -42,6 +42,14 @@ public interface AuthenticationRequest extends Request<Credentials, Authenticati
      */
     AuthenticationRequest setDevice(String device);
 
+    /**
+     * Sets the 'audience' parameter.
+     *
+     * @param audience an audience value
+     * @return itself
+     */
+    AuthenticationRequest setAudience(String audience);
+
     /**
      * Sets the 'access_token' parameter
      *

diff --git a/auth0/src/main/java/com/auth0/android/request/internal/BaseAuthenticationRequest.java b/auth0/src/main/java/com/auth0/android/request/internal/BaseAuthenticationRequest.java
@@ -10,6 +10,7 @@
 import java.util.Map;
 
 import static com.auth0.android.authentication.ParameterBuilder.ACCESS_TOKEN_KEY;
+import static com.auth0.android.authentication.ParameterBuilder.AUDIENCE_KEY;
 import static com.auth0.android.authentication.ParameterBuilder.CONNECTION_KEY;
 import static com.auth0.android.authentication.ParameterBuilder.DEVICE_KEY;
 import static com.auth0.android.authentication.ParameterBuilder.GRANT_TYPE_KEY;
@@ -67,6 +68,18 @@ public AuthenticationRequest setDevice(String device) {
         return this;
     }
 
+    /**
+     * Sets the 'audience' parameter.
+     *
+     * @param audience an audience value
+     * @return itself
+     */
+    @Override
+    public AuthenticationRequest setAudience(String audience) {
+        addParameter(AUDIENCE_KEY, audience);
+        return this;
+    }
+
     /**
      * Sets the 'access_token' parameter
      *

diff --git a/auth0/src/test/java/com/auth0/android/authentication/AuthenticationAPIClientTest.java b/auth0/src/test/java/com/auth0/android/authentication/AuthenticationAPIClientTest.java
@@ -199,6 +199,46 @@ public void shouldLoginWithUserAndPasswordSync() throws Exception {
         assertThat(body, hasEntry("connection", MY_CONNECTION));
     }
 
+    @Test
+    public void shouldLoginWithUserAndPasswordUsingOAuthTokenEndpoint() throws Exception {
+        mockAPI.willReturnSuccessfulLogin();
+        final MockAuthenticationCallback<Credentials> callback = new MockAuthenticationCallback<>();
+
+        client.loginWithToken(SUPPORT_AUTH0_COM, "some-password")
+                .start(callback);
+        assertThat(callback, hasPayloadOfType(Credentials.class));
+
+        final RecordedRequest request = mockAPI.takeRequest();
+        assertThat(request.getHeader("Accept-Language"), is(getDefaultLocale()));
+        Map<String, String> body = bodyFromRequest(request);
+        assertThat(body, hasEntry("client_id", CLIENT_ID));
+        assertThat(body, hasEntry("grant_type", "password"));
+        assertThat(body, hasEntry("username", SUPPORT_AUTH0_COM));
+        assertThat(body, hasEntry("password", "some-password"));
+        assertThat(body, not(hasKey("scope")));
+        assertThat(body, not(hasKey("audience")));
+    }
+
+    @Test
+    public void shouldLoginWithUserAndPasswordSyncUsingOAuthTokenEndpoint() throws Exception {
+        mockAPI.willReturnSuccessfulLogin();
+
+        final Credentials credentials = client
+                .loginWithToken(SUPPORT_AUTH0_COM, "some-password")
+                .execute();
+        assertThat(credentials, is(notNullValue()));
+
+        final RecordedRequest request = mockAPI.takeRequest();
+        assertThat(request.getHeader("Accept-Language"), is(getDefaultLocale()));
+        Map<String, String> body = bodyFromRequest(request);
+        assertThat(body, hasEntry("client_id", CLIENT_ID));
+        assertThat(body, hasEntry("grant_type", "password"));
+        assertThat(body, hasEntry("username", SUPPORT_AUTH0_COM));
+        assertThat(body, hasEntry("password", "some-password"));
+        assertThat(body, not(hasKey("scope")));
+        assertThat(body, not(hasKey("audience")));
+    }
+
     @Test
     public void shouldFetchTokenInfo() throws Exception {
         mockAPI.willReturnTokenInfo();

diff --git a/auth0/src/test/java/com/auth0/android/authentication/ParameterBuilderTest.java b/auth0/src/test/java/com/auth0/android/authentication/ParameterBuilderTest.java
@@ -92,6 +92,12 @@ public void shouldSetScope() throws Exception {
         assertThat(parameters, hasEntry("scope", ParameterBuilder.SCOPE_OFFLINE_ACCESS));
     }
 
+    @Test
+    public void shouldSetAudience() throws Exception {
+        Map<String, Object> parameters = builder.setAudience("https://domain.auth0.com/api").asDictionary();
+        assertThat(parameters, hasEntry("audience", "https://domain.auth0.com/api"));
+    }
+
     @Test
     public void shouldSetDevice() throws Exception {
         Map<String, Object> parameters = builder.setDevice(DEVICE).asDictionary();

diff --git a/auth0/src/test/java/com/auth0/android/authentication/request/AuthenticationRequestMock.java b/auth0/src/test/java/com/auth0/android/authentication/request/AuthenticationRequestMock.java
@@ -53,6 +53,11 @@ public AuthenticationRequest setDevice(String device) {
         return this;
     }
 
+    @Override
+    public AuthenticationRequest setAudience(String audience) {
+        return this;
+    }
+
     @Override
     public AuthenticationRequest setAccessToken(String accessToken) {
         return this;

diff --git a/auth0/src/test/java/com/auth0/android/authentication/request/SignUpRequestTest.java b/auth0/src/test/java/com/auth0/android/authentication/request/SignUpRequestTest.java
@@ -67,6 +67,14 @@ public void shouldSetScope() throws Exception {
         Assert.assertThat(req, is(signUpRequest));
     }
 
+    @Test
+    public void shouldSetAudience() throws Exception {
+        final AuthenticationRequest req = signUpRequest.setAudience("https://domain.auth0.com/api");
+        verify(authenticationMockRequest).setAudience("https://domain.auth0.com/api");
+        Assert.assertThat(req, is(notNullValue()));
+        Assert.assertThat(req, Matchers.<AuthenticationRequest>is(signUpRequest));
+    }
+
     @Test
     public void shouldSetDevice() throws Exception {
         final AuthenticationRequest req = signUpRequest.setDevice("nexus-5x");

diff --git a/auth0/src/test/java/com/auth0/android/request/internal/MockAuthenticationRequest.java b/auth0/src/test/java/com/auth0/android/request/internal/MockAuthenticationRequest.java
@@ -49,6 +49,11 @@ public AuthenticationRequest setDevice(String device) {
         return null;
     }
 
+    @Override
+    public AuthenticationRequest setAudience(String audience) {
+        return null;
+    }
+
     @Override
     public AuthenticationRequest setAccessToken(String accessToken) {
         return null;