Rename Action::AllRead to AllGet

This commit is contained in:
Mubelotix 2025-06-19 11:55:25 +02:00
parent b6b7ede266
commit 9e1cb792f4
No known key found for this signature in database
GPG key ID: 89F391DBCC8CE7F0
2 changed files with 5 additions and 5 deletions

View file

@ -89,7 +89,7 @@ impl HeedAuthStore {
for action in &key.actions { for action in &key.actions {
match action { match action {
Action::All => actions.extend(enum_iterator::all::<Action>()), Action::All => actions.extend(enum_iterator::all::<Action>()),
Action::AllRead => actions.extend(enum_iterator::all::<Action>().filter(|a| a.is_read())), Action::AllGet => actions.extend(enum_iterator::all::<Action>().filter(|a| a.is_read())),
Action::DocumentsAll => { Action::DocumentsAll => {
actions.extend( actions.extend(
[Action::DocumentsGet, Action::DocumentsDelete, Action::DocumentsAdd] [Action::DocumentsGet, Action::DocumentsDelete, Action::DocumentsAdd]

View file

@ -151,7 +151,7 @@ impl Key {
name: Some("Default Read-Only Admin API Key".to_string()), name: Some("Default Read-Only Admin API Key".to_string()),
description: Some("Use it to peek into the instance in a read-only mode. Caution! Do not expose it on a public frontend. It would give access to all other keys".to_string()), description: Some("Use it to peek into the instance in a read-only mode. Caution! Do not expose it on a public frontend. It would give access to all other keys".to_string()),
uid, uid,
actions: vec![Action::AllRead], actions: vec![Action::AllGet],
indexes: vec![IndexUidPattern::all()], indexes: vec![IndexUidPattern::all()],
expires_at: None, expires_at: None,
created_at: now, created_at: now,
@ -235,7 +235,7 @@ pub enum Action {
All = 0, All = 0,
#[serde(rename = "*.get")] #[serde(rename = "*.get")]
#[deserr(rename = "*.get")] #[deserr(rename = "*.get")]
AllRead, AllGet,
#[serde(rename = "search")] #[serde(rename = "search")]
#[deserr(rename = "search")] #[deserr(rename = "search")]
Search, Search,
@ -421,7 +421,7 @@ impl Action {
// It's using an exhaustive match to force the addition of new actions. // It's using an exhaustive match to force the addition of new actions.
match self { match self {
// Any action that expands to others must return false, as it wouldn't be able to expand recursively. // Any action that expands to others must return false, as it wouldn't be able to expand recursively.
All | AllRead | DocumentsAll | IndexesAll | ChatsAll | TasksAll | SettingsAll All | AllGet | DocumentsAll | IndexesAll | ChatsAll | TasksAll | SettingsAll
| StatsAll | MetricsAll | DumpsAll | SnapshotsAll | ChatsSettingsAll => false, | StatsAll | MetricsAll | DumpsAll | SnapshotsAll | ChatsSettingsAll => false,
Search => true, Search => true,
@ -468,7 +468,7 @@ pub mod actions {
use super::Action::*; use super::Action::*;
pub(crate) const ALL: u8 = All.repr(); pub(crate) const ALL: u8 = All.repr();
pub const ALL_READ: u8 = AllRead.repr(); pub const ALL_READ: u8 = AllGet.repr();
pub const SEARCH: u8 = Search.repr(); pub const SEARCH: u8 = Search.repr();
pub const DOCUMENTS_ALL: u8 = DocumentsAll.repr(); pub const DOCUMENTS_ALL: u8 = DocumentsAll.repr();
pub const DOCUMENTS_ADD: u8 = DocumentsAdd.repr(); pub const DOCUMENTS_ADD: u8 = DocumentsAdd.repr();