sync commits #2
Open
sync commits #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This allows other projects to extend the WebPush class and alter the way it works. An example use-case is to replace Guzzle with a ReactPHP based HTTP client for better integration in long-running asynchronous processes.
* Handling ConnectException * Syntax changes Co-authored-by: Louis Lagrange <lagrange.louis@gmail.com>
* Add #[\ReturnTypeWillChange] attribute for PHP 8.1 compatibility. * Update tab to space.
* Add #[\ReturnTypeWillChange] attribute for PHP 8.1 compatibility. * Update tab to space. * A different approach that works across all version.
* Add workflow file for tests on github actions * Remove travis ci * Check CI variable correctly and fix PHPStan issue * Stop running tests on PHP 7.2 * Use openssl_pkey_get_* methods on keys before passing to openssl_pkey_derive * Update link to badge in README * Reduce setup-php extensions to bare minimum * Remove PR branch from workflows file * Add builds for PHP 8.1 * Stop skipping PushServiceTest on CI * Run tests with web-push-testing * Remove invalid assert from PushServiceTest * Add fallback for running WebPushTest on CI environment * Remove PHP 7.2 from tests as that is no longer supported * Resolve phpstan errors * Skip running php-cs-fixer on PHP 8.1 PHP 8.1 is currently not supported by this. * Remove extra conversions of public & private key strings * Throw exception instead of silent fallback when unpack fails
* chore: allow jwt-framework 3.0 * fix: require 3.0 version which explicitly requires php 8.1
Update outdated reference for Mozilla's ServiceWorker cookbooks to point to the main Github repo
* Drop eol php 7.3 7.4 * Add php 8.2 to Github tests * test with disabled `gmp` extension * Add extension suggestion to composer * Remove outdated php 7.x code
* Code style: `php-cs-fixer` * Migrate tests to php 8.0 * add type hints * optimize imports * Add `tests/` to `php-cs-fixer` test * Add script for cs fix * Fix typos * Change `json_encode` throw error
Bumps the github-actions group with 1 update: [actions/checkout](https://github.com/actions/checkout). - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](actions/checkout@v3...v4) --- updated-dependencies: - dependency-name: actions/checkout dependency-type: direct:production update-type: version-update:semver-major dependency-group: github-actions ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps the github-actions group with 1 update: [actions/setup-node](https://github.com/actions/setup-node). - [Release notes](https://github.com/actions/setup-node/releases) - [Commits](actions/setup-node@v3...v4) --- updated-dependencies: - dependency-name: actions/setup-node dependency-type: direct:production update-type: version-update:semver-major dependency-group: github-actions ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* chore: drop eol php 8.0 * chore: update dependencies Drop EOL `guzzle` https://github.com/guzzle/guzzle#version-guidance * chore: update gh action * chore: migrate to phpunit 10 add type make functions static use attributes instead of annotations add strict_types disable new feature suppression of execution warning/error * feat: run all tests, ignore other test fails * feat: add php-cs-fixer rule * fix: remove outdated polyfill for php before 7.3 `openssl_pkey_derive` is always available with current php and openssl. * fix(cs): add dangling comma
* use https * point to final rfc * repo was renamed
test against an additional extension
simplify constructor
More specific exceptions
…390) Now requires openssl with elliptic curve support. This is the usual case.
Add deprecation of FCM Fix library does not send notification. It sends push message.
…g` (#397) * Replace `spomky-labs/base64url` with `paragonie/constant_time_encoding` * Directly using the `Base64UrlSafe` class * Add composer package `paragonie/constant_time_encoding`
* Make use of Guzzle Pool to improve efficiency * Fix documentation: add concurrency parameter where needed * fix checks: add a comma for php-cs-fixer * Add tests for flushPooled * refactor: rename concurrency parameter to requestConcurrency * readme: mention pooling for scaling
* Base64 tests * Fix test --------- Co-authored-by: Louis Lagrange <lagrange.louis@gmail.com>
* chore: allow jwt-library v4 * Update composer.json --------- Co-authored-by: Louis Lagrange <lagrange.louis@gmail.com>
* Update PHP CS Fixer * Remove \Random\RandomException since it doesn't exist in PHP 8.1 We will add it back when we drop support for 8.1
- allow phpunit@11 - upgrade to phpstan@2
phpdocs are deprecated in phpunit11
Co-authored-by: Louis Lagrange <lagrange.louis@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.