Merge pull request #2 from ProjectVG/refactory/refactoring-server

Refactory: 전체적인 서버 코드 리팩토링

Author: ImGdevel

ProjectVG.Api/ApiMiddlewareExtensions.cs

@@ -0,0 +1,67 @@
+using Microsoft.AspNetCore.Builder;
+using ProjectVG.Api.Middleware;
+using ProjectVG.Api.Services;
+
+namespace ProjectVG.Api
+{
+    public static class ApiMiddlewareExtensions
+    {
+        /// <summary>
+        /// API 미들웨어 파이프라인 구성
+        /// </summary>
+        public static IApplicationBuilder UseApiMiddleware(this IApplicationBuilder app, IWebHostEnvironment environment)
+        {
+            // 개발 환경 설정
+            if (environment.IsDevelopment()) {
+                app.UseSwagger();
+                app.UseSwaggerUI(c => {
+                    c.SwaggerEndpoint("/swagger/v1/swagger.json", "ProjectVG API V1");
+                    c.RoutePrefix = "swagger";
+                });
+            }
+
+            // 전역 예외 처리
+            app.UseGlobalExceptionHandler();
+
+            // WebSocket 지원
+            app.UseWebSockets();
+
+            // WebSocket 미들웨어 등록
+            app.UseMiddleware<WebSocketMiddleware>();
+
+            // 요청 로깅 미들웨어
+            app.Use(async (ctx, next) => {
+                var logger = ctx.RequestServices.GetRequiredService<ILogger<Program>>();
+                logger.LogInformation("REQ {method} {path} from {remote}", ctx.Request.Method, ctx.Request.Path, ctx.Connection.RemoteIpAddress);
+                await next();
+            });
+
+            // 인증/인가
+            app.UseAuthentication();
+            app.UseAuthorization();
+
+            // CORS 미들웨어 적용
+            app.UseCors("AllowAll");
+
+            // 컨트롤러 매핑
+            app.UseRouting();
+            app.UseEndpoints(endpoints => {
+                endpoints.MapControllers();
+            });
+
+            return app;
+        }
+
+        /// <summary>
+        /// 개발 환경 전용 기능
+        /// </summary>
+        public static IApplicationBuilder UseDevelopmentFeatures(this IApplicationBuilder app)
+        {
+            // 개발 환경에서 테스트 클라이언트 자동 실행
+            var serviceProvider = app.ApplicationServices;
+            serviceProvider.GetRequiredService<TestClientLauncher>().Launch();
+
+            return app;
+        }
+    }
+}

ProjectVG.Api/ApiServiceCollectionExtensions.cs

@@ -0,0 +1,61 @@
+using Microsoft.Extensions.DependencyInjection;
+using ProjectVG.Api.Services;
+using ProjectVG.Api.Filters;
+using Microsoft.AspNetCore.Authentication.Negotiate;
+
+namespace ProjectVG.Api
+{
+    public static class ApiServiceCollectionExtensions
+    {
+        /// <summary>
+        /// API 서비스 등록
+        /// </summary>
+        public static IServiceCollection AddApiServices(this IServiceCollection services)
+        {
+            services.AddControllers(options => {
+                options.Filters.Add<ModelStateValidationFilter>();
+            });
+
+            services.AddEndpointsApiExplorer();
+            services.AddSwaggerGen(c => {
+                c.SwaggerDoc("v1", new Microsoft.OpenApi.Models.OpenApiInfo {
+                    Title = "ProjectVG API",
+                    Version = "v1",
+                    Description = "ProjectVG API Server"
+                });
+            });
+
+            services.AddSingleton<TestClientLauncher>();
+
+            return services;
+        }
+
+        /// <summary>
+        /// 인증 및 인가 서비스
+        /// </summary>
+        public static IServiceCollection AddApiAuthentication(this IServiceCollection services)
+        {
+            services.AddAuthentication(NegotiateDefaults.AuthenticationScheme)
+                .AddNegotiate();
+
+            services.AddAuthorization(options => {
+                options.FallbackPolicy = null;
+            });
+
+            return services;
+        }
+
+        /// <summary>
+        /// 개발용 CORS 정책
+        /// </summary>
+        public static IServiceCollection AddDevelopmentCors(this IServiceCollection services)
+        {
+            services.AddCors(options => {
+                options.AddPolicy("AllowAll",
+                    policy => policy.AllowAnyOrigin().AllowAnyMethod().AllowAnyHeader());
+            });
+
+            return services;
+        }
+    }
+}

ProjectVG.Api/Configuration/EnvironmentVariableConfigurationProvider.cs

@@ -14,15 +14,15 @@ public EnvironmentVariableConfigurationProvider(IConfiguration configuration)
 
         public override void Load()
         {
-            var data = new Dictionary<string, string>();
+            var data = new Dictionary<string, string?>();
 
             // 모든 설정을 순회하면서 환경 변수 참조를 찾아서 치환
             ReplaceEnvironmentVariables(_configuration, data, "");
 
             Data = data;
         }
 
-        private void ReplaceEnvironmentVariables(IConfiguration configuration, Dictionary<string, string> data, string prefix)
+        private void ReplaceEnvironmentVariables(IConfiguration configuration, Dictionary<string, string?> data, string prefix)
         {
             foreach (var child in configuration.GetChildren())
             {

ProjectVG.Api/Controllers/AuthController.cs

@@ -1,7 +1,8 @@
 using Microsoft.AspNetCore.Mvc;
 using ProjectVG.Application.Services.User;
 using ProjectVG.Application.Models.User;
-using Microsoft.Extensions.Logging;
+using ProjectVG.Api.Models.Auth.Request;
+using ProjectVG.Api.Models.Auth.Response;
 
 namespace ProjectVG.Api.Controllers
 {
@@ -18,191 +19,62 @@ public AuthController(IUserService userService, ILogger<AuthController> logger)
             _logger = logger;
         }
 
-        /// <summary>
-        /// 회원가입
-        /// </summary>
         [HttpPost("register")]
-        public async Task<ActionResult<AuthResponseDto>> Register([FromBody] RegisterRequestDto request)
+        public async Task<ActionResult<AuthResponse>> Register([FromBody] RegisterRequest request)
         {
-            try
+            var userDto = request.ToUserDto();
+            var createdUser = await _userService.CreateUserAsync(userDto);
+            
+            var response = new AuthResponse
             {
-                if (!ModelState.IsValid)
-                {
-                    return BadRequest(ModelState);
-                }
-
-                var userDto = new UserDto
-                {
-                    Username = request.Username,
-                    Name = request.Name,
-                    Email = request.Email,
-                    Provider = "local",
-                    ProviderId = request.Username,
-                    IsActive = true
-                };
-
-                var createdUser = await _userService.CreateUserAsync(userDto);
-                
-                var response = new AuthResponseDto
-                {
-                    Success = true,
-                    Message = "회원가입이 완료되었습니다.",
-                    UserId = createdUser.Id,
-                    Username = createdUser.Username,
-                    Email = createdUser.Email
-                };
-
-                _logger.LogInformation("새 사용자 회원가입 완료: {Username}", createdUser.Username);
-                return Ok(response);
-            }
-            catch (InvalidOperationException ex)
-            {
-                return BadRequest(new AuthResponseDto
-                {
-                    Success = false,
-                    Message = ex.Message
-                });
-            }
-            catch (Exception ex)
-            {
-                _logger.LogError(ex, "회원가입 중 오류가 발생했습니다");
-                return StatusCode(500, new AuthResponseDto
-                {
-                    Success = false,
-                    Message = "회원가입 중 내부 서버 오류가 발생했습니다."
-                });
-            }
+                Success = true,
+                Message = "회원가입이 완료되었습니다.",
+                UserId = createdUser.Id,
+                Username = createdUser.Username,
+                Email = createdUser.Email
+            };
+
+            _logger.LogInformation("새 사용자 회원가입 완료: {Username}", createdUser.Username);
+            return Ok(response);
         }
 
-        /// <summary>
-        /// 로그인
-        /// </summary>
         [HttpPost("login")]
-        public async Task<ActionResult<AuthResponseDto>> Login([FromBody] LoginRequestDto request)
+        public async Task<ActionResult<AuthResponse>> Login([FromBody] LoginRequest request)
         {
-            try
-            {
-                if (!ModelState.IsValid)
-                {
-                    return BadRequest(ModelState);
-                }
-
-                // 사용자명으로 사용자 조회
-                var user = await _userService.GetUserByUsernameAsync(request.Username);
-                if (user == null)
-                {
-                    return Unauthorized(new AuthResponseDto
-                    {
-                        Success = false,
-                        Message = "사용자명 또는 비밀번호가 올바르지 않습니다."
-                    });
-                }
-
-                // TODO: 실제 비밀번호 검증 로직 추가 필요
-
-                var response = new AuthResponseDto
-                {
-                    Success = true,
-                    Message = "로그인이 완료되었습니다.",
-                    UserId = user.Id,
-                    Username = user.Username,
-                    Email = user.Email
-                };
-
-                _logger.LogInformation("사용자 로그인 완료: {Username}", user.Username);
-                return Ok(response);
-            }
-            catch (Exception ex)
+            var user = await _userService.GetUserByUsernameAsync(request.Username);
+            var response = new AuthResponse
             {
-                _logger.LogError(ex, "로그인 중 오류가 발생했습니다");
-                return StatusCode(500, new AuthResponseDto
-                {
-                    Success = false,
-                    Message = "로그인 중 내부 서버 오류가 발생했습니다."
-                });
-            }
+                Success = true,
+                Message = "로그인이 완료되었습니다.",
+                UserId = user.Id,
+                Username = user.Username,
+                Email = user.Email
+            };
+
+            _logger.LogInformation("사용자 로그인 완료: {Username}", user.Username);
+            return Ok(response);
         }
 
-        /// <summary>
-        /// 사용자명 중복 확인
-        /// </summary>
         [HttpGet("check-username/{username}")]
-        public async Task<ActionResult<CheckResponseDto>> CheckUsername(string username)
+        public async Task<ActionResult<CheckResponse>> CheckUsername(string username)
         {
-            try
-            {
-                var exists = await _userService.UsernameExistsAsync(username);
-                return Ok(new CheckResponseDto
-                {
-                    Exists = exists,
-                    Message = exists ? "이미 사용 중인 사용자명입니다." : "사용 가능한 사용자명입니다."
-                });
-            }
-            catch (Exception ex)
+            var exists = await _userService.UsernameExistsAsync(username);
+            return Ok(new CheckResponse
             {
-                _logger.LogError(ex, "사용자명 중복 확인 중 오류가 발생했습니다");
-                return StatusCode(500, new CheckResponseDto
-                {
-                    Exists = false,
-                    Message = "중복 확인 중 내부 서버 오류가 발생했습니다."
-                });
-            }
+                Exists = exists,
+                Message = exists ? "이미 사용 중인 사용자명입니다." : "사용 가능한 사용자명입니다."
+            });
         }
 
-        /// <summary>
-        /// 이메일 중복 확인
-        /// </summary>
         [HttpGet("check-email/{email}")]
-        public async Task<ActionResult<CheckResponseDto>> CheckEmail(string email)
+        public async Task<ActionResult<CheckResponse>> CheckEmail(string email)
         {
-            try
-            {
-                var exists = await _userService.EmailExistsAsync(email);
-                return Ok(new CheckResponseDto
-                {
-                    Exists = exists,
-                    Message = exists ? "이미 사용 중인 이메일입니다." : "사용 가능한 이메일입니다."
-                });
-            }
-            catch (Exception ex)
+            var exists = await _userService.EmailExistsAsync(email);
+            return Ok(new CheckResponse
             {
-                _logger.LogError(ex, "이메일 중복 확인 중 오류가 발생했습니다");
-                return StatusCode(500, new CheckResponseDto
-                {
-                    Exists = false,
-                    Message = "중복 확인 중 내부 서버 오류가 발생했습니다."
-                });
-            }
+                Exists = exists,
+                Message = exists ? "이미 사용 중인 이메일입니다." : "사용 가능한 이메일입니다."
+            });
         }
     }
-
-    // DTO 클래스들
-    public class RegisterRequestDto
-    {
-        public string Username { get; set; } = string.Empty;
-        public string Name { get; set; } = string.Empty;
-        public string Email { get; set; } = string.Empty;
-        public string Password { get; set; } = string.Empty;
-    }
-
-    public class LoginRequestDto
-    {
-        public string Username { get; set; } = string.Empty;
-        public string Password { get; set; } = string.Empty;
-    }
-
-    public class AuthResponseDto
-    {
-        public bool Success { get; set; }
-        public string Message { get; set; } = string.Empty;
-        public Guid? UserId { get; set; }
-        public string? Username { get; set; }
-        public string? Email { get; set; }
-    }
-
-    public class CheckResponseDto
-    {
-        public bool Exists { get; set; }
-        public string Message { get; set; } = string.Empty;
-    }
 }