Remove local login bypass
TaxBaik CI/CD / build-and-deploy (push) Successful in 3m5s

This commit is contained in:
2026-07-07 23:44:21 +09:00
parent 2939de70d5
commit dad8eb9da1
3 changed files with 2 additions and 31 deletions
-2
View File
@@ -136,8 +136,6 @@ dotnet watch run --project .\src\TaxBaik.Web\TaxBaik.Web.csproj
- **username**: `admin` - **username**: `admin`
- **password**: `<TAXBAIK_ADMIN_TEST_PASSWORD>` or current rotated admin password - **password**: `<TAXBAIK_ADMIN_TEST_PASSWORD>` or current rotated admin password
- 개발 환경에서는 `test_admin / admin123` 로도 로그인 가능
> ⚠️ **중요**: 프로덕션 배포 시 비밀번호 변경 필수이며, 검증용 비밀번호는 Gitea Secrets로 관리 > ⚠️ **중요**: 프로덕션 배포 시 비밀번호 변경 필수이며, 검증용 비밀번호는 Gitea Secrets로 관리
--- ---
+2 -2
View File
@@ -64,8 +64,8 @@ dotnet watch run --project .\src\TaxBaik.Web\TaxBaik.Web.csproj
- [`src/TaxBaik.Web/appsettings.json`](/D:/JobRoomz/taxbaik/src/TaxBaik.Web/appsettings.json) - [`src/TaxBaik.Web/appsettings.json`](/D:/JobRoomz/taxbaik/src/TaxBaik.Web/appsettings.json)
- [`src/TaxBaik.Web/appsettings.Development.json`](/D:/JobRoomz/taxbaik/src/TaxBaik.Web/appsettings.Development.json) - [`src/TaxBaik.Web/appsettings.Development.json`](/D:/JobRoomz/taxbaik/src/TaxBaik.Web/appsettings.Development.json)
- 로그인 검증: - 로그인 검증:
- 개발 우회 계정 `test_admin / admin123` - 로컬 DB의 실제 관리자 계정
- 또는 로컬 DB의 `admin` 계정 - 로그인 실패 시 DB 연결 오류와 자격 증명 오류를 구분해서 본다
### Production ### Production
-27
View File
@@ -46,18 +46,6 @@ public class AuthService
} }
catch (Exception ex) when (_environment.IsDevelopment()) catch (Exception ex) when (_environment.IsDevelopment())
{ {
if (IsLocalE2ETestCredentials(username, password))
{
_logger.LogWarning(ex, "개발 환경에서 DB 없이 로컬 E2E 관리자 로그인 허용: {Username}", username);
return AuthResult.Success(GenerateTokenPair(new AdminUser
{
Id = 0,
Username = username,
PasswordHash = string.Empty,
CreatedAt = DateTime.UtcNow
}));
}
_logger.LogWarning(ex, "개발 환경에서 관리자 로그인 DB 조회 실패: {Username}", username); _logger.LogWarning(ex, "개발 환경에서 관리자 로그인 DB 조회 실패: {Username}", username);
return AuthResult.DatabaseUnavailable(); return AuthResult.DatabaseUnavailable();
} }
@@ -70,18 +58,6 @@ public class AuthService
if (user == null) if (user == null)
{ {
_logger.LogWarning("로그인 시도: 존재하지 않는 사용자 '{Username}'", username); _logger.LogWarning("로그인 시도: 존재하지 않는 사용자 '{Username}'", username);
if (_environment.IsDevelopment() && IsLocalE2ETestCredentials(username, password))
{
_logger.LogWarning("개발 환경에서 시드 계정 없이 로컬 E2E 관리자 로그인 허용: {Username}", username);
return AuthResult.Success(GenerateTokenPair(new AdminUser
{
Id = 0,
Username = username,
PasswordHash = string.Empty,
CreatedAt = DateTime.UtcNow
}));
}
return AuthResult.InvalidCredentials(); return AuthResult.InvalidCredentials();
} }
@@ -265,9 +241,6 @@ public class AuthService
&& System.Security.Cryptography.CryptographicOperations.FixedTimeEquals(valueBytes, expectedBytes); && System.Security.Cryptography.CryptographicOperations.FixedTimeEquals(valueBytes, expectedBytes);
} }
private static bool IsLocalE2ETestCredentials(string username, string password) =>
string.Equals(username, "test_admin", StringComparison.OrdinalIgnoreCase) &&
string.Equals(password, "admin123", StringComparison.Ordinal);
} }
public class AuthTokenPair(string accessToken, string refreshToken, int expiresIn) public class AuthTokenPair(string accessToken, string refreshToken, int expiresIn)