Ver Fonte

Add factory class for RequestMatcher composition

Closes gh-12751
Christian Schuster há 2 anos atrás
pai
commit
6791f3208e

+ 66 - 0
web/src/main/java/org/springframework/security/web/util/matcher/RequestMatchers.java

@@ -0,0 +1,66 @@
+/*
+ * Copyright 2002-2023 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.springframework.security.web.util.matcher;
+
+import java.util.List;
+
+/**
+ * A factory class to create {@link RequestMatcher} instances.
+ *
+ * @author Christian Schuster
+ * @since 6.1
+ */
+public final class RequestMatchers {
+
+	/**
+	 * Creates a {@link RequestMatcher} that matches if at least one of the given
+	 * {@link RequestMatcher}s matches, if <code>matchers</code> are empty then the
+	 * returned matcher never matches.
+	 * @param matchers the {@link RequestMatcher}s to use
+	 * @return the any-of composed {@link RequestMatcher}
+	 * @see OrRequestMatcher
+	 */
+	public static RequestMatcher anyOf(RequestMatcher... matchers) {
+		return (matchers.length > 0) ? new OrRequestMatcher(List.of(matchers)) : (request) -> false;
+	}
+
+	/**
+	 * Creates a {@link RequestMatcher} that matches if all the given
+	 * {@link RequestMatcher}s match, if <code>matchers</code> are empty then the returned
+	 * matcher always matches.
+	 * @param matchers the {@link RequestMatcher}s to use
+	 * @return the all-of composed {@link RequestMatcher}
+	 * @see AndRequestMatcher
+	 */
+	public static RequestMatcher allOf(RequestMatcher... matchers) {
+		return (matchers.length > 0) ? new AndRequestMatcher(List.of(matchers)) : (request) -> true;
+	}
+
+	/**
+	 * Creates a {@link RequestMatcher} that matches if the given {@link RequestMatcher}
+	 * does not match.
+	 * @param matcher the {@link RequestMatcher} to use
+	 * @return the inverted {@link RequestMatcher}
+	 */
+	public static RequestMatcher not(RequestMatcher matcher) {
+		return (request) -> !matcher.matches(request);
+	}
+
+	private RequestMatchers() {
+	}
+
+}

+ 86 - 0
web/src/test/java/org/springframework/security/web/util/matcher/RequestMatchersTests.java

@@ -0,0 +1,86 @@
+/*
+ * Copyright 2002-2023 the original author or authors.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.springframework.security.web.util.matcher;
+
+import org.junit.jupiter.api.Test;
+
+import static org.assertj.core.api.Assertions.assertThat;
+
+/**
+ * Tests for {@link RequestMatchers}.
+ *
+ * @author Christian Schuster
+ */
+class RequestMatchersTests {
+
+	@Test
+	void checkAnyOfWhenOneMatchThenMatch() {
+		RequestMatcher composed = RequestMatchers.anyOf((r) -> false, (r) -> true);
+		boolean match = composed.matches(null);
+		assertThat(match).isTrue();
+	}
+
+	@Test
+	void checkAnyOfWhenNoneMatchThenNotMatch() {
+		RequestMatcher composed = RequestMatchers.anyOf((r) -> false, (r) -> false);
+		boolean match = composed.matches(null);
+		assertThat(match).isFalse();
+	}
+
+	@Test
+	void checkAnyOfWhenEmptyThenNotMatch() {
+		RequestMatcher composed = RequestMatchers.anyOf();
+		boolean match = composed.matches(null);
+		assertThat(match).isFalse();
+	}
+
+	@Test
+	void checkAllOfWhenOneNotMatchThenNotMatch() {
+		RequestMatcher composed = RequestMatchers.allOf((r) -> false, (r) -> true);
+		boolean match = composed.matches(null);
+		assertThat(match).isFalse();
+	}
+
+	@Test
+	void checkAllOfWhenAllMatchThenMatch() {
+		RequestMatcher composed = RequestMatchers.allOf((r) -> true, (r) -> true);
+		boolean match = composed.matches(null);
+		assertThat(match).isTrue();
+	}
+
+	@Test
+	void checkAllOfWhenEmptyThenMatch() {
+		RequestMatcher composed = RequestMatchers.allOf();
+		boolean match = composed.matches(null);
+		assertThat(match).isTrue();
+	}
+
+	@Test
+	void checkNotWhenMatchThenNotMatch() {
+		RequestMatcher composed = RequestMatchers.not((r) -> true);
+		boolean match = composed.matches(null);
+		assertThat(match).isFalse();
+	}
+
+	@Test
+	void checkNotWhenNotMatchThenMatch() {
+		RequestMatcher composed = RequestMatchers.not((r) -> false);
+		boolean match = composed.matches(null);
+		assertThat(match).isTrue();
+	}
+
+}