質問編集履歴

ユーザ名とパスワードを追記

2020/06/16 21:42

投稿

waito

スコア23

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -384,6 +384,8 @@
 必要であればご確認をお願い致します。
+ウェブアプリのユーザ名とパスワードはそれぞれuser1とdemoです。
 [https://github.com/uekiGityuto/test-session-control](https://github.com/uekiGityuto/test-session-control)

不要な処理のコメントアウト

2020/06/16 21:42

投稿

waito

スコア23

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -154,7 +154,7 @@
 多重ログインチェックを機能させるためには、UserDetailsを実装したクラスの中で、
-equalsとhashCodeメソッドのオーバーライドを適切に行う必要らしく、
+equalsとhashCodeメソッドのオーバーライドを適切に行うことが必要らしく、
 それが出来ていないのではないかと考えています。
@@ -318,20 +318,18 @@
-	public User getUser() {
-		return user;
-	}
-	/*
-	 * lombokを使わずに実装してみた
+	/* lombokを使わずに実装してみた
 	 *
+	 * public User getUser() {
+	 *		return user;
+	 *	}
+	 *
 	 * @Override
 	 * public int hashCode() {

原因を絞り込めましたので追記しました。

2020/06/16 18:05

投稿

waito

スコア23

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -128,9 +128,275 @@
+## 追記
+問題のプロジェクトとは別に、
+多重ログイン禁止処理を確かめるためのプロジェクトを新規に作成して試してみました。
+すると、WebSecurityConfigurerAdapterを継承したクラスに、
+上記と同様、最大セッション数を1とするコードを書くと2重ログインを禁止出来ました。
+そこから少しずつ、問題のプロジェクトに近づけていくと、
+ログインに使用しているUSERテーブルのNAMEカラムを外部キーにもつMUTTERテーブルを作成し、
+USERテーブルと関連性をもたせたタイミングで2重ログインが禁止できなくなりました。
+ここからは予想なのですが、[この記事](https://qiita.com/rubytomato@github/items/a9a19f4a1cd5d732e5f8)によると、
+多重ログインチェックを機能させるためには、UserDetailsを実装したクラスの中で、
+equalsとhashCodeメソッドのオーバーライドを適切に行う必要らしく、
+それが出来ていないのではないかと考えています。
+その観点から考えたのですが、結局何が問題か分からず、アドバイスをお願い致します。
+なお、DB操作にはJPAを利用しています。
+[Userエンティティクラス]
+```Java
+package sample.spring.domain;
+import java.io.Serializable;
+import java.util.List;
+import javax.persistence.CascadeType;
+import javax.persistence.Column;
+import javax.persistence.Entity;
+import javax.persistence.FetchType;
+import javax.persistence.GeneratedValue;
+import javax.persistence.GenerationType;
+import javax.persistence.Id;
+import javax.persistence.OneToMany;
+import javax.persistence.Table;
+import com.fasterxml.jackson.annotation.JsonIgnore;
+import lombok.AllArgsConstructor;
+import lombok.Data;
+import lombok.NoArgsConstructor;
+import lombok.ToString;
+@Entity
+@Table(name = "USER")
+@Data
+@NoArgsConstructor
+@AllArgsConstructor
+@ToString(exclude = "mutters")
+public class User implements Serializable {
+	private static final long serialVersionUID = 1L;
+	@Id
+	@GeneratedValue(strategy = GenerationType.IDENTITY)
+	@Column(name = "USER_ID")
+	private Integer Id;
+	@Column(unique = true, nullable = false, name = "NAME", length = 64)
+	private String name;
+	@Column(nullable = false, name = "PASS", length = 80)
+	@JsonIgnore
+	private String pass;
+	//　以下をコメントアウトすると2重ログインが禁止出来る
+	@OneToMany(cascade = CascadeType.ALL, fetch = FetchType.LAZY, mappedBy = "user")
+	private List<Mutter> mutters;
+}
+```
+[UserDetailsを実装したUserクラスを継承したクラス]
+```Java
+package sample.spring.service;
+import java.util.Collection;
+import org.springframework.security.core.GrantedAuthority;
+import sample.spring.domain.User;
+import lombok.Data;
+import lombok.EqualsAndHashCode;
+@Data
+@EqualsAndHashCode(callSuper = true)
+public class LoginUserDetails extends org.springframework.security.core.userdetails.User {
+	private static final long serialVersionUID = 1L;
+	private final User user;
+	public LoginUserDetails(User user, Collection<GrantedAuthority> authorities) {
+		super(user.getName(), user.getPass(), authorities);
+		this.user = user;
+	}
+	public User getUser() {
+		return user;
+	}
+	/*
+	 * lombokを使わずに実装してみた
+	 *
+	 * @Override
+	 * public int hashCode() {
+	 * 	return user.getName().hashCode();
+	 * }
+	 *
+	 * @Override
+	 * public boolean equals(Object rhs) {
+	 * 	if (rhs instanceof User) {
+	 * 		return user.getName().equals(((LoginUserDetails) rhs).user.getName()); }
+	 * 		return false;
+	 * }
+	 *
+	 * @Override
+	 * public String toString(){
+	 * 	StringBuilder sb = new StringBuilder();
+	 * 	sb.append("Username: ").append(this.user.getName()).append("; ");
+	 * 	sb.append("Password: [PROTECTED]; ");
+	 *
+	 * 	return sb.toString();
+	 * }
+	 */
+}
+```
+その他のクラスはGitHubにソースを配置したので、
+必要であればご確認をお願い致します。
+[https://github.com/uekiGityuto/test-session-control](https://github.com/uekiGityuto/test-session-control)
 ## 補足
-上記で省略していたSecurityConfigの全量を書きます。
+上記で省略していた問題のあるプロジェクトで使用しているSecurityConfigの全量です。