-
-
Notifications
You must be signed in to change notification settings - Fork 108
Deprecate the PHAR metadata #1152
Copy link
Copy link
Closed
Description
Feature Request
Setting the PHAR metadata was supported because it was a PHAR option. I am however not convinced this is a good thing as:
- I never saw a good practical example.
- It used to expose to security risks and in a way still does. And by design there is no control about how the metadata is set.
One notable exception is @llaville's https://github.com/llaville/box-manifest. That being said the core of the project is to provide a manifest in different formats and I am not convinced using the metadata here is a good idea (and Box should offer another way than relying on the metadata setting to achieve this).
Thoughts @sebastianbergmann @theseer? Do you have a use case for the PHAR metadata?
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels