CubeCoders / AMP

Issue tracking and documentation for AMP
https://cubecoders.com/AMP
207 stars 38 forks source link

Permissions Issue: File Manager #985

Open Cidvond opened 8 months ago

Cidvond commented 8 months ago

Bug Report

System Information

I confirm:

Symptoms

Reproduction

1) This is a fresh install of AMP
2) I created a template role
3) I deployed an application instance of Palworld
4) Go into the application instance
5) click configruation -> role managment -> and clicked create Template Role
6) Clicked on the new template role in the application instance -> File Manager -> assigned the following as green check marks:
    *Browse Files (FileManager.FileManager.BrowseFiles)
    *Download Files (FileManager.FileManager.DownloadFiles)
    *Upload Files (FileManager.FileManager.UploadFiles)
    *Rename Files (FileManager.FileManager.RenameFiles)
    *Copy Files (FileManager.FileManager.CopyFiles)
    *Trash Files (FileManager.FileManager.TrashFiles)
7) I then created a template for Palworld with the following settings:
    *Base Application = Palworld
    *Template Role = The one I just created 
    *Start Instance on Boot = Enabled
    *Clone Role Into User = disabled
    *Match datastore tags = disabled
8) apply changes
9) Deploy this Template
    *Deployment Type = Create New User
    *Select Template = Palworld
    *New Username = TheOldMan
    *Password = something random I make
    *Friendly Name = Palworld-ID (ID = numbers that count up starting at 1000)
    *Post creation = Do Nothing
10) Click Deploy Template
11) log in as the user TheOldMan and it shows the file manager on the left right after logging in. If I click on it, the File Manager will show all the files in the ADS including the ADS folder.
12) I have tried to reject(Red X)the following permissions in ADS -> Configuration -> Role Management -> Template Role I created, while keeping the template role permissions how they are to allow the instance application to still have File Manager. This had no change on anything and still allowed the user to have the assigned permissions outside of the Application Instance.
    *File Manager (FileManager.*)
Cidvond commented 8 months ago

AppInstancePermission ADSFilePermissions ADSFileAccess

IceOfWraith commented 8 months ago

Marked this one as a security issue like we discussed. Thanks for posting the bug.

PhonicUK commented 8 months ago

What happens if you explicitly deny access to the file manager at the ADS level?