Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configure HTTP server options via parameters #23

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

youwe-petervanderwal
Copy link
Contributor

Let the application configure the HTTP server options (for example the bodySizeLimit) via the parameters section in esb.yml.

Background: I ran into the problem that my ESB application needed to accept payloads larger than the default 128KB

Let the application configure the HTTP server options (for example the bodySizeLimit) via the parameters section in esb.yml.
The Combination PHP <7.4 and maglnet/composer-require-checker doesn't work with Composer 2. User Composer version 1 for PHP 7.2 and 7.3.
Newer versions causing errors:

ECS >= 8.3: PHP Fatal error:  Uncaught OutOfBoundsException: Package "symplify/easy-coding-standard" is not installed in phar:///home/youwe/Projects/webgriffe-esb/vendor/phpstan/phpstan/phpstan.phar/vendor/composer/InstalledVersions.php:1081

PhpStan >= 0.12.67:  Ignored error pattern /Unable to resolve the template type .*? in call to function Amp\\call/ was not matched in reported errors. Ignored error pattern /Unable to resolve the template type .*? in instantiation of class Amp\\Producer/ was not matched in reported errors.
PhpStan: Method Webgriffe\Esb\Exception\ElasticSearch\JobNotFoundException::__construct() has parameter $code with no typehint specified.
@@ -56,8 +56,8 @@
"pda/pheanstalk": "^3.1",
"mikey179/vfsstream": "^1.6",
"amphp/artax": "^3.0",
"phpstan/phpstan": "^0.12",
"symplify/easy-coding-standard-prefixed": "^8.1"
"phpstan/phpstan": "^0.12 <=0.12.66",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not really happy with this change.... But with the higher versions I ran into problems with phpstan and ecs which weren't straight forward to fix (PhpStan complaining about amphp code, ecs refusing to run at all)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant