Skip to content

Commit

Permalink
Rename flag to userModified - updated implies it has been updated in …
Browse files Browse the repository at this point in the history
…the store.

Change some tabs to spaces
  • Loading branch information
fhanik committed Sep 14, 2015
1 parent a73acc7 commit 1d24d6e
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 18 deletions.
Expand Up @@ -19,18 +19,18 @@
*/ */
abstract public class AuthEvent extends ApplicationEvent { abstract public class AuthEvent extends ApplicationEvent {


private boolean userUpdated = true; private boolean userModified = true;


public AuthEvent(UaaUser user, boolean userUpdated) { public AuthEvent(UaaUser user, boolean userUpdated) {
super(user); super(user);
this.userUpdated = userUpdated; this.userModified = userUpdated;
} }


public UaaUser getUser() { public UaaUser getUser() {
return (UaaUser) source; return (UaaUser) source;
} }


public boolean isUserUpdated() { public boolean isUserModified() {
return userUpdated; return userModified;
} }
} }
@@ -1,5 +1,5 @@
/******************************************************************************* /*******************************************************************************
* Cloud Foundry * Cloud Foundry
* Copyright (c) [2009-2014] Pivotal Software, Inc. All Rights Reserved. * Copyright (c) [2009-2014] Pivotal Software, Inc. All Rights Reserved.
* *
* This product is licensed to you under the Apache License, Version 2.0 (the "License"). * This product is licensed to you under the Apache License, Version 2.0 (the "License").
Expand All @@ -13,7 +13,6 @@


package org.cloudfoundry.identity.uaa.authentication.manager; package org.cloudfoundry.identity.uaa.authentication.manager;


import org.cloudfoundry.identity.uaa.authentication.Origin;
import org.cloudfoundry.identity.uaa.user.UaaUser; import org.cloudfoundry.identity.uaa.user.UaaUser;
import org.springframework.security.core.GrantedAuthority; import org.springframework.security.core.GrantedAuthority;


Expand All @@ -29,8 +28,8 @@ public Collection<? extends GrantedAuthority> getExternalAuthorities() {


private boolean addGroups = false; private boolean addGroups = false;


public ExternalGroupAuthorizationEvent(UaaUser user, boolean userUpdated, Collection<? extends GrantedAuthority> externalAuthorities, boolean addGroups) { public ExternalGroupAuthorizationEvent(UaaUser user, boolean userModified, Collection<? extends GrantedAuthority> externalAuthorities, boolean addGroups) {
super(user, userUpdated); super(user, userModified);
this.addGroups = addGroups; this.addGroups = addGroups;
this.externalAuthorities = externalAuthorities; this.externalAuthorities = externalAuthorities;
} }
Expand Down
Expand Up @@ -70,17 +70,17 @@ protected String getEmail(UaaUser user, LdapUserDetails details) {


@Override @Override
protected UaaUser userAuthenticated(Authentication request, UaaUser user) { protected UaaUser userAuthenticated(Authentication request, UaaUser user) {
boolean userUpdated = false; boolean userModified = false;
//we must check and see if the email address has changed between authentications //we must check and see if the email address has changed between authentications
if (request.getPrincipal() !=null && request.getPrincipal() instanceof ExtendedLdapUserDetails) { if (request.getPrincipal() !=null && request.getPrincipal() instanceof ExtendedLdapUserDetails) {
ExtendedLdapUserDetails details = (ExtendedLdapUserDetails)request.getPrincipal(); ExtendedLdapUserDetails details = (ExtendedLdapUserDetails)request.getPrincipal();
UaaUser fromRequest = getUser(details, getExtendedAuthorizationInfo(request)); UaaUser fromRequest = getUser(details, getExtendedAuthorizationInfo(request));
if (fromRequest.getEmail()!=null && !fromRequest.getEmail().equals(user.getEmail())) { if (fromRequest.getEmail()!=null && !fromRequest.getEmail().equals(user.getEmail())) {
user = user.modifyEmail(fromRequest.getEmail()); user = user.modifyEmail(fromRequest.getEmail());
userUpdated = true; userModified = true;
} }
} }
ExternalGroupAuthorizationEvent event = new ExternalGroupAuthorizationEvent(user, userUpdated, request.getAuthorities(), isAutoAddAuthorities()); ExternalGroupAuthorizationEvent event = new ExternalGroupAuthorizationEvent(user, userModified, request.getAuthorities(), isAutoAddAuthorities());
publish(event); publish(event);
return getUserDatabase().retrieveUserById(user.getId()); return getUserDatabase().retrieveUserById(user.getId());
} }
Expand Down
Expand Up @@ -12,12 +12,6 @@
*******************************************************************************/ *******************************************************************************/
package org.cloudfoundry.identity.uaa.scim.bootstrap; package org.cloudfoundry.identity.uaa.scim.bootstrap;


import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
import java.util.List;
import java.util.Set;

import org.apache.commons.logging.Log; import org.apache.commons.logging.Log;
import org.apache.commons.logging.LogFactory; import org.apache.commons.logging.LogFactory;
import org.cloudfoundry.identity.uaa.authentication.Origin; import org.cloudfoundry.identity.uaa.authentication.Origin;
Expand Down Expand Up @@ -187,7 +181,7 @@ public void onApplicationEvent(AuthEvent event) {
addToGroup(exEvent.getUser().getId(), authority.getAuthority(), exEvent.getUser().getOrigin(), exEvent.isAddGroups()); addToGroup(exEvent.getUser().getId(), authority.getAuthority(), exEvent.getUser().getOrigin(), exEvent.isAddGroups());
} }
//update the user itself //update the user itself
if(event.isUserUpdated()) { if(event.isUserModified()) {
//update the user itself //update the user itself
ScimUser user = getScimUser(event.getUser()); ScimUser user = getScimUser(event.getUser());
updateUser(user, event.getUser(), false); updateUser(user, event.getUser(), false);
Expand Down

0 comments on commit 1d24d6e

Please sign in to comment.