-
Notifications
You must be signed in to change notification settings - Fork 4.8k
Font Library: fix endpoint permissions #54751
Copy link
Copy link
Closed
Labels
[Feature] TypographyFont and typography-related issues and PRsFont and typography-related issues and PRs[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Type] BugAn existing feature does not function as intendedAn existing feature does not function as intended
Description
It is necessary to fix the font library endpoints font permissions check.
Check the right folder
We need to check for write permissions in the right folder:
Currently, wp-content/upload/fonts/ is checked instead of wp/content/fonts.
Endpoints
GET /fonts/collections/GET /fonts/collections/<id>/
Should not check for write permissions on disk.
POST /fonts
Should check for write permissions on disk only if any font families received have font faces defined with reference to files:downloadFromUrloruploadedFile.
Should return HTTP error code 403 if there are no permissions.
DELETE /fonts
Should not check write permissions. The font asset will be try to be removed and if it fails it will return error for that particular font face.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
[Feature] TypographyFont and typography-related issues and PRsFont and typography-related issues and PRs[Status] In ProgressTracking issues with work in progressTracking issues with work in progress[Type] BugAn existing feature does not function as intendedAn existing feature does not function as intended
Type
Fields
Give feedbackNo fields configured for issues without a type.