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

Set default permission class to IsAuthenticated #2238

Closed
2 tasks
sainak opened this issue Jun 5, 2024 · 1 comment · Fixed by #2268
Closed
2 tasks

Set default permission class to IsAuthenticated #2238

sainak opened this issue Jun 5, 2024 · 1 comment · Fixed by #2268
Assignees
Labels
P0 Urgent

Comments

@sainak
Copy link
Member

sainak commented Jun 5, 2024

  • set permissions to [] on required public endpoints
  • remove explicit permission attribute if it IsAuthenticated only
@sainak sainak added the P0 Urgent label Jun 5, 2024
@sainak sainak self-assigned this Jun 6, 2024
@dhruv-goyal-10
Copy link
Contributor

@sainak Can you assign this issue to me?

dhruv-goyal-10 added a commit to dhruv-goyal-10/care that referenced this issue Jun 17, 2024
set permissions to [] on required public endpoints

removed explicit permission attribute if it IsAuthenticated only

Fixes ohcnetwork#2238
dhruv-goyal-10 added a commit to dhruv-goyal-10/care that referenced this issue Jun 17, 2024
set permissions to [] on required public endpoints

removed explicit permission attribute if it IsAuthenticated only

Fixes ohcnetwork#2238
dhruv-goyal-10 added a commit to dhruv-goyal-10/care that referenced this issue Jun 17, 2024
set permissions to [] on required public endpoints

removed explicit permission attribute if it IsAuthenticated only

Fixes ohcnetwork#2238
dhruv-goyal-10 added a commit to dhruv-goyal-10/care that referenced this issue Jun 17, 2024
set permissions to [] on required public endpoints

removed explicit permission attribute if it IsAuthenticated only

Fixes ohcnetwork#2238
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P0 Urgent
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants