在 Spring Security 中添加新用户时出现异常

Question

我有一个用户类和角色类和用户角色类。现在，每次我尝试添加一个具有一组已经存在的角色的新用户时，它都会引发一个正确的唯一错误。但理想情况下，如果新角色已经存在，它不应该尝试保存它。下面我添加了我所有的表格和保存方法。

@Table(name = "t_user")
@Data
public class User {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "id")
    private Long id;

    @Column(name = "mobile_number")
    private String mobileNumber;

   
    @Column(name = "email")
    private String email;

   
    @Column(name = "first_name")
    private String firstName;

    @Size(max = 100)
    @NotBlank(message = "Last name can not be empty")
    @Column(name = "last_name")
    private String lastName;

    @Column(name = "is_archived")
    private Boolean isArchived = false;

    @Column(name = "qualification")
    private String qualification;

    @JsonIgnore
    @Column(name="password")
    private String password;

    @ManyToMany(cascade = CascadeType.ALL, fetch = FetchType.EAGER)
    @JoinTable(name = "t_user_role", joinColumns = {@JoinColumn(name = "user_id", referencedColumnName = "id")},
            inverseJoinColumns = {
                    @JoinColumn(name = "role_id", referencedColumnName = "id")})
    private Set<Role> roles = new HashSet<>();


}

---

@Data
@Table(name = "m_role")
@Entity
public class Role {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "id")
    private Long id;

    @Column(name="name")
    private String name;

}

---

@Data
@Table(name = "t_user_role")
@Entity
public class UserRole {

    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    @Column(name = "id")
    private Long id;

    @ManyToOne
    @JoinColumn(name = "user_id")
    private User user;

    @ManyToOne
    @JoinColumn(name = "role_id")
    private Role role;
}

我保存用户的方法：

User newUser = new User();
        newUser.setFirstName(user.getFirstName());
        newUser.setLastName(user.getLastName());
        newUser.setEmail(user.getEmail());
        newUser.setMobileNumber(user.getPassword());
        newUser.setPassword(bcryptEncoder.encode(user.getPassword()));
        newUser.setRoles(user.getRoles());
        return userRepository.save(newUser);
    }

以下是创建用户的post请求格式：

{
    "firstName":"first",
    "lastName":"name",
    "email":"email@gmail.com",
    "mobileNumber":"1110122223",
    "password":"1234567890",
    "roles":[{
        "name":"ADMIN"
    }]
}

我不想插入应该是理想的角色（如果存在）。但这是我在使用角色实现 Spring Security 时发现的标准方式

Answer 1

您的请求缺少id 的角色。因为id 不存在。 Spring尝试在role表中添加一个新角色。

{
    "firstName":"first",
    "lastName":"name",
    "email":"email@gmail.com",
    "mobileNumber":"1110122223",
    "password":"1234567890",
    "roles":[{
        "id" : "" // ID must be present here.
        "name":"ADMIN"
    }]
}

或者从role -> name，你可以从角色表中获取Role entity/obj，并设置在User对象中。

[更新 2]：

@ManyToMany(cascade = CascadeType.DETACH, fetch = FetchType.LAZY)
  @ToString.Exclude
  private Set<Role> roles = new HashSet<>();

您需要将级联类型从“ALL”更改为“DETACH”。见：ALL 表示如果你保存 USER，ROLE 也会被保存，如果你删除 USER，角色也应该被删除。这不是我们想要的。您只需要使用“ROLE”，无需以任何方式操作“ROLE”表记录。

Answer 2

根据我的理解，您的要求是：

1. 用户实体
2. 角色实体，每个用户都有多个角色
3. 如果角色是从客户端传递的，您希望仅当数据库中不存在该角色时才保存该角色，否则您想使用现有角色（更新：根据 cmets 和我的意见，永远不会理想的事情）

在您的情况下，我建议让 Spring 处理 User->Roles 关系，如下所示：

public class User {

  ... all fields
  @ManyToMany(cascade = CascadeType.ALL, fetch = FetchType.LAZY)
  @ToString.Exclude
  private Set<Role> roles = new HashSet<>();
}

public class Role {
    @Id
    @GeneratedValue(strategy = GenerationType.IDENTITY)
    private Integer id;
    private String name;
}

在角色存储库中，您需要一个方法：Optional<Role> findByName(String Name);

在层之间（最好在服务层），试试这个：

public Map<String, Object> addUser(User user) {
    // perform validations

    user.getRoles().forEach(role -> {
        Role existing = roleRepository.findByName(role.getName())
                       .orElse(new Role(role.getName())); // using optional to create new role with name passed in from the client
        if (existing.getId() != null) user.setRole(existing); // UPDATE: NOT IDEAL
    });
    
    
    ... other tasks
    userRepository.save(user); // this all saves the correct role
    return yourResponseMap;
}

其他说明：

1. 我们通常更喜欢保持 fetches Lazy，而不是 Eager。但在某些情况下，您可能需要 Eager 检索，这取决于您。
2. 在我看来，让 Spring Data JPA 处理第三个表更方便。
3. org.hibernate.PersistentObjectException: detached entity passed to persist 发生在您尝试直接保存从客户端传入的角色而不将其加载到您的应用程序时（请参阅“添加用户”的服务层方法）。
4. 查看此link，您可能会发现它很有帮助。