Skip to content

Http add verbs LINK and UNLINK#1230

Merged
roxblnfk merged 5 commits intospiral:masterfrom
gam6itko:http-verb-link
Jul 3, 2025
Merged

Http add verbs LINK and UNLINK#1230
roxblnfk merged 5 commits intospiral:masterfrom
gam6itko:http-verb-link

Conversation

@gam6itko
Copy link
Copy Markdown
Contributor

Why?

In FOSRestBundle we have support of LINK and UNLINK verbs.

But if I use this HTTP methods in sf3 then I get error

[Spiral\Router\Exception\RouteException]
Invalid HTTP verb `LINK` in vendor/spiral/framework/src/Router/src/Traits/VerbsTrait.php:25

https://www.w3.org/Protocols/HTTP/Methods/Link.html
https://www.w3.org/Protocols/HTTP/Methods/Unlink.html

@gam6itko gam6itko changed the title Http verb link Http add verbs LINK and UNLINK Jun 11, 2025
@gam6itko
Copy link
Copy Markdown
Contributor Author

gam6itko commented Jul 3, 2025

prefer lowest

spiral/attributes              3.1.8 PHP attributes reader
spiral/code-style              2.2.2 Code style and static analysis tools rulesets collection
spiral/composer-publish-plugin 1.1.2
spiral/nyholm-bridge           1.3.0 Spiral Framework: Nyholm PSR-7/PSR-17 bridge
spiral/testing                 2.8.3 Spiral Framework testing SDK
spiral/validator               1.5.4 Nested validation, Checkers, Conditional Validation

prefer stable

spiral/attributes              3.1.8 PHP attributes reader
spiral/code-style              2.2.2 Code style and static analysis tools rulesets collection
spiral/composer-publish-plugin 1.1.2
spiral/nyholm-bridge           1.3.0 Spiral Framework: Nyholm PSR-7/PSR-17 bridge
spiral/testing                 2.9.0 Spiral Framework testing SDK
spiral/validator               1.5.4 Nested validation, Checkers, Conditional Validation

@roxblnfk roxblnfk merged commit 3259b8c into spiral:master Jul 3, 2025
9 of 10 checks passed
@gam6itko gam6itko deleted the http-verb-link branch August 12, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants