[PAR-778] Bump integration-core to ~5.3.0#19
Merged
Conversation
Picks up StoreUrlProviderInterface (sequra/integration-core#62), which lets integrations expose a cheap storeUrl lookup for the webhook HMAC payload instead of paying the cost of building a full StoreInfo on every register, delete and inbound webhook validation. Fully backwards- compatible: existing integrations keep working unchanged via the legacy getStoreInfo() path.
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
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.
What is the goal?
Bumps
sequra/integration-corefrom~5.2.0to~5.3.0. v5.3.0 introduces an optionalStoreUrlProviderInterface(sequra/integration-core#62) that lets integrations expose a cheapgetStoreUrl()lookup for the webhook HMAC payload, avoiding the cost of building a fullStoreInfoon every register, delete and inbound webhook validation.The bump is fully backwards-compatible:
StoreUrlProviderInterfaceis a new optional companion toStoreInfoServiceInterface, andStoreInfoServiceInterfaceitself is unchanged — no integrator is forced to change anything to upgrade.References
How is it being implemented?
~5.2.0to~5.3.0.composer.lockrefreshed viacomposer update sequra/integration-core --with-dependencies. Only the core itself moved (v5.2.0 → v5.3.0); no transitive churn.src/change. The new interface is a downstream integrator-level opt-in; the middleware itself does not provide aStoreInfoServiceInterfaceimplementation, so nothing here needs to opt in.Caveats
StoreInfoServiceInterfacedo not need to change anything to upgrade — the new interface is pure opt-in.getStoreInfo()is expensive (live API calls to populate fields the HMAC does not use) can adopt by addingimplements StoreUrlProviderInterfacealongside their existingStoreInfoServiceInterfaceand returning a cheap storeUrl string. Signatures are byte-identical across both paths, so adoption never invalidates a previously-issued webhook URL.Does it affect (changes or update) any sensitive data?
No — this change only touches
composer.json/composer.lock. The HMAC payload, signature secret and inbound webhook validation behaviour upstream are unchanged.How is it tested?
composer validate --strictpasses after the bump.vendor/bin/phpunitwas not re-run locally (the local test env is not bootstrapped — pre-existing limitation unchanged by this PR). The middlewaresrc/is untouched, so existing tests are not affected.How is it going to be deployed?
Standard deployment. Tag the next minor (v3.2.0) once merged; downstream integrators pick it up via their existing
^3.0constraints onsequra/middleware.