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.

Sign up for GitHub

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

Jump to bottom

Bug: Files cannot be edited or API permissions changed when using Access Restriction or Header Authentication #210

Closed
TheFrazer opened this issue Nov 23, 2024 · 2 comments · Fixed by #211
Closed

Bug: Files cannot be edited or API permissions changed when using Access Restriction or Header Authentication #210

TheFrazer opened this issue Nov 23, 2024 · 2 comments · Fixed by #211
Labels
bug Something isn't working

Comments

@TheFrazer
Copy link

When using Access Restriction or Header Authentication as Authentication Method, trying to modify files results in a 401 Unauthorized Error.

I reinstalled it with a fresh config, chose Access Restriction, uploaded a file, and then tried to modify it (change password, etc.). It returned a 401, too, so it is not a configuration problem.

The request is send to /api/files/modify. I would guess that since no user is authenticated it wouldn't work. So when choosing Access Restriction is it not possible to modify files?
@Forceu Forceu added the bug Something isn't working label Nov 27, 2024
@Forceu
Copy link
Owner

Forceu commented Nov 27, 2024

Thanks for your feedback, I will look into it!

@Forceu Forceu mentioned this issue Nov 28, 2024
Merged
@Forceu Forceu pinned this issue Nov 28, 2024
@Forceu Forceu changed the title Modify Files with Access Restriction Files cannot be edited or API permissions changed when using Access Restriction or Header Authentication Nov 28, 2024
@Forceu Forceu changed the title Files cannot be edited or API permissions changed when using Access Restriction or Header Authentication Bug: Files cannot be edited or API permissions changed when using Access Restriction or Header Authentication Nov 28, 2024
Forceu added a commit that referenced this issue Nov 28, 2024
@Forceu
Changed delete button in upload menu to API call, fixed modify API ca…
a1ca369 
…ll in menu #210
Forceu added a commit that referenced this issue Dec 5, 2024
@Forceu
Use API for most of UI interactions with an internal API key (#211), …
ddc72b0 
…breaking API changes

* Changed color of API procressing status

* Added internal system key, use system key for API page perm changes, BREAKING: removed session auth

* API: Added /auth/delete, added option to include basic parameters in /auth/create, migrated API overview page completely to use API calls

* Added toast in API menu for clipboard, removed reference to session authentication in documentation

* Changed delete button in upload menu to API call, fixed modify API call in menu #210

* Set Api key in API menu for changing permissions, renamed title for Deleting Uploads permission, refactoring

* Added tests, refactoring
@Forceu
Copy link
Owner

Forceu commented Dec 5, 2024

Fixed in ddc72b0

@Forceu Forceu unpinned this issue Dec 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use API for most of UI interactions with an internal API key Forceu/Gokapi
2 participants
@Forceu @TheFrazer