Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add documentation about the secure serialization feature of RESTEasy Reactive #20741

Merged
merged 1 commit into from
Oct 14, 2021

Conversation

geoand
Copy link
Contributor

@geoand geoand commented Oct 13, 2021

No description provided.

@quarkus-bot

This comment has been minimized.

@sberyozkin
Copy link
Member

@geoand Hi Georgios, can you please add a mini section to security.adoc linking to the new resteasy-reactive section ? I've been trying to keep all the security features referenced from the main entry doc

@geoand
Copy link
Contributor Author

geoand commented Oct 13, 2021

Good idea, done!

@geoand
Copy link
Contributor Author

geoand commented Oct 14, 2021

Is the PR now good for you @sberyozkin?

@sberyozkin
Copy link
Member

Sorry @geoand I signed off yesterday probably immediately after this comment :-), 100% is good now :-)

@sberyozkin sberyozkin self-requested a review October 14, 2021 08:39
@sberyozkin sberyozkin merged commit 978060e into quarkusio:main Oct 14, 2021
@quarkus-bot quarkus-bot bot added this to the 2.5 - main milestone Oct 14, 2021
@geoand
Copy link
Contributor Author

geoand commented Oct 14, 2021

Thanks!

@geoand geoand deleted the secure-field-doc branch October 14, 2021 08:46

public class Person {

@SecureField("id")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is supposed to read:

@SecuredField(rolesAllowed = "admin")

I can supply a PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indeed! A PR would be welcome

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants