Skip to content
This repository was archived by the owner on Feb 23, 2023. It is now read-only.

Spring security method missing hint #1518

Closed
benoit-laplante opened this issue Mar 2, 2022 · 2 comments · Fixed by #1523
Closed

Spring security method missing hint #1518

benoit-laplante opened this issue Mar 2, 2022 · 2 comments · Fixed by #1523
Assignees
Labels
type: compatibility Native image compatibility issue
Milestone

Comments

@benoit-laplante
Copy link

MethodSecurityExpressionRoot typehint DELCLARED_METHODS and DECLARED_FIELDS are missing to expose properties fieldObject and returnObject in spel expression like

@PostFilter("hasPermission(filterObject.id, 'POJO', read)") List<POJO> listAll();

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Mar 2, 2022
@sdeleuze
Copy link
Contributor

sdeleuze commented Mar 3, 2022

@eleftherias Do you think those should be configured out of the box?

@eleftherias
Copy link
Contributor

@sdeleuze Yes, I do think they should be configured out of the box.
I can update the GlobalMethodSecurityHints and add an example of this issue to one of the samples.

@eleftherias eleftherias self-assigned this Mar 4, 2022
@eleftherias eleftherias added type: compatibility Native image compatibility issue and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Mar 4, 2022
eleftherias added a commit to eleftherias/spring-native that referenced this issue Mar 4, 2022
Support use of filterObject and returnObject in PreFilter and PostFilter annotations.

Closes spring-atticgh-1518
@sdeleuze sdeleuze added this to the 0.11.4 milestone Mar 7, 2022
sdeleuze pushed a commit that referenced this issue Mar 7, 2022
Support use of filterObject and returnObject in PreFilter and PostFilter annotations.

Closes gh-1518
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: compatibility Native image compatibility issue
Development

Successfully merging a pull request may close this issue.

4 participants