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.

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

Added a cataloger for PHP PECL Packages #2604

Merged
merged 1 commit into from
Apr 2, 2024

Conversation

LaurentGoderre
Copy link
Contributor

@LaurentGoderre LaurentGoderre commented Feb 7, 2024

Add support for PHP PECL and PEAR extensions: https://pecl.php.net/

@LaurentGoderre LaurentGoderre changed the title wip Added a cataloger for PHP PECL Packages Feb 7, 2024
@LaurentGoderre LaurentGoderre force-pushed the php-pecl branch 2 times, most recently from dffa0db to a32740a Compare February 7, 2024 17:26
@github-actions github-actions bot added the json-schema Changes the json schema label Feb 7, 2024

This comment has been minimized.

@LaurentGoderre LaurentGoderre marked this pull request as ready for review February 7, 2024 18:20
@github-actions github-actions bot removed the json-schema Changes the json schema label Feb 7, 2024
@github-actions github-actions bot added the json-schema Changes the json schema label Feb 7, 2024
@LaurentGoderre LaurentGoderre force-pushed the php-pecl branch 2 times, most recently from 55d35a0 to c459aef Compare March 6, 2024 17:45
@LaurentGoderre
Copy link
Contributor Author

@pombredanne fyi

@LaurentGoderre LaurentGoderre force-pushed the php-pecl branch 2 times, most recently from eb48968 to c2e1d96 Compare March 28, 2024 13:18
Copy link
Contributor

@kzantow kzantow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @LaurentGoderre! I noted a couple of small things to update and the rest looks good; shout if you need help with any of it

syft/pkg/cataloger/php/parse_pecl_serialized.go Outdated Show resolved Hide resolved
syft/pkg/php.go Outdated Show resolved Hide resolved
@LaurentGoderre
Copy link
Contributor Author

@kzantow Updated

Copy link
Contributor

@kzantow kzantow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @LaurentGoderre! The schema looks good now. I left a few more comments about the type assertions -- these will need to get updated. Consider the other comments non-blocking

syft/pkg/cataloger/php/parse_pecl_serialized.go Outdated Show resolved Hide resolved
syft/pkg/cataloger/php/parse_pecl_serialized.go Outdated Show resolved Hide resolved
syft/pkg/cataloger/php/parse_pecl_serialized.go Outdated Show resolved Hide resolved
syft/pkg/cataloger/php/parse_pecl_serialized.go Outdated Show resolved Hide resolved
syft/pkg/cataloger/php/parse_pecl_serialized.go Outdated Show resolved Hide resolved
Signed-off-by: Laurent Goderre <laurent.goderre@docker.com>
@LaurentGoderre
Copy link
Contributor Author

@kzantow updated

Copy link
Contributor

@kzantow kzantow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks much @LaurentGoderre !

@kzantow kzantow merged commit e023362 into anchore:main Apr 2, 2024
11 checks passed
@LaurentGoderre LaurentGoderre deleted the php-pecl branch April 2, 2024 16:17
brian-ebarb pushed a commit to brian-ebarb/syft that referenced this pull request Apr 17, 2024
Signed-off-by: Laurent Goderre <laurent.goderre@docker.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
json-schema Changes the json schema
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants