Add test for security annotations in rest-data-panache

spring/spring-data/pom.xml

@@ -35,6 +35,10 @@
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-hibernate-validator</artifactId>
         </dependency>
+        <dependency>
+            <groupId>io.quarkus</groupId>
+            <artifactId>quarkus-elytron-security-properties-file</artifactId>
+        </dependency>
         <dependency>
             <groupId>io.quarkus.qe</groupId>
             <artifactId>quarkus-test-service-database</artifactId>

spring/spring-data/src/main/java/io/quarkus/ts/spring/data/rest/secured/DenyAllRepository.java

@@ -0,0 +1,27 @@
+package io.quarkus.ts.spring.data.rest.secured;
+
+import java.util.Optional;
+
+import javax.annotation.security.DenyAll;
+import javax.annotation.security.PermitAll;
+import javax.annotation.security.RolesAllowed;
+
+import org.springframework.data.repository.CrudRepository;
+import org.springframework.data.rest.core.annotation.RepositoryRestResource;
+import org.springframework.data.rest.core.annotation.RestResource;
+
+import io.quarkus.ts.spring.data.rest.Library;
+
+@RepositoryRestResource(path = "/secured/deny-all")
+@DenyAll
+public interface DenyAllRepository extends CrudRepository<Library, Long> {
+    @Override
+    @RestResource
+    @PermitAll
+    Iterable<Library> findAll();
+
+    @Override
+    @RestResource
+    @RolesAllowed("admin")
+    Optional<Library> findById(Long aLong);
+}

spring/spring-data/src/main/java/io/quarkus/ts/spring/data/rest/secured/PermitAllRepository.java

@@ -0,0 +1,21 @@
+package io.quarkus.ts.spring.data.rest.secured;
+
+import javax.annotation.security.DenyAll;
+import javax.annotation.security.PermitAll;
+
+import org.springframework.data.domain.Page;
+import org.springframework.data.domain.Pageable;
+import org.springframework.data.jpa.repository.JpaRepository;
+import org.springframework.data.rest.core.annotation.RepositoryRestResource;
+import org.springframework.data.rest.core.annotation.RestResource;
+
+import io.quarkus.ts.spring.data.rest.Library;
+
+@RepositoryRestResource(path = "/secured/permit-all")
+@PermitAll
+public interface PermitAllRepository extends JpaRepository<Library, Long> {
+    @Override
+    @RestResource
+    @DenyAll
+    Page<Library> findAll(Pageable pageable);
+}

spring/spring-data/src/main/java/io/quarkus/ts/spring/data/rest/secured/PublicRepository.java

@@ -0,0 +1,19 @@
+package io.quarkus.ts.spring.data.rest.secured;
+
+import javax.annotation.security.DenyAll;
+
+import org.springframework.data.domain.Page;
+import org.springframework.data.domain.Pageable;
+import org.springframework.data.repository.PagingAndSortingRepository;
+import org.springframework.data.rest.core.annotation.RepositoryRestResource;
+import org.springframework.data.rest.core.annotation.RestResource;
+
+import io.quarkus.ts.spring.data.rest.Library;
+
+@RepositoryRestResource(path = "/secured/public")
+public interface PublicRepository extends PagingAndSortingRepository<Library, Long> {
+    @Override
+    @RestResource
+    @DenyAll
+    Page<Library> findAll(Pageable pageable);
+}

spring/spring-data/src/main/java/io/quarkus/ts/spring/data/rest/secured/RolesAllowedRepository.java

@@ -0,0 +1,19 @@
+package io.quarkus.ts.spring.data.rest.secured;
+
+import java.util.Optional;
+
+import javax.annotation.security.PermitAll;
+import javax.annotation.security.RolesAllowed;
+
+import org.springframework.data.repository.CrudRepository;
+import org.springframework.data.rest.core.annotation.RepositoryRestResource;
+
+import io.quarkus.ts.spring.data.rest.Library;
+
+@RepositoryRestResource(path = "/secured/roles-allowed")
+@RolesAllowed("admin")
+public interface RolesAllowedRepository extends CrudRepository<Library, Long> {
+    @Override
+    @PermitAll
+    Optional<Library> findById(Long aLong);
+}

spring/spring-data/src/main/resources/application.properties

@@ -1,3 +1,12 @@
 quarkus.datasource.db-kind=postgresql
 quarkus.hibernate-orm.database.generation=drop-and-create
 quarkus.hibernate-orm.sql-load-script=import.sql
+
+# Basic security setup
+quarkus.http.auth.basic=true
+quarkus.security.users.embedded.enabled=true
+quarkus.security.users.embedded.plain-text=true
+quarkus.security.users.embedded.users.admin=admin
+quarkus.security.users.embedded.users.user=user
+quarkus.security.users.embedded.roles.admin=admin
+quarkus.security.users.embedded.roles.user=user

spring/spring-data/src/test/java/io/quarkus/ts/spring/data/rest/secured/OpenShiftSecuredRepositoryRestResourcesIT.java

@@ -0,0 +1,7 @@
+package io.quarkus.ts.spring.data.rest.secured;
+
+import io.quarkus.test.scenarios.OpenShiftScenario;
+
+@OpenShiftScenario
+public class OpenShiftSecuredRepositoryRestResourcesIT extends SecuredRepositoryRestResourcesIT {
+}

spring/spring-data/src/test/java/io/quarkus/ts/spring/data/rest/secured/SecuredRepositoryRestResourcesIT.java

@@ -0,0 +1,160 @@
+package io.quarkus.ts.spring.data.rest.secured;
+
+import static io.restassured.RestAssured.given;
+
+import org.apache.http.HttpStatus;
+import org.junit.jupiter.api.Tag;
+import org.junit.jupiter.api.Test;
+
+import io.quarkus.test.scenarios.QuarkusScenario;
+import io.quarkus.test.scenarios.annotations.DisabledOnQuarkusVersion;
+import io.quarkus.ts.spring.data.AbstractDbIT;
+import io.restassured.http.ContentType;
+
+@Tag("QUARKUS-2788")
+@QuarkusScenario
+public class SecuredRepositoryRestResourcesIT extends AbstractDbIT {
+    private static final long NONEXISTENT_ENTITY_ID = 999;
+    private static final String BASE_URL = "/secured";
+
+    private String getUrl(String path) {
+        return BASE_URL + path;
+    }
+
+    @Test
+    void publicResourceNoAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .get(getUrl("/public/" + NONEXISTENT_ENTITY_ID))
+                .then().statusCode(HttpStatus.SC_NOT_FOUND);
+    }
+
+    @Test
+    void publicResourceDenyAllMethodNoAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .get(getUrl("/public"))
+                .then().statusCode(HttpStatus.SC_UNAUTHORIZED);
+    }
+
+    @Test
+    void publicResourceDenyAllMethodAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .auth().basic("admin", "admin")
+                .get(getUrl("/public"))
+                .then().statusCode(HttpStatus.SC_FORBIDDEN);
+    }
+
+    @Test
+    void denyAllResourceNoAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .delete(getUrl("/deny-all/" + NONEXISTENT_ENTITY_ID))
+                .then().statusCode(HttpStatus.SC_UNAUTHORIZED);
+    }
+
+    @Test
+    void denyAllResourceAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .auth().basic("admin", "admin")
+                .delete(getUrl("/deny-all/" + NONEXISTENT_ENTITY_ID))
+                .then().statusCode(HttpStatus.SC_FORBIDDEN);
+    }
+
+    @Test
+    void denyAllResourcePermitAllMethodNoAuth() {
+        given()
+                .accept(ContentType.JSON)
+                .get(getUrl("/deny-all"))
+                .then().statusCode(HttpStatus.SC_OK);
+    }
+
+    @Test
+    void denyAllResourceRolesAllowedMethodNoAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .get(getUrl("/deny-all/" + NONEXISTENT_ENTITY_ID))
+                .then().statusCode(HttpStatus.SC_UNAUTHORIZED);
+    }
+
+    @Test
+    void denyAllResourceRolesAllowedMethodAuthForbidden() {
+        app.given()
+                .accept(ContentType.JSON)
+                .auth().basic("user", "user")
+                .get(getUrl("/deny-all/" + NONEXISTENT_ENTITY_ID))
+                .then().statusCode(HttpStatus.SC_FORBIDDEN);
+    }
+
+    @Test
+    void denyAllResourceRolesAllowedMethodAuthPermitted() {
+        app.given()
+                .accept(ContentType.JSON)
+                .auth().basic("admin", "admin")
+                .get(getUrl("/deny-all/" + NONEXISTENT_ENTITY_ID))
+                .then().statusCode(HttpStatus.SC_NOT_FOUND);
+    }
+
+    @Test
+    void permitAllResourceNoAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .get(getUrl("/permit-all/" + NONEXISTENT_ENTITY_ID))
+                .then().statusCode(HttpStatus.SC_NOT_FOUND);
+    }
+
+    @Test
+    void permitAllResourceDenyAllMethodNoAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .get(getUrl("/permit-all"))
+                .then().statusCode(HttpStatus.SC_UNAUTHORIZED);
+    }
+
+    @Test
+    void permitAllResourceDenyAllMethodAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .auth().basic("admin", "admin")
+                .get(getUrl("/permit-all"))
+                .then().statusCode(HttpStatus.SC_FORBIDDEN);
+    }
+
+    @Test
+    void rolesAllowedResourceNoAuth() {
+        app.given()
+                .accept(ContentType.JSON)
+                .get(getUrl("/roles-allowed"))
+                .then().statusCode(HttpStatus.SC_UNAUTHORIZED);
+    }
+
+    @Test
+    void rolesAllowedResourceAuthForbidden() {
+        app.given()
+                .accept(ContentType.JSON)
+                .auth().basic("user", "user")
+                .get(getUrl("/roles-allowed"))
+                .then().statusCode(HttpStatus.SC_FORBIDDEN);
+    }
+
+    @Test
+    void rolesAllowedResourceAuthPermitted() {
+        app.given()
+                .accept(ContentType.JSON)
+                .auth().basic("admin", "admin")
+                .get(getUrl("/roles-allowed"))
+                .then().statusCode(HttpStatus.SC_OK);
+    }
+
+    // Fix for https://github.com/quarkusio/quarkus/issues/30358 has been backported to Quarkus 2.13 and 2.16, but not to 2.14 and 2.15.
+    @DisabledOnQuarkusVersion(version = "(2\\.14\\..*)|(2\\.15\\..*)", reason = "https://github.com/quarkusio/quarkus/issues/30358")
+    @Test
+    void rolesAllowedResourcePermitAllMethodWithoutRestResourceAnnotation() {
+        app.given()
+                .accept(ContentType.JSON)
+                .get(getUrl("/roles-allowed/" + NONEXISTENT_ENTITY_ID))
+                .then().statusCode(HttpStatus.SC_NOT_FOUND);
+    }
+}

sql-db/panache-flyway/pom.xml

@@ -35,6 +35,10 @@
             <groupId>io.quarkus</groupId>
             <artifactId>quarkus-hibernate-validator</artifactId>
         </dependency>
+        <dependency>
+            <groupId>io.quarkus</groupId>
+            <artifactId>quarkus-elytron-security-properties-file</artifactId>
+        </dependency>
         <dependency>
             <groupId>org.testcontainers</groupId>
             <artifactId>mysql</artifactId>

sql-db/panache-flyway/src/main/java/io/quarkus/ts/sqldb/panacheflyway/secured/EntityDenyAllResource.java

@@ -0,0 +1,30 @@
+package io.quarkus.ts.sqldb.panacheflyway.secured;
+
+import java.util.List;
+
+import javax.annotation.security.DenyAll;
+import javax.annotation.security.PermitAll;
+import javax.annotation.security.RolesAllowed;
+
+import io.quarkus.hibernate.orm.rest.data.panache.PanacheEntityResource;
+import io.quarkus.panache.common.Page;
+import io.quarkus.panache.common.Sort;
+import io.quarkus.rest.data.panache.MethodProperties;
+import io.quarkus.rest.data.panache.ResourceProperties;
+import io.quarkus.ts.sqldb.panacheflyway.ApplicationEntity;
+
+@ResourceProperties(path = "/secured/entity/deny-all")
+@DenyAll
+public interface EntityDenyAllResource extends PanacheEntityResource<ApplicationEntity, Long> {
+    @Override
+    @PermitAll
+    long count();
+
+    @Override
+    @RolesAllowed("admin")
+    List<ApplicationEntity> list(Page page, Sort sort);
+
+    @Override
+    @MethodProperties(rolesAllowed = "admin")
+    ApplicationEntity get(Long aLong);
+}

sql-db/panache-flyway/src/main/java/io/quarkus/ts/sqldb/panacheflyway/secured/EntityPermitAllResource.java

@@ -0,0 +1,16 @@
+package io.quarkus.ts.sqldb.panacheflyway.secured;
+
+import javax.annotation.security.DenyAll;
+import javax.annotation.security.PermitAll;
+
+import io.quarkus.hibernate.orm.rest.data.panache.PanacheEntityResource;
+import io.quarkus.rest.data.panache.ResourceProperties;
+import io.quarkus.ts.sqldb.panacheflyway.ApplicationEntity;
+
+@ResourceProperties(path = "/secured/entity/permit-all")
+@PermitAll
+public interface EntityPermitAllResource extends PanacheEntityResource<ApplicationEntity, Long> {
+    @Override
+    @DenyAll
+    long count();
+}

sql-db/panache-flyway/src/main/java/io/quarkus/ts/sqldb/panacheflyway/secured/EntityPublicResource.java

@@ -0,0 +1,14 @@
+package io.quarkus.ts.sqldb.panacheflyway.secured;
+
+import javax.annotation.security.DenyAll;
+
+import io.quarkus.hibernate.orm.rest.data.panache.PanacheEntityResource;
+import io.quarkus.rest.data.panache.ResourceProperties;
+import io.quarkus.ts.sqldb.panacheflyway.ApplicationEntity;
+
+@ResourceProperties(path = "/secured/entity/public")
+public interface EntityPublicResource extends PanacheEntityResource<ApplicationEntity, Long> {
+    @Override
+    @DenyAll
+    long count();
+}

sql-db/panache-flyway/src/main/java/io/quarkus/ts/sqldb/panacheflyway/secured/EntityResourcePropertiesRolesAllowedResource.java

@@ -0,0 +1,9 @@
+package io.quarkus.ts.sqldb.panacheflyway.secured;
+
+import io.quarkus.hibernate.orm.rest.data.panache.PanacheEntityResource;
+import io.quarkus.rest.data.panache.ResourceProperties;
+import io.quarkus.ts.sqldb.panacheflyway.ApplicationEntity;
+
+@ResourceProperties(path = "/secured/entity/resource-properties-roles-allowed", rolesAllowed = "admin")
+public interface EntityResourcePropertiesRolesAllowedResource extends PanacheEntityResource<ApplicationEntity, Long> {
+}

sql-db/panache-flyway/src/main/java/io/quarkus/ts/sqldb/panacheflyway/secured/EntityRolesAllowedResource.java

@@ -0,0 +1,12 @@
+package io.quarkus.ts.sqldb.panacheflyway.secured;
+
+import javax.annotation.security.RolesAllowed;
+
+import io.quarkus.hibernate.orm.rest.data.panache.PanacheEntityResource;
+import io.quarkus.rest.data.panache.ResourceProperties;
+import io.quarkus.ts.sqldb.panacheflyway.ApplicationEntity;
+
+@ResourceProperties(path = "/secured/entity/roles-allowed")
+@RolesAllowed("admin")
+public interface EntityRolesAllowedResource extends PanacheEntityResource<ApplicationEntity, Long> {
+}