refactor(OauthProperties): isAllowedOrigin() 허용 범위 축소 및 로직 개선

Author: leegwichan

src/main/java/timeeat/client/oauth/OauthProperties.java

@@ -58,6 +58,11 @@ private void validateOrigin(String origin) {
 
     public boolean isAllowedOrigin(String origin) {
         return allowedOrigins.stream()
-                .anyMatch(allowedOrigin -> origin.trim().startsWith(allowedOrigin));
+                .anyMatch(allowedOrigin -> isMatchedOrigin(allowedOrigin, origin));
+    }
+
+    private boolean isMatchedOrigin(String allowedOrigin, String origin) {
+        return origin.trim().equals(allowedOrigin)
+                || origin.trim().equals(allowedOrigin + "/");
     }
 }

src/test/java/timeeat/client/oauth/OauthPropertiesTest.java

@@ -5,7 +5,6 @@
 
 import java.util.List;
 import org.junit.jupiter.api.Nested;
-import org.junit.jupiter.api.Test;
 import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.NullAndEmptySource;
 import org.junit.jupiter.params.provider.ValueSource;
@@ -56,7 +55,8 @@ class IsAllowedOrigin {
             assertThat(isAllowed).isTrue();
         }
 
-        @Test
+        @ParameterizedTest
+        @ValueSource(strings = {"https://not-allowed.com", "http://localhost:8080/path", "http://localhost:8080nono"})
         void 허용되지_않은_오리진인_경우_false를_반환한다() {
             OauthProperties oauthProperties = new OauthProperties("client-id", "/path",
                     List.of("http://localhost:8080"));