Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

Move call to check_access to call_if #1472

Merged
merged 7 commits into from
Jan 4, 2022

Conversation

chkeita
Copy link
Contributor

@chkeita chkeita commented Nov 22, 2021

Summary of the Pull Request

This PR moves the check for group membership before the deserialization of the request.
closes #1465

PR Checklist

Validation Steps Performed

running check-pr

@chkeita chkeita marked this pull request as ready for review November 29, 2021 18:38
@chkeita chkeita requested a review from stishkin December 1, 2021 18:59
@chkeita chkeita enabled auto-merge (squash) January 3, 2022 23:51
@chkeita chkeita merged commit 5515aa1 into microsoft:main Jan 4, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Feb 3, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move call to check_access to call_if
2 participants