Add @PermissionsAllowed to Panache Resources about quarkus

[sberyozkin]

CC @michalvavrik

[quarkus-bot]

/cc @FroMage (panache), @geoand (rest-data-panache), @loicmathieu (panache)

[geoand]

Seems reasonable.

@geromueller would you be interested in contributing this feature?

[michalvavrik]

Unless @geromueller is interested, I'll have a look.

[michalvavrik]

FYI https://quarkus.io/guides/rest-data-panache#securing-endpoints documents @DenyAll, but this annotation is also not propagated (has no function at all...). Also other standard security annotations with exception of the @RolesAllowed are not supported ATM. I'll try to fix it as part of this issue.

[michalvavrik]

Apologies, there just wasn't any test for it nor security dep, it is correctly propagated. @geromueller the @PermissionsAllowed works as well. Maybe this is just a documentation issue. Did you assume it doesn't work because it is not documented? That can be fixed.

[michalvavrik]

@geromueller please provide reproducer or more context if you found a scenario where it does not work, thank you

[geromueller]

Hi! I actually assumed it. Is it implemented in reactive and classic as well? Thank you!

[michalvavrik]

Hi! I actually assumed it.

np

Is it implemented in reactive and classic as well? Thank you!

Yes, but I've used current main for testing, I can't speak for past versions (though it should work anyway). I'll document it and add some test to both classic and reactive.