feat: make signature generation more flexible by allowing relative URLs

[MichaelBrauner]

Here is the PR regarding this issue: #136

In the current implementation, the UriSigner always generates an absolute URL when signing. However, this might not be the desired behavior in some cases.

This PR adds a configuration option (use_relative_path) to allow the use of relative URLs when generating signatures. By default, the option is set to false, which maintains the existing behavior of generating absolute URLs.

[eidenschink]

This configuration option would be great. I have a use case where I want to target to a completely different host running on another framework; the hosts endpoint then proxies the request (or rather re-creates it) to the Symfony Controller.
If only the relative path is signed I can simply make the necessary hostname change in the TWIG template.

[shakaran]

@MichaelBrauner could you rebase for resolve the conflicts? Thanks

[MichaelBrauner]

Hi, this PR is now ready for review.

However, I wasn't able to run the test suite locally – vendor/bin/simple-phpunit fails with:

Trait "Symfony\Bridge\PhpUnit\Legacy\PolyfillAssertTrait" not found in Assert.php

I tried with PHP 8.1 and 8.2, removed and reinstalled the vendor folder and the PHPUnit Bridge multiple times, but no success.
Maybe you could verify if the test suite runs on your end?

Thanks!

[MichaelBrauner]

Can you rerun the checks?
It should pass: https://github.com/MichaelBrauner/verify-email-bundle/actions/runs/15507669530

[bocharsky-bw]

Sorry for the delay.

I reran the CI, but we still have errors here

[melbings]

@bocharsky-bw would you be so kind to re-run CI so that we can see the errors? The old logs have been archived.

[bocharsky-bw]

@melbings I reran the CI for you, I hope this helps! Though I see this PR now should be rebased due to some changes in master, and there are some conflicts that should be resolved too.

UPD: Yeah, probably the conflicts should be resolved first. If you want to follow up on this PR and push it forward - please, feel free to create a new PR based on these changes

[melbings]

@bocharsky-bw I'm afraid we're abandoning this pull request on our side. We will use Symfony's UriSigner component to deal with signed URLs instead. Nevertheless, thank you for your work and your support.

[ToshY]

@bocharsky-bw @jrushlow Any chance this can still be merged?

[melbings]

@bocharsky-bw @jrushlow Any chance this can still be merged?

I'm afraid that someone would need to resolve merge issues that have come up since we originally worked on this. Unfortunately, my company cannot work on this anymore. I assume that Claude & co. should have quite an easy time finishing this up, though.