You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@usergrid.apache.org by mr...@apache.org on 2016/09/07 03:39:13 UTC
[1/6] usergrid git commit: Fix issue with admin user confirmation
flag not working when trying to disable emails.
Repository: usergrid
Updated Branches:
refs/heads/feature-flags 5b42c0a60 -> ad0494a41
Fix issue with admin user confirmation flag not working when trying to disable emails.
Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/1894d61d
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/1894d61d
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/1894d61d
Branch: refs/heads/feature-flags
Commit: 1894d61d4d0f2eaa2519f5b94ff39208ee65996d
Parents: 9fae803
Author: Michael Russo <mr...@apigee.com>
Authored: Tue Sep 6 18:53:38 2016 -0700
Committer: Michael Russo <mr...@apigee.com>
Committed: Tue Sep 6 18:53:38 2016 -0700
----------------------------------------------------------------------
.../management/cassandra/ManagementServiceImpl.java | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/usergrid/blob/1894d61d/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
----------------------------------------------------------------------
diff --git a/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java b/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
index 21c6983..84b573b 100644
--- a/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
+++ b/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
@@ -2647,14 +2647,16 @@ public class ManagementServiceImpl implements ManagementService {
public void sendAdminUserConfirmationEmail( UUID organizationId, UserInfo user ) throws Exception {
- String token = getConfirmationTokenForAdminUser(user.getUuid(), 0, organizationId);
- OrganizationConfig orgConfig = organizationId != null ?
+ if (properties.newAdminUsersRequireConfirmation()) {
+ String token = getConfirmationTokenForAdminUser(user.getUuid(), 0, organizationId);
+ OrganizationConfig orgConfig = organizationId != null ?
getOrganizationConfigByUuid(organizationId) : getOrganizationConfigForUserInfo(user);
- String confirmation_url = orgConfig.getFullUrl(WorkflowUrl.ADMIN_CONFIRMATION_URL, user.getUuid().toString()) +
+ String confirmation_url = orgConfig.getFullUrl(WorkflowUrl.ADMIN_CONFIRMATION_URL, user.getUuid().toString()) +
"?token=" + token;
- sendAdminUserEmail( user, "User Account Confirmation: " + user.getEmail(),
- emailMsg( hashMap( "confirm_email", user.getEmail() ).map( "confirmation_url", confirmation_url ),
- PROPERTIES_EMAIL_ADMIN_CONFIRMATION ) );
+ sendAdminUserEmail(user, "User Account Confirmation: " + user.getEmail(),
+ emailMsg(hashMap("confirm_email", user.getEmail()).map("confirmation_url", confirmation_url),
+ PROPERTIES_EMAIL_ADMIN_CONFIRMATION));
+ }
}
[5/6] usergrid git commit: Merge branch 'hotfix-20160819'
Posted by mr...@apache.org.
Merge branch 'hotfix-20160819'
Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/efb5bfbf
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/efb5bfbf
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/efb5bfbf
Branch: refs/heads/feature-flags
Commit: efb5bfbfd25eb6e8cd4f0d25a8056e7fd874ae9d
Parents: d090154 b567014
Author: Michael Russo <mr...@apigee.com>
Authored: Tue Sep 6 20:38:39 2016 -0700
Committer: Michael Russo <mr...@apigee.com>
Committed: Tue Sep 6 20:38:39 2016 -0700
----------------------------------------------------------------------
.../usergrid/rest/management/users/UserResource.java | 2 +-
.../apache/usergrid/rest/management/AdminUsersIT.java | 2 +-
.../apache/usergrid/management/ManagementService.java | 2 +-
.../management/cassandra/ManagementServiceImpl.java | 13 ++++++++-----
4 files changed, 11 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
[3/6] usergrid git commit: Merge branch 'master' into feature-flags
Posted by mr...@apache.org.
Merge branch 'master' into feature-flags
Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/f38b131a
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/f38b131a
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/f38b131a
Branch: refs/heads/feature-flags
Commit: f38b131a0dc50a81b41fb19067ef976638ac20cb
Parents: 5b42c0a d090154
Author: Michael Russo <mr...@apigee.com>
Authored: Tue Sep 6 18:55:39 2016 -0700
Committer: Michael Russo <mr...@apigee.com>
Committed: Tue Sep 6 18:55:39 2016 -0700
----------------------------------------------------------------------
.../management/cassandra/ManagementServiceImpl.java | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/usergrid/blob/f38b131a/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
----------------------------------------------------------------------
[6/6] usergrid git commit: Merge branch 'master' into feature-flags
Posted by mr...@apache.org.
Merge branch 'master' into feature-flags
Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/ad0494a4
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/ad0494a4
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/ad0494a4
Branch: refs/heads/feature-flags
Commit: ad0494a41a0866c15247eadbac0aa88de3c283b7
Parents: f38b131 efb5bfb
Author: Michael Russo <mr...@apigee.com>
Authored: Tue Sep 6 20:38:56 2016 -0700
Committer: Michael Russo <mr...@apigee.com>
Committed: Tue Sep 6 20:38:56 2016 -0700
----------------------------------------------------------------------
.../usergrid/rest/management/users/UserResource.java | 2 +-
.../apache/usergrid/rest/management/AdminUsersIT.java | 2 +-
.../apache/usergrid/management/ManagementService.java | 2 +-
.../management/cassandra/ManagementServiceImpl.java | 13 ++++++++-----
4 files changed, 11 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/usergrid/blob/ad0494a4/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
----------------------------------------------------------------------
[4/6] usergrid git commit: Fix reactivateTest and handle
re-activation differently from auto-activation with confirmation
requirements.
Posted by mr...@apache.org.
Fix reactivateTest and handle re-activation differently from auto-activation with confirmation requirements.
Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/b5670149
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/b5670149
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/b5670149
Branch: refs/heads/feature-flags
Commit: b56701499ae9531b6524b7742f45532f727af2ef
Parents: 1894d61
Author: Michael Russo <mr...@apigee.com>
Authored: Tue Sep 6 20:36:52 2016 -0700
Committer: Michael Russo <mr...@apigee.com>
Committed: Tue Sep 6 20:36:52 2016 -0700
----------------------------------------------------------------------
.../usergrid/rest/management/users/UserResource.java | 2 +-
.../apache/usergrid/rest/management/AdminUsersIT.java | 2 +-
.../apache/usergrid/management/ManagementService.java | 2 +-
.../management/cassandra/ManagementServiceImpl.java | 13 ++++++++-----
4 files changed, 11 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/usergrid/blob/b5670149/stack/rest/src/main/java/org/apache/usergrid/rest/management/users/UserResource.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/main/java/org/apache/usergrid/rest/management/users/UserResource.java b/stack/rest/src/main/java/org/apache/usergrid/rest/management/users/UserResource.java
index af37cf5..066f734 100644
--- a/stack/rest/src/main/java/org/apache/usergrid/rest/management/users/UserResource.java
+++ b/stack/rest/src/main/java/org/apache/usergrid/rest/management/users/UserResource.java
@@ -433,7 +433,7 @@ public class UserResource extends AbstractContextResource {
ApiResponse response = createApiResponse();
- management.startAdminUserActivationFlow( null, user );
+ management.startAdminUserActivationFlow( null, user, true);
response.setAction( "reactivate user" );
return response;
http://git-wip-us.apache.org/repos/asf/usergrid/blob/b5670149/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
----------------------------------------------------------------------
diff --git a/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java b/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
index 3adc719..a8bd834 100644
--- a/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
+++ b/stack/rest/src/test/java/org/apache/usergrid/rest/management/AdminUsersIT.java
@@ -534,7 +534,7 @@ public class AdminUsersIT extends AbstractRestIT {
@Test
public void reactivateTest() throws Exception {
//call reactivate endpoint on default user
- clientSetup.getRestClient().management().users().user( clientSetup.getUsername() ).reactivate();
+ clientSetup.getRestClient().management().users().user( clientSetup.getUsername() ).reactivate().get();
refreshIndex();
//Create mocked inbox and check to see if you recieved an email in the users inbox.
http://git-wip-us.apache.org/repos/asf/usergrid/blob/b5670149/stack/services/src/main/java/org/apache/usergrid/management/ManagementService.java
----------------------------------------------------------------------
diff --git a/stack/services/src/main/java/org/apache/usergrid/management/ManagementService.java b/stack/services/src/main/java/org/apache/usergrid/management/ManagementService.java
index 5ac1713..2b88b07 100644
--- a/stack/services/src/main/java/org/apache/usergrid/management/ManagementService.java
+++ b/stack/services/src/main/java/org/apache/usergrid/management/ManagementService.java
@@ -247,7 +247,7 @@ public interface ManagementService {
void removeOrganizationApplication( UUID organizationId, UUID applicationId ) throws Exception;
- void startAdminUserActivationFlow( UUID organizationId, UserInfo user ) throws Exception;
+ void startAdminUserActivationFlow(UUID organizationId, UserInfo user, boolean fromReactivate) throws Exception;
void sendAdminUserEmail( UserInfo user, String subject, String html ) throws Exception;
http://git-wip-us.apache.org/repos/asf/usergrid/blob/b5670149/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
----------------------------------------------------------------------
diff --git a/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java b/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
index 84b573b..98cfedf 100644
--- a/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
+++ b/stack/services/src/main/java/org/apache/usergrid/management/cassandra/ManagementServiceImpl.java
@@ -921,7 +921,7 @@ public class ManagementServiceImpl implements ManagementService {
if ( !user.getEmail().equals( properties.getProperty( PROPERTIES_SYSADMIN_LOGIN_EMAIL ) )
&& !user.getEmail().equals( properties .getProperty( PROPERTIES_TEST_ACCOUNT_ADMIN_USER_EMAIL ) ) ) {
if(!tokens.isExternalSSOProviderEnabled()) {
- this.startAdminUserActivationFlow(organizationId, userInfo);
+ this.startAdminUserActivationFlow(organizationId, userInfo, false);
}
}
@@ -2556,9 +2556,14 @@ public class ManagementServiceImpl implements ManagementService {
@Override
- public void startAdminUserActivationFlow( UUID organizationId, UserInfo user ) throws Exception {
+ public void startAdminUserActivationFlow(UUID organizationId, UserInfo user,
+ boolean fromReactivate) throws Exception {
+ // re-activation flow ( or auto-activated flow )
if ( user.isActivated() ) {
- sendAdminUserConfirmationEmail( organizationId, user );
+ // send email always if from re-activation, otherwise check the property flag
+ if ( fromReactivate || newAdminUsersRequireConfirmation() ) {
+ sendAdminUserConfirmationEmail(organizationId, user);
+ }
sendAdminUserActivatedEmail( user );
sendSysAdminNewAdminActivatedNotificationEmail( organizationId, user );
}
@@ -2647,7 +2652,6 @@ public class ManagementServiceImpl implements ManagementService {
public void sendAdminUserConfirmationEmail( UUID organizationId, UserInfo user ) throws Exception {
- if (properties.newAdminUsersRequireConfirmation()) {
String token = getConfirmationTokenForAdminUser(user.getUuid(), 0, organizationId);
OrganizationConfig orgConfig = organizationId != null ?
getOrganizationConfigByUuid(organizationId) : getOrganizationConfigForUserInfo(user);
@@ -2656,7 +2660,6 @@ public class ManagementServiceImpl implements ManagementService {
sendAdminUserEmail(user, "User Account Confirmation: " + user.getEmail(),
emailMsg(hashMap("confirm_email", user.getEmail()).map("confirmation_url", confirmation_url),
PROPERTIES_EMAIL_ADMIN_CONFIRMATION));
- }
}
[2/6] usergrid git commit: Merge branch 'hotfix-20160819'
Posted by mr...@apache.org.
Merge branch 'hotfix-20160819'
Project: http://git-wip-us.apache.org/repos/asf/usergrid/repo
Commit: http://git-wip-us.apache.org/repos/asf/usergrid/commit/d090154e
Tree: http://git-wip-us.apache.org/repos/asf/usergrid/tree/d090154e
Diff: http://git-wip-us.apache.org/repos/asf/usergrid/diff/d090154e
Branch: refs/heads/feature-flags
Commit: d090154e7a5fac4a4ac07456685e4d6325262722
Parents: 867060f 1894d61
Author: Michael Russo <mr...@apigee.com>
Authored: Tue Sep 6 18:54:20 2016 -0700
Committer: Michael Russo <mr...@apigee.com>
Committed: Tue Sep 6 18:54:20 2016 -0700
----------------------------------------------------------------------
.../management/cassandra/ManagementServiceImpl.java | 14 ++++++++------
1 file changed, 8 insertions(+), 6 deletions(-)
----------------------------------------------------------------------