Re: [pdm-devel] [PATCH proxmox 1/3] access-control: refactor api module to be more hirachical

["Shannon Sterz" <s.sterz@proxmox.com>]

On Fri Sep 26, 2025 at 10:26 AM CEST, Dominik Csapak wrote:
>
>
> On 9/24/25 4:51 PM, Shannon Sterz wrote:
>> this is mainly in preparation of factoring out more api endpoints
>> related to access control. the refactoring is done in a way where
>> users should not need to adapt.
>>
>> Signed-off-by: Shannon Sterz <s.sterz@proxmox.com>
>> ---
>>   proxmox-access-control/src/{api.rs => api/acl.rs} | 0
>>   proxmox-access-control/src/api/mod.rs             | 8 ++++++++
>>   2 files changed, 8 insertions(+)
>>   rename proxmox-access-control/src/{api.rs => api/acl.rs} (100%)
>>   create mode 100644 proxmox-access-control/src/api/mod.rs
>>
>> diff --git a/proxmox-access-control/src/api.rs b/proxmox-access-control/src/api/acl.rs
>> similarity index 100%
>> rename from proxmox-access-control/src/api.rs
>> rename to proxmox-access-control/src/api/acl.rs
>> diff --git a/proxmox-access-control/src/api/mod.rs b/proxmox-access-control/src/api/mod.rs
>> new file mode 100644
>> index 00000000..59dc32e2
>> --- /dev/null
>> +++ b/proxmox-access-control/src/api/mod.rs
>> @@ -0,0 +1,8 @@
>> +mod tokens;
>> +pub use tokens::{
>> +    API_METHOD_DELETE_TOKEN, API_METHOD_GENERATE_TOKEN, API_METHOD_LIST_TOKENS,
>> +    API_METHOD_READ_TOKEN, API_METHOD_UPDATE_TOKEN,
>> +};
>> +
>> +mod acl;
>> +pub use acl::{ACL_ROUTER, API_METHOD_READ_ACL, API_METHOD_UPDATE_ACL, ROLE_ROUTER};
>
> it seems the hunk here would belong into proxmox 3/3 since there is no
> 'tokens' module yet?

yes, sorry must have messed up in my haste on wednesday, will fix that
in a v2.


_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel