θΏ™ζ˜―indexlocζδΎ›ηš„ζœεŠ‘οΌŒδΈθ¦θΎ“ε…₯任何密码
Skip to content

Conversation

@timothycarambat
Copy link
Member

Pull Request Type

  • ✨ feat
  • πŸ› fix
  • ♻️ refactor
  • πŸ’„ style
  • πŸ”¨ chore
  • πŸ“ docs

What is in this change?

Add explicit permissions on each flex and strict route
Patch issues with role escalation and CRUD of users
Patch permissions on all routes for coverage
Improve middleware to accept role array for clarity
Refactor various routes and paths
Add security to all routes where minimal security could be required.

Additional Information

Functionally, the roles and permissions are the same to the end user. This was a backend chore task that sets up AnythingLLM for greater permission access in future.

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

Add explicit permissions on each flex and strict route
Patch issues with role escalation and CRUD of users
Patch permissions on all routes for coverage
Improve middleware to accept role array for clarity
@Mintplex-Labs Mintplex-Labs deleted a comment from review-agent-prime bot Jan 20, 2024
@timothycarambat timothycarambat self-assigned this Jan 20, 2024
@timothycarambat timothycarambat merged commit 9a237db into master Jan 22, 2024
@timothycarambat timothycarambat deleted the permission-overhaul branch January 22, 2024 22:14
cabwds pushed a commit to cabwds/anything-llm that referenced this pull request Jul 3, 2025
* Implement total permission overhaul
Add explicit permissions on each flex and strict route
Patch issues with role escalation and CRUD of users
Patch permissions on all routes for coverage
Improve middleware to accept role array for clarity

* update comments

* remove permissions to API-keys for manager. Manager could generate API-key and using high-privelege api-key give themselves admin

* update sidebar permissions for multi-user and single user

* update options for mobile sidebar
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants