Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.6k
Fix form file field enctype#7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Closed
Closed
Uh oh!
There was an error while loading.Please reload this page.
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
dawehner pushed a commit to dawehner/symfony that referenced this pull requestJul 1, 2013
lots of cleanups while debugging an unrelated issue
dawehner pushed a commit to dawehner/symfony that referenced this pull requestJul 1, 2013
dunglas added a commit to dunglas/symfony that referenced this pull requestSep 21, 2015
Test with PHP7
SofHad pushed a commit to SofHad/symfony that referenced this pull requestOct 12, 2015
nicolas-grekas added a commit that referenced this pull requestNov 27, 2015
…ion on-demand (nicolas-grekas)This PR was merged into the 2.8 branch.Discussion----------[Bridge\PhpUnit] Display the stack trace of a deprecation on-demand| Q | A| ------------- | ---| Bug fix? | no| New feature? | yes| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets | -| License | MIT| Doc PR | -This PR has been essential into making#16708, and I think everyone working on a 3.0 migration will need this.Example:```$ SYMFONY_DEPRECATIONS_HELPER=/FormExtensionBootstrap3LayoutTest/ ./phpunit src/Symfony/Bridge/Twig/PHPUnit 4.8.18 by Sebastian Bergmann and contributors.Testing src/Symfony/Bridge/Twig/................................................Remaining deprecation triggered by Symfony\Bridge\Twig\Tests\Extension\FormExtensionBootstrap3LayoutTest::testSingleChoiceGrouped:The value "false" for the "choices_as_values" option is deprecated since version 2.8 and will not be supported anymore in 3.0. Set this option to "true" and flip the contents of the "choices" option instead.Stack trace:#0 src/Symfony/Component/Form/Extension/Core/Type/ChoiceType.php(286): trigger_error()#1 src/Symfony/Component/OptionsResolver/OptionsResolver.php(962): Symfony\Component\Form\Extension\Core\Type\ChoiceType->Symfony\Component\Form\Extension\Core\Type\{closure}()#2 src/Symfony/Component/Form/Extension/Core/Type/ChoiceType.php(277): Symfony\Component\OptionsResolver\OptionsResolver->offsetGet()#3 src/Symfony/Component/OptionsResolver/OptionsResolver.php(962): Symfony\Component\Form\Extension\Core\Type\ChoiceType->Symfony\Component\Form\Extension\Core\Type\{closure}()#4 src/Symfony/Component/OptionsResolver/OptionsResolver.php(791): Symfony\Component\OptionsResolver\OptionsResolver->offsetGet()#5 src/Symfony/Component/Form/ResolvedFormType.php(156): Symfony\Component\OptionsResolver\OptionsResolver->resolve()#6 src/Symfony/Component/Form/FormFactory.php(119): Symfony\Component\Form\ResolvedFormType->createBuilder()#7 src/Symfony/Component/Form/FormFactory.php(48): Symfony\Component\Form\FormFactory->createNamedBuilder()#8 src/Symfony/Component/Form/Tests/AbstractBootstrap3LayoutTest.php(540): Symfony\Component\Form\FormFactory->createNamed()#9 [internal function]: Symfony\Component\Form\Tests\AbstractBootstrap3LayoutTest->testSingleChoiceGrouped()#10 {main}KO src/Symfony/Bridge/Twig/```Commits-------5a88fb6 [Bridge\PhpUnit] Display the stack trace of a deprecation on-demand
jderusse pushed a commit to jderusse/symfony that referenced this pull requestMar 30, 2020
This PR was merged into the master branch.Discussion----------Fixed the versions for the FOSUserBundle advisoryThis fixes the impacted versions.I also found that the time you used in this files are using the Europe/Paris timezone, not UTC. Is it expected or is it a mistake ?Anyway, I used the same timezone when updating it.Commits-------dc0ddd1 Fixed the versions for the FOSUserBundle advisory
fabpot added a commit that referenced this pull requestSep 19, 2020
This PR was merged into the 5.2-dev branch.Discussion----------[VarDumper] Support for ReflectionAttribute| Q | A| ------------- | ---| Branch? | master| Bug fix? | no| New feature? | yes| Deprecations? | no| Tickets | N/A| License | MIT| Doc PR | not neededVarDumper currently does not understand that certain reflection objects might have attributes attached to it. Dumping a `ReflectionAttribute` just yields `ReflectionAttribute {#4711}` which is not really helpful. This PR attempts to fix this.```ReflectionAttribute {#4711 name: "App\MyAttribute" arguments: array:2 [ 0 => "one" "extra" => "hello" ]}```While working on this, I noticed that class constants (which can be reflected on since PHP 7.1) are just dumped as plain values, so I've also added a caster for `ReflectionClasConstant` as bonus.The full output for the `LotsOfAttributes` fixture class that is included with is PR looks like this:<details>```^ ReflectionClass {#7 +name: "Symfony\Component\VarDumper\Tests\Fixtures\LotsOfAttributes" modifiers: "final" implements: [] constants: array:1 [ 0 => ReflectionClassConstant {#20 +name: "SOME_CONSTANT" +class: "Symfony\Component\VarDumper\Tests\Fixtures\LotsOfAttributes" modifiers: "public" value: "some value" attributes: array:2 [ 0 => ReflectionAttribute {#33 name: "Symfony\Component\VarDumper\Tests\Fixtures\RepeatableAttribute" arguments: array:1 [ 0 => "one" ] } 1 => ReflectionAttribute {#34 name: "Symfony\Component\VarDumper\Tests\Fixtures\RepeatableAttribute" arguments: array:1 [ 0 => "two" ] } ] } ] properties: array:1 [ "someProperty" => ReflectionProperty {#19 +name: "someProperty" +class: "Symfony\Component\VarDumper\Tests\Fixtures\LotsOfAttributes" modifiers: "private" attributes: array:1 [ 0 => ReflectionAttribute {#30 name: "Symfony\Component\VarDumper\Tests\Fixtures\MyAttribute" arguments: array:2 [ 0 => "one" "extra" => "hello" ] } ] } ] methods: array:1 [ "someMethod" => ReflectionMethod {#21 +name: "someMethod" +class: "Symfony\Component\VarDumper\Tests\Fixtures\LotsOfAttributes" returnType: "void" parameters: { $someParameter: ReflectionParameter {#28 +name: "someParameter" position: 0 attributes: array:1 [ 0 => ReflectionAttribute {#42 name: "Symfony\Component\VarDumper\Tests\Fixtures\MyAttribute" arguments: array:1 [ 0 => "three" ] } ] typeHint: "string" } } attributes: array:1 [ 0 => ReflectionAttribute {#27 name: "Symfony\Component\VarDumper\Tests\Fixtures\MyAttribute" arguments: array:1 [ 0 => "two" ] } ] modifiers: "public" } ] attributes: array:1 [ 0 => ReflectionAttribute {#22 name: "Symfony\Component\VarDumper\Tests\Fixtures\MyAttribute" arguments: [] } ] extra: { file: "./src/Symfony/Component/VarDumper/Tests/Fixtures/LotsOfAttributes.php" line: "15 to 28" isUserDefined: true }}```</details>Commits-------34dbf01 [VarDumper] Support for ReflectionAttribute.
OskarStark pushed a commit to PGLongo/symfony that referenced this pull requestDec 22, 2020
# Das ist die erste Commit-Beschreibung:[Notifier] add support for gatewayapi-notifier# Die Commit-Beschreibungsymfony#2 wird ausgelassen:# Make GatewayAPITransportFactory final# Die Commit-Beschreibungsymfony#3 wird ausgelassen:# Fix typo in gatewaypi# Die Commit-Beschreibungsymfony#4 wird ausgelassen:# Fix typo# Die Commit-Beschreibungsymfony#5 wird ausgelassen:# Update README.md# Die Commit-Beschreibungsymfony#6 wird ausgelassen:# Rename GatewayAPI -> GatewayApi# Die Commit-Beschreibungsymfony#7 wird ausgelassen:# Add GatewayApiTransportFactory tests# Die Commit-Beschreibungsymfony#8 wird ausgelassen:# Add GatewayApiTransportTest testSend# Die Commit-Beschreibungsymfony#9 wird ausgelassen:# Add GatewayApiTransportTest testSupportsSmsMessage# Die Commit-Beschreibungsymfony#10 wird ausgelassen:# Add GatewayApiTransportTest testNotSupportsChatMessage# Die Commit-Beschreibungsymfony#11 wird ausgelassen:# Lint code# Die Commit-Beschreibungsymfony#12 wird ausgelassen:# Fix name and email in composer.json## Co-authored-by: Tobias Nyholm <tobias.nyholm@gmail.com># Die Commit-Beschreibungsymfony#13 wird ausgelassen:# Remove extra from composer.json# Die Commit-Beschreibungsymfony#14 wird ausgelassen:# Update version tag in GatewayApiTransport# Die Commit-Beschreibungsymfony#15 wird ausgelassen:# Add /.gitattributes export-ignore in .gitattributes# Die Commit-Beschreibungsymfony#16 wird ausgelassen:# Update UnsupportedSchemeException# Die Commit-Beschreibungsymfony#17 wird ausgelassen:# Add required space in phpdoc# Die Commit-Beschreibungsymfony#18 wird ausgelassen:# Update GatewayApiTransportFactory.php## Update version tag in GatewayApiTransportFactory# Die Commit-Beschreibungsymfony#19 wird ausgelassen:# [Notifier] add support for gatewayapi-notifier# Die Commit-Beschreibungsymfony#20 wird ausgelassen:# Make GatewayAPITransportFactory final# Die Commit-Beschreibungsymfony#21 wird ausgelassen:# Make GatewayAPITransportFactory final# Die Commit-Beschreibungsymfony#22 wird ausgelassen:# Fix typo in gatewaypi# Die Commit-Beschreibungsymfony#23 wird ausgelassen:# Fix typo# Die Commit-Beschreibungsymfony#24 wird ausgelassen:# Update README.md
wouterj added a commit that referenced this pull requestFeb 19, 2021
This PR was merged into the 5.3-dev branch.Discussion----------[Security] Added debug:firewall command| Q | A| ------------- | ---| Branch? | 5.x| Bug fix? | no| New feature? | yes| Deprecations? | no| Tickets |Fix#39321| License | MIT| Doc PR |symfony/symfony-docs#14982| Tags | #SymfonyHackday### Subtasks- [x] Add list view (for use without arguments)- [x] Add more information to list view- [x] Add detail view (for use with `firewall` argument)- [x] Add more information to detail view table- [x] Add authenticators list- [x] Add event listeners & events (copy from `debug:event-listener`)- [x] Add `--include-listeners` option (default: false)- [x] Add helptext- [x] Add documentation### Moved outside of current scope- Add allowed badges### Usage (and example output) for a list`bin/console debug:firewall````Firewalls========= The following firewalls are defined: -------- Name -------- dev public main -------- // To view details of a specific firewall, re-run this command with a firewall name. (e.g. debug:firewall // main)```### Usage (and example output) for details`bin/console debug:firewall main````Firewall "main"=============== ----------------------- --------------------------------------------------- Option Value ----------------------- --------------------------------------------------- Name main Context main Lazy Yes Stateless No User Checker security.user_checker Provider security.user.provider.concrete.app_user_provider Entry Point App\Security\LoginFormAuthenticator Access Denied URL Access Denied Handler ----------------------- ---------------------------------------------------User switching-------------- ----------- --------------------------------------------------- Option Value ----------- --------------------------------------------------- Parameter test Provider security.user.provider.concrete.app_user_provider User Role ROLE_SWITCH_POSSIBLE ----------- ---------------------------------------------------Event listeners for firewall "main"==================================="Symfony\Component\Security\Http\Event\LoginSuccessEvent" event--------------------------------------------------------------- ------- -------------------------------------------------------------------------------------------- ---------- Order Callable Priority ------- -------------------------------------------------------------------------------------------- ----------#1 Symfony\Component\Security\Http\EventListener\UserCheckerListener::postCheckCredentials() 256#2 Symfony\Component\Security\Http\EventListener\SessionStrategyListener::onSuccessfulLogin() 0#3 Symfony\Component\Security\Http\EventListener\RememberMeListener::onSuccessfulLogin() 0#4 App\Security\UpdateLastLogin::__invoke() 0#5 Symfony\Component\Security\Http\EventListener\PasswordMigratingListener::onLoginSuccess() 0 ------- -------------------------------------------------------------------------------------------- ----------"Symfony\Component\Security\Http\Event\LogoutEvent" event--------------------------------------------------------- ------- ------------------------------------------------------------------------------------------- ---------- Order Callable Priority ------- ------------------------------------------------------------------------------------------- ----------#1 Symfony\Component\Security\Http\EventListener\DefaultLogoutListener::onLogout() 64#2 Symfony\Component\Security\Http\EventListener\SessionLogoutListener::onLogout() 0#3 Symfony\Component\Security\Http\EventListener\RememberMeLogoutListener::onLogout() 0#4 Symfony\Component\Security\Http\EventListener\CsrfTokenClearingLogoutListener::onLogout() 0 ------- ------------------------------------------------------------------------------------------- ----------"Symfony\Component\Security\Http\Event\CheckPassportEvent" event---------------------------------------------------------------- ------- ------------------------------------------------------------------------------------------ ---------- Order Callable Priority ------- ------------------------------------------------------------------------------------------ ----------#1 Symfony\Component\Security\Http\EventListener\LoginThrottlingListener::checkPassport() 2080#2 Symfony\Component\Security\Http\EventListener\UserProviderListener::checkPassport() 2048#3 Symfony\Component\Security\Http\EventListener\UserProviderListener::checkPassport() 1024#4 Symfony\Component\Security\Http\EventListener\CsrfProtectionListener::checkPassport() 512#5 Symfony\Component\Security\Http\EventListener\UserCheckerListener::preCheckCredentials() 256#6 App\Security\DisallowBannedUsers::__invoke() 0#7 Symfony\Component\Security\Http\EventListener\CheckCredentialsListener::checkPassport() 0 ------- ------------------------------------------------------------------------------------------ ----------"Symfony\Component\Security\Http\Event\LoginFailureEvent" event--------------------------------------------------------------- ------- ----------------------------------------------------------------------------------- ---------- Order Callable Priority ------- ----------------------------------------------------------------------------------- ----------#1 Symfony\Component\Security\Http\EventListener\RememberMeListener::onFailedLogin() 0 ------- ----------------------------------------------------------------------------------- ----------Authenticators for firewall "main"================================== //@todo: List authenticator information```Commits-------a9dea1d [Security] Added debug:firewall command
derrabus pushed a commit to derrabus/symfony that referenced this pull requestJul 25, 2023
This PR was merged into the master branch.Discussion----------Test Diactoros Factory with PHP 5.4They now support PHP 5.4 too.Commits-------bab1530 Test Diactoros Factory with PHP 5.4
fabpot added a commit that referenced this pull requestJul 30, 2023
…repo (fabpot, dunglas, KorvinSzanto, xabbuh, aimeos, ahundiak, Danielss89, rougin, csunolgomez, Jérôme Parmentier, mtibben, Nyholm, ajgarlag, uphlewis, samnela, grachevko, nicolas-grekas, tinyroy, danizord, Daniel Degasperi, rbaarsma, Ekman, 4rthem, derrabus, mleczakm, iluuu1994, Tobion, chalasr, lemon-juice, franmomu, cidosx, erikn69, AurelienPillevesse)This PR was merged into the 6.4 branch.Discussion----------[PsrHttpMessageBridge] Import the bridge into the monorepo| Q | A| ------------- | ---| Branch? | 6.4| Bug fix? | no| New feature? | yes| Deprecations? | no| Tickets | N/A| License | MIT| Doc PR | TODO⚠️ Don't squash!I propose to import the `symfony/psr-http-message-bridge` package into the Symfony monorepo for further maintenance.Commits-------e40dd66 [PsrHttpMessageBridge] Patch return types and fix CS266c09f [PsrHttpMessageBridge] Import the bridge into the monorepo0c0323a Add 'src/Symfony/Bridge/PsrHttpMessage/' from commit '581ca6067eb62640de5ff08ee1ba6850a0ee472e'581ca60 Prepare release 2.3.145d0349 Fix CS6410dda bug#122 Don't rely on Request::getPayload() to populate the parsed body (nicolas-grekas)ef03b6d Don't rely on Request::getPayload() to populate the parsed body3c62b81 minor#120 Prepare release 2.3.0 (derrabus)96acbfd Prepare release 2.3.07eedd34 feature#119 Implement ValueResolverInterface (derrabus)0b54b85 Implement ValueResolverInterface6b2f5df feature#117 Leverage `Request::getPayload()` to populate the parsed body of PSR-7 requests (AurelienPillevesse)3a8caad Leverage `Request::getPayload()` to populate the parsed body of PSR-7 requests18c9e82 minor#118 Add native types where possible (derrabus)4fd4323 Add native types where possible28a732c minor#115 Prepare the 2.2.0 release (derrabus)7944831 cs fix99ddcaa Prepare the 2.2.0 release8a5748d feature#113 Bump psr/http-message version (erikn69)ec83c1c Bump psr/http-message version694016e feature#114 Drop support for Symfony 4 (derrabus)b360b35 Drop support for Symfony 4998d8d2 minor#111 Adjustments for PHP CS Fixer 3 (derrabus)5fa5f62 Adjustments for PHP CS Fixer 3a125b93 minor#110 Add PHP 8.2 to CI (derrabus)4592df2 Add PHP 8.2 to CI4617ac3 bug#109 perf: ensure timely flush stream buffers (cidosx)8c8a75b perf: ensure timely flush stream buffersd444f85 Update changelog155a7ae bug#107 Ignore invalid HTTP headers when creating PSR7 objects (nicolas-grekas)9a78a16 Ignore invalid HTTP headers when creating PSR7 objectsbdb2871 minor#104 Add missing .gitattributes (franmomu)808561a Add missing .gitattributes316f5cb bug#103 Fix for wrong type passed to moveTo() (lemon-juice)7f3b5c1 Fix for wrong type passed to moveTo()22b37c8 minor#101 Release v2.1.2 (chalasr)c382d76 Release v2.1.2c81476c feature#100 Allow Symfony 6 (chalasr)c7a0be3 Allow Symfony 6df83a38 minor#98 Add PHP 8.1 to CI (derrabus)b2bd334 Add PHP 8.1 to CI824711c minor#99 Add return types to fixtures (derrabus)f8f70fa Add return types to fixturesd558dcd minor#97 Inline $tmpDir (derrabus)d152649 Inline $tmpDirf12a9e6 minor#96 Run PHPUnit on GitHub Actions (derrabus)ab64c69 Run PHPUnit on GitHub Actionsc901299 bug#95 Allow `psr/log` 2 and 3 (derrabus)8e13ae4 Allow psr/log 2 and 326068fa Minor cleanups87fabb9 Fix copyright year3d9241f minor#92 remove link to sensio extra bundle which removed psr7 support (Tobion)7078739 remove link to sensio extra bundle which removed psr7 support81db2d4 feature#89 PSR HTTP message converters for controllers (derrabus)aa26e61 PSR HTTP message converters for controllerse62b239 minor#91 Fix CS (derrabus)2bead22 Fix CS488df9b minor#90 Fix CI failures with Xdebug 3 and test on PHP 7.4/8.0 as well (derrabus)a6697fd Fix CI failures with Xdebug 3 and test on PHP 7.4/8.0 as wellc62f7d0 Update branch-alias51a21cb Update changeloga20fff9 bug#87 Fix populating server params from URI in HttpFoundationFactory (nicolas-grekas)4933e04 bug#86 Create cookies as raw in HttpFoundationFactory (nicolas-grekas)66095a5 Fix populating server params from URI in HttpFoundationFactory42cca49 Create cookies as raw in HttpFoundationFactorycffb3a8 bug#85 Fix BinaryFileResponse with range to psr response conversion (iluuu1994)5d5932d Fix BinaryFileResponse with range to psr response conversione44f249 bug#81 Don't normalize query string in PsrHttpFactory (nicolas-grekas)bc25829 Don't normalize query string in PsrHttpFactorydf735ec bug#78 Fix populating default port and headers in HttpFoundationFactory (mleczakm)4f30401 Fix populating default port and headers in HttpFoundationFactory1309b64 bug#77 fix conversion for https requests (4rthem)e86de3f minor#79 Allow installation on php 8 (derrabus)9243f93 Allow installation on php 8.d336c73 fix conversion for https requests126903c Fix format of CHANGELOG.mdce709cd feature#75 Remove deprecated code (fabpot)dfc5238 Remove deprecated code9d3e80d bug#72 Use adapter for UploadedFile objects (nicolas-grekas)a4f9f6d Use adapter for UploadedFile objectsec7892b Fix CHANGELOG, bump branch-alias7ab4fe4 minor#70 Updated CHANGELOG (rbaarsma)9ad4bcc Updated CHANGELOGc4c904a minor#71 Cleanup after bump to Symfony v4.4 (nicolas-grekas)e9a9557 Cleanup after bump to Symfony v4.43d10a6c feature#66 Add support for streamed Symfony request (Ekman)df26630 Add support for streamed Symfony request5aa8ca9 bug#69 Allow Symfony 5.0 (rbaarsma)1158149 Allow Symfony 5.081ae86d Merge branch '1.1'a33352a bug#64 Fixed createResponse (ddegasperi)7a4b449 minor#65 Fix tests (ajgarlag)19905b0 Fix tests580de38 Fixed createResponse9ab9d71 minor#63 Added links to documentation (Nyholm)59b9406 Added links to documentationc1cb51c feature#50 Add support for streamed response (danizord)4133c7a bug#48 Convert Request/Response multiple times (Nyholm)8564bf7 Convert Request/Response multiple times7cc1605 Add support for streamed responseaebc14b feature#62 bump to PHP 7.1 (nicolas-grekas)8e10923 bump to PHP 7.15e5e0c3 Revert "Undeprecate DiactorosFactory for 1.1"921f866 Undeprecate DiactorosFactory for 1.18592ca3 bug#61 removed 'Set-Cookie' from header when it is already converted to a Symfony header cookie (tinyroy)dd1111e removed 'Set-Cookie' from header when it is already converted to a Symfony header cookieba672d8 bump branch-alias5f9a032 typof2c48c5 fix tests3a52e44 bug#59 Fix SameSite attribute conversion from PSR7 to HttpFoundation (ajgarlag)5ee1f8f Fix SameSite attribute conversion from PSR7 to HttpFoundationf6d7d3a bug#58 [Bugfix] Typo header set-sookie (grachevko)16eb6e1 minor#57 Excluded tests from classmap (samnela)36a8065 Deprecate DiactorosFactory, use nyholm/psr7 for tests5076934 bug#54Fix#51 (compatability issue with zendframework/zend-diactoros ^2.0) (uphlewis)757ea81 [Bugfix] Typo header set-sookie25f9c3a Excluded tests from classmap8ff61e5 Fix compatability issue with "zendframework/zend-diactoros": "^2.0." (#51)53c15a6 updated CHANGELOGc821241 bumped version to 1.1f26d01f minor#47 Updated changelog (Nyholm)c2282e3 Updated changelogeddd6c8 feature#43 Create PSR-7 messages using PSR-17 factories (ajgarlag)dd81b4b Create PSR-7 messages using PSR-17 factoriesf11f173 feature#45 Fixed broken build (Nyholm)8780dd3 Fixed broken buildc2b7579 bug#30 Fix the request target in PSR7 Request (mtibben)94fcfa5 Fix the request target in PSR7 Request64640ee minor#38 Run PHP 5.3 tests on Precise (Lctrs)64c0cb0 Run PHP 5.3 tests on Preciseb209840 minor#32 Allow Symfony 4 (dunglas)97635f1 Allow Symfony 4147a238 minor#31 test suite compatibility with PHPUnit 6 (xabbuh)f5c46f0 test suite compatibility with PHPUnit 666085f2 preparing 1.0 release533d3e4 added a CHANGELOG for 1.014269f9 bug#28 Fix REQUEST_METHOD on symfony request (csunol)98ab85a Fix REQUEST_METHOD on symfony request29be4f8 updated LICENCE yeard2db47c removed obsolete CHANGELOG file1c30b17 bug#22Fixes#16 Symfony Request created without any URI (rougin)a59c572Fixes#16 Symfony Request created without any URI7a5aa92 bug#23Fixes#9 Bridge error when no file is selected (ahundiak, Danielss89)a1a631a Update assert error messagee5d62e6 Fixes based on code-review101b608 Handles null file in createrequest bridge.d16c63c bug#18 Allow multiple calls to Request::getContent() (aimeos)9624b8b Allow multiple calls to Request::getContent()9c747c4 Merge pull request#19 from xabbuh/travis-configa388c43 update Travis CI configurationac5cd86 minor#14 Remove use of deprecated 'deep' parameter in tests (KorvinSzanto)305c0fe Remove use of deprecated 'deep' parameter3664dc0 minor#7 Test Diactoros Factory with PHP 5.4 (dunglas)bab1530 Test Diactoros Factory with PHP 5.4d7660b8 Suggest psr/http-message-implementationdc7e308 removed the branch alias for now as we are pre 1.03f8977e feature#1 Initial support (dunglas)ca41146 Initial support01b110b added the initial set of files
chalasr added a commit that referenced this pull requestMar 5, 2024
…lishing a message. (jwage)This PR was squashed before being merged into the 6.4 branch.Discussion----------[Messenger] [Amqp] Handle AMQPConnectionException when publishing a message.| Q | A| ------------- | ---| Branch? | 6.4| Bug fix? | yes| New feature? | no| Deprecations? | no| Issues |Fix#36538Fix#48241| License | MITIf you have a message handler that dispatches messages to another queue, you can encounter `AMQPConnectionException` with the message "Library error: a SSL error occurred" or "a socket error occurred" depending on if you are using tls or not or if you are running behind a load balancer or not.You can manually reproduce this issue by dispatching a message where the handler then dispatches another message to a different queue, then go to rabbitmq admin and close the connection manually, then dispatch another message and when the message handler goes to dispatch the other message, you will get this exception:```a socket error occurred#0 /vagrant/vendor/symfony/amqp-messenger/Transport/AmqpTransport.php(60): Symfony\Component\Messenger\Bridge\Amqp\Transport\AmqpSender->send()#1 /vagrant/vendor/symfony/messenger/Middleware/SendMessageMiddleware.php(62): Symfony\Component\Messenger\Bridge\Amqp\Transport\AmqpTransport->send()#2 /vagrant/vendor/symfony/messenger/Middleware/FailedMessageProcessingMiddleware.php(34): Symfony\Component\Messenger\Middleware\SendMessageMiddleware->handle()#3 /vagrant/vendor/symfony/messenger/Middleware/DispatchAfterCurrentBusMiddleware.php(61): Symfony\Component\Messenger\Middleware\FailedMessageProcessingMiddleware->handle()#4 /vagrant/vendor/symfony/messenger/Middleware/RejectRedeliveredMessageMiddleware.php(41): Symfony\Component\Messenger\Middleware\DispatchAfterCurrentBusMiddleware->handle()#5 /vagrant/vendor/symfony/messenger/Middleware/AddBusNameStampMiddleware.php(37): Symfony\Component\Messenger\Middleware\RejectRedeliveredMessageMiddleware->handle()#6 /vagrant/vendor/symfony/messenger/Middleware/TraceableMiddleware.php(40): Symfony\Component\Messenger\Middleware\AddBusNameStampMiddleware->handle()#7 /vagrant/vendor/symfony/messenger/MessageBus.php(70): Symfony\Component\Messenger\Middleware\TraceableMiddleware->handle()#8 /vagrant/vendor/symfony/messenger/TraceableMessageBus.php(38): Symfony\Component\Messenger\MessageBus->dispatch()#9 /vagrant/src/Messenger/MessageBus.php(37): Symfony\Component\Messenger\TraceableMessageBus->dispatch()#10 /vagrant/vendor/symfony/mailer/Mailer.php(66): App\Messenger\MessageBus->dispatch()#11 /vagrant/src/Mailer/Mailer.php(83): Symfony\Component\Mailer\Mailer->send()#12 /vagrant/src/Mailer/Mailer.php(96): App\Mailer\Mailer->send()#13 /vagrant/src/MessageHandler/Trading/StrategySubscriptionMessageHandler.php(118): App\Mailer\Mailer->sendEmail()#14 /vagrant/src/MessageHandler/Trading/StrategySubscriptionMessageHandler.php(72): App\MessageHandler\Trading\StrategySubscriptionMessageHandler->handle()#15 /vagrant/vendor/symfony/messenger/Middleware/HandleMessageMiddleware.php(152): App\MessageHandler\Trading\StrategySubscriptionMessageHandler->__invoke()#16 /vagrant/vendor/symfony/messenger/Middleware/HandleMessageMiddleware.php(91): Symfony\Component\Messenger\Middleware\HandleMessageMiddleware->callHandler()#17 /vagrant/vendor/symfony/messenger/Middleware/SendMessageMiddleware.php(71): Symfony\Component\Messenger\Middleware\HandleMessageMiddleware->handle()#18 /vagrant/vendor/symfony/messenger/Middleware/FailedMessageProcessingMiddleware.php(34): Symfony\Component\Messenger\Middleware\SendMessageMiddleware->handle()#19 /vagrant/vendor/symfony/messenger/Middleware/DispatchAfterCurrentBusMiddleware.php(68): Symfony\Component\Messenger\Middleware\FailedMessageProcessingMiddleware->handle()#20 /vagrant/vendor/symfony/messenger/Middleware/RejectRedeliveredMessageMiddleware.php(41): Symfony\Component\Messenger\Middleware\DispatchAfterCurrentBusMiddleware->handle()#21 /vagrant/vendor/symfony/messenger/Middleware/AddBusNameStampMiddleware.php(37): Symfony\Component\Messenger\Middleware\RejectRedeliveredMessageMiddleware->handle()#22 /vagrant/vendor/symfony/messenger/Middleware/TraceableMiddleware.php(40): Symfony\Component\Messenger\Middleware\AddBusNameStampMiddleware->handle()#23 /vagrant/vendor/symfony/messenger/MessageBus.php(70): Symfony\Component\Messenger\Middleware\TraceableMiddleware->handle()#24 /vagrant/vendor/symfony/messenger/TraceableMessageBus.php(38): Symfony\Component\Messenger\MessageBus->dispatch()#25 /vagrant/vendor/symfony/messenger/RoutableMessageBus.php(54): Symfony\Component\Messenger\TraceableMessageBus->dispatch()#26 /vagrant/vendor/symfony/messenger/Worker.php(162): Symfony\Component\Messenger\RoutableMessageBus->dispatch()#27 /vagrant/vendor/symfony/messenger/Worker.php(109): Symfony\Component\Messenger\Worker->handleMessage()#28 /vagrant/vendor/symfony/messenger/Command/ConsumeMessagesCommand.php(238): Symfony\Component\Messenger\Worker->run()#29 /vagrant/vendor/symfony/console/Command/Command.php(326): Symfony\Component\Messenger\Command\ConsumeMessagesCommand->execute()#30 /vagrant/vendor/symfony/console/Application.php(1096): Symfony\Component\Console\Command\Command->run()#31 /vagrant/vendor/symfony/framework-bundle/Console/Application.php(126): Symfony\Component\Console\Application->doRunCommand()#32 /vagrant/vendor/symfony/console/Application.php(324): Symfony\Bundle\FrameworkBundle\Console\Application->doRunCommand()#33 /vagrant/vendor/symfony/framework-bundle/Console/Application.php(80): Symfony\Component\Console\Application->doRun()#34 /vagrant/vendor/symfony/console/Application.php(175): Symfony\Bundle\FrameworkBundle\Console\Application->doRun()#35 /vagrant/vendor/symfony/runtime/Runner/Symfony/ConsoleApplicationRunner.php(49): Symfony\Component\Console\Application->run()#36 /vagrant/vendor/autoload_runtime.php(29): Symfony\Component\Runtime\Runner\Symfony\ConsoleApplicationRunner->run()#37 /vagrant/bin/console(11): require_once('...')#38 {main}```TODO:- [x] Add test for retry logic when publishing messagesCommits-------f123370 [Messenger] [Amqp] Handle AMQPConnectionException when publishing a message.
xabbuh added a commit that referenced this pull requestAug 12, 2024
This PR was merged into the 5.4 branch.Discussion----------[Yaml] 🐛 throw ParseException on invalid date| Q | A| ------------- | ---| Branch? | 5.4 <!-- see below -->| Bug fix? | yes| New feature? | no <!-- please update src/**/CHANGELOG.md files -->| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->| Issues | None <!-- prefix each issue number with "Fix #", no need to create an issue if none exists, explain below instead -->| License | MIT(found insymfony-tools/docs-builder#179)When parsing the following yaml:```date: 6418-75-51````symfony/yaml` will throw an exception:```$ php main.phpPHP Fatal error: Uncaught Exception: Failed to parse time string (6418-75-51) at position 6 (5): Unexpected character in /tmp/symfony-yaml/vendor/symfony/yaml/Inline.php:714Stack trace:#0 /tmp/symfony-yaml/vendor/symfony/yaml/Inline.php(714): DateTimeImmutable->__construct()#1 /tmp/symfony-yaml/vendor/symfony/yaml/Inline.php(312): Symfony\Component\Yaml\Inline::evaluateScalar()#2 /tmp/symfony-yaml/vendor/symfony/yaml/Inline.php(80): Symfony\Component\Yaml\Inline::parseScalar()#3 /tmp/symfony-yaml/vendor/symfony/yaml/Parser.php(790): Symfony\Component\Yaml\Inline::parse()#4 /tmp/symfony-yaml/vendor/symfony/yaml/Parser.php(341): Symfony\Component\Yaml\Parser->parseValue()#5 /tmp/symfony-yaml/vendor/symfony/yaml/Parser.php(86): Symfony\Component\Yaml\Parser->doParse()#6 /tmp/symfony-yaml/vendor/symfony/yaml/Yaml.php(77): Symfony\Component\Yaml\Parser->parse()#7 /tmp/symfony-yaml/main.php(8): Symfony\Component\Yaml\Yaml::parse()#8 {main} thrown in /tmp/symfony-yaml/vendor/symfony/yaml/Inline.php on line 714```This is because the "month" is invalid. Fixing the "month" will trigger about the same issue because the "day" would be invalid.With the current change it will throw a `ParseException`.Commits-------6d71a7e 🐛 throw ParseException on invalid date
This pull request wasclosed.
Sign up for freeto 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.