Feature Request

Summary

Hi.

Is possible implement a message in the check Cpu Performance Informing the result performance in success or error case?

Thank you.

Bug Report

Summary

#16 added code does work in SecurityAdvisory::__construct() that causes temporary files to be created. Related: liip/LiipMonitorBundle#261.

Ping @paras-malhotra.

This issue lists Renovate updates and detected dependencies. Read the Dependency Dashboard docs to learn more.

Repository problems

These problems occurred while renovating this repository. View logs.

• WARN: Use matchDepNames instead of matchPackageNames

Edited/Blocked

These updates have been manually edited so Renovate will no longer make changes. To discard all commits and start over, click on a checkbox.

• Update dependency vimeo/psalm to v5

Open

These updates have all been created already. Click a checkbox below to force a retry/rebase of any.

• Update Laminas packages (laminas/laminas-ci-matrix-action, laminas/laminas-continuous-integration-action)
• Update dependency psalm/plugin-phpunit to ^0.19.0
• Update dependency enlightn/security-checker to v2
• Update dependency phpunit/phpunit to v11
• Lock file maintenance
• Click on this checkbox to rebase all open PRs at once

Detected dependencies

• Check this box to trigger a request for Renovate to run again on this repository

Feature Request

Summary

To be prepared for the december release of PHP 8.0, this repository has some additional TODOs to be tested against the new major version.

In order to make this repository compatible, one has to follow these steps:

• Modify composer.json to provide support for PHP 8.0 by adding the constraint ~8.0.0
• Modify composer.json to drop support for PHP less than 7.3
• Modify composer.json to implement phpunit 9.3 which supports PHP 7.3+
• Modify .travis.yml to ignore platform requirements when installing composer dependencies (simply add --ignore-platform-reqs to COMPOSER_ARGS env variable)
• Modify .travis.yml to add PHP 8.0 to the matrix (NOTE: Do not allow failures as PHP 8.0 has a feature freeze since 2020-08-04!)
• Modify source code in case there are incompatibilities with PHP 8.0

Feature Request

Summary

doctrine/migrations 3.0 has been released, so we'd need to add support for it here

Like https://packagist.org/packages/beberlei/assert or https://packagist.org/packages/peridot-php/leo

I would be happy to provide a PR if you accept such a contribution.

Originally posted by @sagikazarmark at zendframework/zend-diagnostics#51

RFC

Goal

Provide a structured class to use when providing extra details about a executed check.

Background

I'm looking into using this package to generate an API health endpoint, for that we'd need more data besides the fact that a check was successful. In previous PRs I already added some data to some checks, but got a review point to see if we can make this more structured/typed.

The idea is to add a ResultData object so Checks can provide extra data in a set format.

Considerations

Big question is, are the current Check's data returns covered by BC?
DiskUsage::check()->getData is now always an int, even though it's typed to mixed|null.

If we were to have a class available, these checks couldn't just switch over. Otherwise users of this package, who currently assume it's an int, will suddenly get something else.

Proposal(s)

Add a new ResultData object into Laminas\Diagnostics\Result, or Laminas\Diagnostics\Result\Data.
Preferably not simply key + value, but:

• key (uptime / responseTime / connections / utilization)
• value
• valueType (second, milisecond, percentage)

Quick proposal:

$data = new ResultData();
$data->addDetail(uptime, 500, seconds);

return new Success("Connected successfully", $data);

Appendix

I got this as review point in a previous PR: #57 (comment).
The PR was merged already (thanks for that), but I didn't want to 'forget about it'.

Bug Report

Summary

The dependency enlightn/security-checker should be in the require and not in the require-dev of Compoer.

Current behavior

After a composer update, I get this error:
Unable to find "Enlightn\SecurityChecker\AdvisoryAnalyzer" class. Please install "enlightn/security-checker" library to use this Check.

Could you plz set the enlightn/security-checker dependency in the require part of composer.
Thnaks

Bug Report

Summary

Currently the test-setup uses quite some time and resources to setup all the required external services like redis, memcached and mongo but then the tests actually using these services are skipped.

I tried to add the tests during enabling PHP8.3 testing but all tests failed - not only the 8.3 ones but all.

So this is something that should be done separately.

I managed to add the tests by adding this code into the qa-job in .github/workflows/continuous-integration.yml:

    env:
      TESTS_LAMINAS_DIAGNOSTICS_MEMCACHE_ENABLED: true
      TESTS_LAMINAS_DIAGNOSTICS_MEMCACHED_ENABLED: true
      TESTS_LAMINAS_DIAGNOSTICS_MONGO_ENABLED: true
      TESTS_LAMINAS_DIAGNOSTICS_RABBITMQ_ENABLED: true
      TESTS_LAMINAS_DIAGNOSTICS_REDIS_ENABLED: true

Current behavior

All tests are failing

How to reproduce

Add the yml excerpt to the described file and let the tests run

Expected behavior

All tests green

Bug Report

Summary

I am running php 7.2.22 with composer 1.9.3.
I have a project that require the package https://packagist.org/packages/liip/monitor-bundle#2.11.2
That package requires zendframework/zenddiagnostics ^1.5.

Current behavior

Somehow laminas-diagnostics is installed in version 1.6.
Code in the monitor-bundle breaks because it can't load the ZendDiagnostics\Runner\Reporter\ReporterInterface.
When I run composer why laminas/laminas-diagnostics it points to the liip/monitor-bundle.

I fail to undestand how laminas gets installed while the composer.json for liip/monitor-bundle version 2.11.2 never mention it. It explicitly requires zendframework/zenddiagnostics ^1.5.

Any idea what's going on ?

I fixed the issue by forcing the oro/health-check-bundle to install the liip/monitor-bundle v2.12.1 which require laminas. But I still want to figure out why the install with the version 2.11.2 of the package fails ( it install laminas but still use some zendframework interfaces)

How to reproduce

Installing the package oro/health-check-bundle in version 3.1.5 should trigger the same issue.

Expected behavior

That package zendframework/zenddiagnostics should be installed.

Hello!

Please look at https://github.com/queue-interop/queue-interop project. Using interfaces from it allows us reuse some implementations like enqueue, which supports a lot of transports. You can outsource some code

Originally posted by @makasim at zendframework/zend-diagnostics#85

Feature Request

Summary

As decided during the Technical-Steering-Committee Meeting on August 3rd, 2020, Laminas wants to implement vimeo/psalm in all packages.

Implementing psalm is quite easy.

Required

• Create a .psalm.xml.dist in the project root
• Copy and paste the contents from this psalm.xml.dist
• Run $ composer require vimeo/psalm
• Run $ vendor/bin/psalm --set-baseline=psalm-baseline.xml
• Add a composer script static-analysis with the command psalm --shepherd --stats
• Add a new line to script: in .travis.yml: - if [[ $TEST_COVERAGE == 'true' ]]; then composer static-analysis ; fi
• Remove phpstan from the project (phpstan.neon.dist, .travis.yml entry, composer.json require-dev and scripts)

Optional

• Fix as many psalm errors as possible.