🚀 [Deploy] - DiscordListener 오류 해결 반영 (#215)

* 🐛 [Fix] - Discord 소셜 로그인 전환 nickname 이슈 해결 (#205) * refactor: 코드 스타일 수정 * fix: Work 생성 날짜 기준으로 내림차순 설정 * fix: PeerReview일 경우 추가 * Chore: 프로젝트 readme 꾸미기 * chore: README 수정 * Fix: 로그아웃 시, 쿠키 삭제 코드 수정 * Refactor: 필요없는 코드 삭제 * !HOTFIX: 명령어 이름 수정에 따른 listener 코드 수정 * Chore: DB 정보 수정 * Fix: JDA 로그 정보 기록 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Feat: Discord 소셜 로그인 구현 * Feat: Discord 소셜 로그인 구현 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * 🐛 [Fix] - Discord 소셜 로그인 전환 (#207) * refactor: 코드 스타일 수정 * fix: Work 생성 날짜 기준으로 내림차순 설정 * fix: PeerReview일 경우 추가 * Chore: 프로젝트 readme 꾸미기 * chore: README 수정 * Fix: 로그아웃 시, 쿠키 삭제 코드 수정 * Refactor: 필요없는 코드 삭제 * !HOTFIX: 명령어 이름 수정에 따른 listener 코드 수정 * Chore: DB 정보 수정 * Fix: JDA 로그 정보 기록 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Feat: Discord 소셜 로그인 구현 * Feat: Discord 소셜 로그인 구현 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * 🐛 [Fix] - Discord 소셜 로그인 전환 nickname 이슈 해결 #51 (#209) * refactor: 코드 스타일 수정 * fix: Work 생성 날짜 기준으로 내림차순 설정 * fix: PeerReview일 경우 추가 * Chore: 프로젝트 readme 꾸미기 * chore: README 수정 * Fix: 로그아웃 시, 쿠키 삭제 코드 수정 * Refactor: 필요없는 코드 삭제 * !HOTFIX: 명령어 이름 수정에 따른 listener 코드 수정 * Chore: DB 정보 수정 * Fix: JDA 로그 정보 기록 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Feat: Discord 소셜 로그인 구현 * Feat: Discord 소셜 로그인 구현 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Fix: Discord 소셜 로그인 전환 중 생긴 이슈 해결 * Chore: 프론트엔드 redirect url 수정 * Fix: 디스코드 소셜 로그인 이슈 해결 * 🐛 [Fix] - 소셜 로그인 후, 추가 정보 입력 받는 기능 오류 (#212) * refactor: 코드 스타일 수정 * fix: Work 생성 날짜 기준으로 내림차순 설정 * fix: PeerReview일 경우 추가 * Chore: 프로젝트 readme 꾸미기 * chore: README 수정 * Fix: 로그아웃 시, 쿠키 삭제 코드 수정 * Refactor: 필요없는 코드 삭제 * !HOTFIX: 명령어 이름 수정에 따른 listener 코드 수정 * Chore: DB 정보 수정 * Fix: JDA 로그 정보 기록 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Feat: Discord 소셜 로그인 구현 * Feat: Discord 소셜 로그인 구현 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Fix: updateUserInfo 오류 해결 * Fix: 디스코드 전환으로 인한 오류 해결 * 🐛 [Fix] - DiscordListener 오류 해결 (#214) * refactor: 코드 스타일 수정 * fix: Work 생성 날짜 기준으로 내림차순 설정 * fix: PeerReview일 경우 추가 * Chore: 프로젝트 readme 꾸미기 * chore: README 수정 * Fix: 로그아웃 시, 쿠키 삭제 코드 수정 * Refactor: 필요없는 코드 삭제 * !HOTFIX: 명령어 이름 수정에 따른 listener 코드 수정 * Chore: DB 정보 수정 * Fix: JDA 로그 정보 기록 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Feat: Discord 소셜 로그인 구현 * Feat: Discord 소셜 로그인 구현 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Chore: Discord 소셜 로그인 Credentials 추가 * Refactor: 코드 스타일 수정 * Refactor: 코드 스타일 수정 * Refactor: 코드 스타일 수정 * Fix: 디스코드 아이디 추가 * Fix: 기본 프로필 사진 지정 * Fix: serialId => discordId로 전환 * Refactor: 코드 스타일 수정
9oormthon-univ · Jul 24, 2024 · 27e8a2f · 27e8a2f
1 parent 0cb40b3
commit 27e8a2f
Show file tree

Hide file tree

Showing 7 changed files with 82 additions and 40 deletions.
diff --git a/src/main/java/goormthon/team28/startup_valley/discord/listener/DiscordListener.java b/src/main/java/goormthon/team28/startup_valley/discord/listener/DiscordListener.java
@@ -24,6 +24,7 @@
 @Slf4j
 @RequiredArgsConstructor
 public class DiscordListener extends ListenerAdapter {
+
     private final UserService userService;
     private final TeamService teamService;
     private final MemberService memberService;
@@ -33,10 +34,13 @@ public class DiscordListener extends ListenerAdapter {
     private final WorkService workService;
     private final GptService gptService;
     private final ReviewService reviewService;
+
     @Override
     public void onSlashCommandInteraction(SlashCommandInteractionEvent event) {
+
         LocalDate nowLocalDate = LocalDate.now();
         LocalDateTime nowLocalDateTime = LocalDateTime.now();
+
         switch (event.getName()) {
             case "1-팀원업데이트":
                 // 봇을 제외한 사용자 목록 생성
@@ -55,9 +59,8 @@ public void onSlashCommandInteraction(SlashCommandInteractionEvent event) {
                     log.info("멤버 생성 완료");
 
                     event.reply("팀 과 팀 멤버를 연결했어요! '파트입력하기' 명령어를 통해 역할을 알려주세요 !!").setEphemeral(true).queue();
-                } else {
+                } else
                     event.reply("웹에 회원가입이 필요합니다!\n\n" + "회원가입 해주세요 !! : " + noSignUp.toString()).setEphemeral(true).queue();
-                }
                 break;
 
             case "2-파트입력하기":
@@ -397,65 +400,84 @@ public void onSlashCommandInteraction(SlashCommandInteractionEvent event) {
                 break;
         }
     }
+
     public List<String> findNoSignUp(List<Member> members) {
+
         List<String> usersWithoutInfo = new ArrayList<>();
         for (Member member : members) {
-            String serialId = member.getUser().getName();
-            if (!userService.isExisted(serialId)) {
-                usersWithoutInfo.add(serialId);
-            }
+            String discordId = member.getUser().getName();
+            if (!userService.isExisted(discordId))
+                usersWithoutInfo.add(discordId);
         }
         return usersWithoutInfo;
     }
+
     private Team myTeam(SlashCommandInteractionEvent event){
         Optional<Team> optionalTeam = teamService.findByGuildId(event.getGuild().getId());
-        if (optionalTeam.isEmpty()){
+        if (optionalTeam.isEmpty())
             event.reply("팀원업데이트를 통해 팀을 만들어주세요 !").setEphemeral(true).queue();
-        }
         return optionalTeam.get();
     }
-    private goormthon.team28.startup_valley.domain.User getUser(SlashCommandInteractionEvent event, String userId){
-        Optional<goormthon.team28.startup_valley.domain.User> optionalUser = userService.findBySerialId(userId);
-        if (optionalUser.isEmpty()){
+
+    private goormthon.team28.startup_valley.domain.User getUser(
+            SlashCommandInteractionEvent event,
+            String userId
+    ){
+        Optional<goormthon.team28.startup_valley.domain.User> optionalUser = userService.findByDiscordId(userId);
+        if (optionalUser.isEmpty())
             event.reply("웹에 회원가입을 먼저 진행해주세요 !").setEphemeral(true).queue();
-        }
         return optionalUser.get();
     }
-    private goormthon.team28.startup_valley.domain.Member getMember(SlashCommandInteractionEvent event, String userId){
-        Optional<goormthon.team28.startup_valley.domain.Member> optionalMember = memberService.findByTeamAndUser(
-                myTeam(event),
-                getUser(event, userId)
-        );
-        if (optionalMember.isEmpty()){
-            event.reply("팀원을 조회할 수 없습니다 ㅠㅠ. 팀원 업데이트를 통해 변경 사항을 적용해주세요 ~ !").setEphemeral(true).queue();
-        }
+
+    private goormthon.team28.startup_valley.domain.Member getMember(
+            SlashCommandInteractionEvent event,
+            String userId
+    ){
+        Optional<goormthon.team28.startup_valley.domain.Member> optionalMember
+                = memberService.findByTeamAndUser(myTeam(event), getUser(event, userId));
+        if (optionalMember.isEmpty())
+            event.reply("팀원을 조회할 수 없습니다 ㅠㅠ. 팀원 업데이트를 통해 변경 사항을 적용해주세요 ~ !")
+                 .setEphemeral(true)
+                 .queue();
         return optionalMember.get();
     }
-    private Optional<Scrum> getProcessingScrum(SlashCommandInteractionEvent event, String userId){
+
+    private Optional<Scrum> getProcessingScrum(
+            SlashCommandInteractionEvent event,
+            String userId
+    ){
         Optional<Scrum> nowScrum = scrumService.findNowScrum(getMember(event, userId));
         if (nowScrum.isEmpty()){
-            event.reply("시작된 작업을 찾을 수 없습니다 ㅠㅠ, 작업을 먼저 시작해주세요").setEphemeral(true).queue();
+            event.reply("시작된 작업을 찾을 수 없습니다 ㅠㅠ, 작업을 먼저 시작해주세요")
+                 .setEphemeral(true)
+                 .queue();
         }
         return nowScrum;
     }
+
     private Optional<Work> getMyProcessingWork(SlashCommandInteractionEvent event){
         String userId = event.getUser().getName();
         return workService.findNotOverWork(getProcessingScrum(event, userId).get(), getMember(event, userId));
     }
-    public void saveOrGetMember(Team team, Member discordMember, SlashCommandInteractionEvent event) {
-        goormthon.team28.startup_valley.domain.Member teamMember = memberService.saveOrGetMember(
-                team,
-                getUser(event, discordMember.getUser().getName())
-        );
 
-        if (isTeamLeaderNotSet(team) && isGuildOwner(discordMember, event)) {
+    public void saveOrGetMember(
+            Team team,
+            Member discordMember,
+            SlashCommandInteractionEvent event
+    ) {
+        goormthon.team28.startup_valley.domain.Member teamMember
+                = memberService.saveOrGetMember(team, getUser(event, discordMember.getUser().getName()));
+
+        if (isTeamLeaderNotSet(team) && isGuildOwner(discordMember, event))
             teamService.updateLeader(team.getId(), teamMember);
-        }
     }
+
     public boolean isTeamLeaderNotSet(Team team) {
         return team.getLeader() == null;
     }
+
     public boolean isGuildOwner(Member discordMember, SlashCommandInteractionEvent event) {
-        return discordMember.getUser().getName().equals(event.getGuild().getOwner().getUser().getName());
+        return discordMember.getUser().getName()
+                .equals(event.getGuild().getOwner().getUser().getName());
     }
 }
diff --git a/src/main/java/goormthon/team28/startup_valley/domain/User.java b/src/main/java/goormthon/team28/startup_valley/domain/User.java
@@ -9,14 +9,14 @@
 import lombok.Getter;
 import lombok.NoArgsConstructor;
 import org.hibernate.annotations.DynamicUpdate;
-import org.springframework.transaction.annotation.Transactional;
 
 @Entity
 @Getter
 @DynamicUpdate
 @Table(name = "users")
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 public class User {
+
     @Id
     @GeneratedValue(strategy = GenerationType.IDENTITY)
     @Column(name = "id")
@@ -25,36 +25,44 @@ public class User {
     /*  사용자 인증 및 권한 정보   */
     @Column(name = "serial_id", nullable = false, unique = true)
     private String serialId; // discord serial id
+
     @Column(name = "password", nullable = false)
     private String password;
-    @Column(name = "discord_id")
-    private Long discordId;
+
+    @Column(name = "discord_id", nullable = false, unique = true)
+    private String discordId;
+
     @Column(name = "role", nullable = false)
     @Enumerated(EnumType.STRING)
     private ERole role;
+
     @Column(name = "provider", nullable = false)
     @Enumerated(EnumType.STRING)
     private EProvider provider;
 
     /*  사용자 이용 정보  */
     @Column(name = "nickname")
     private String nickname;
+
     @Column(name = "profile_image")
     @Enumerated(EnumType.STRING)
     private EProfileImage profileImage;
+
     @Column(name = "refresh_Token")
     private String refreshToken;
 
     @Builder
     public User(
             String serialId,
             String password,
+            String discordId,
             ERole role,
             EProfileImage profileImage,
             EProvider provider
     ) {
         this.serialId = serialId;
         this.password = password;
+        this.discordId = discordId;
         this.role = role;
         this.profileImage = profileImage;
         this.provider = provider;

diff --git a/src/main/java/goormthon/team28/startup_valley/repository/UserRepository.java b/src/main/java/goormthon/team28/startup_valley/repository/UserRepository.java
@@ -19,8 +19,8 @@ public interface UserRepository extends JpaRepository<User, Long> {
     @Query("update User u set u.refreshToken = :refreshToken where u.id = :userId")
     void updateRefreshToken(Long userId, String refreshToken);
     Optional<User> findById(Long userId);
-    Optional<User> findBySerialId(String serialId);
-    boolean existsBySerialId(String serialId);
+    Optional<User> findByDiscordId(String discordId);
+    boolean existsByDiscordId(String discordId);
     boolean existsByIdAndNicknameIsNull(Long id);
 
     interface UserSecurityForm {

diff --git a/src/main/java/goormthon/team28/startup_valley/security/info/DiscordOauth2UserInfo.java b/src/main/java/goormthon/team28/startup_valley/security/info/DiscordOauth2UserInfo.java
@@ -13,4 +13,9 @@ public DiscordOauth2UserInfo(Map<String, Object> attributes) {
     public String getId() {
         return this.attributes.get("id").toString();
     }
+
+    @Override
+    public String getDiscordId() {
+        return this.attributes.get("username").toString();
+    }
 }
diff --git a/src/main/java/goormthon/team28/startup_valley/security/info/factory/Oauth2UserInfo.java b/src/main/java/goormthon/team28/startup_valley/security/info/factory/Oauth2UserInfo.java
@@ -10,4 +10,5 @@
 public abstract class Oauth2UserInfo {
     protected final Map<String, Object> attributes;
     public abstract String getId();
+    public abstract String getDiscordId();
 }
diff --git a/.../java/goormthon/team28/startup_valley/security/service/CustomOauth2UserDetailService.java b/.../java/goormthon/team28/startup_valley/security/service/CustomOauth2UserDetailService.java
@@ -1,6 +1,7 @@
 package goormthon.team28.startup_valley.security.service;
 
 import goormthon.team28.startup_valley.domain.User;
+import goormthon.team28.startup_valley.dto.type.EProfileImage;
 import goormthon.team28.startup_valley.dto.type.EProvider;
 import goormthon.team28.startup_valley.dto.type.ERole;
 import goormthon.team28.startup_valley.repository.UserRepository;
@@ -53,8 +54,10 @@ public OAuth2User loadUser(
                                             bCryptPasswordEncoder
                                                     .encode(UUID.randomUUID().toString())
                                     )
-                                    .provider(provider)
+                                    .discordId(oauth2UserInfo.getDiscordId())
                                     .role(ERole.USER)
+                                    .profileImage(EProfileImage.TOMATO)
+                                    .provider(provider)
                                     .build()
                     );
                     return UserRepository.UserSecurityForm.invoke(newUser);

diff --git a/src/main/java/goormthon/team28/startup_valley/service/UserService.java b/src/main/java/goormthon/team28/startup_valley/service/UserService.java
@@ -21,13 +21,16 @@
 @Service
 @RequiredArgsConstructor
 public class UserService {
+
     private final UserRepository userRepository;
     private final MemberRepository memberRepository;
-    public Optional<User> findBySerialId(String serialId){
-        return userRepository.findBySerialId(serialId);
+
+    public Optional<User> findByDiscordId(String discordId){
+        return userRepository.findByDiscordId(discordId);
     }
-    public boolean isExisted(String serialId){
-        return userRepository.existsBySerialId(serialId);
+
+    public boolean isExisted(String discordId){
+        return userRepository.existsByDiscordId(discordId);
     }
 
     public UserDto getUserInfo(Long userId) {