e-moe / guzzle6-bundle Goto Github PK

First, thanks for your bundle.

I'm trying to use it with symfony 2.8-dev (not sure if this is the problem or not). Anyway I'm trying to override the monolog format in config.yml in a parameters: section, but it doesn't seem to work for some reason.

Do you have any clues as to what I might be doing wrong? It's not a typo, I've verified, however when things get compiled into the cache even, the default value and not my overridden value is used.

While i realise that this package is named "guzzle6-bundle", it would be nice to have support for Guzzle 7 if possible.

In Symfony 4 the stopwatch component is not included by default.
Requiring in a Symfony 4 application throws the following error:

The service "guzzle.client" has a dependency on a non-existent service "debug.stopwatch".

Fix:
Add the component as dependency.
composer req symfony/stopwatch

This bundle can not be installed on Symfony 5.0. Can requirements be changed so that it requires symfony/framework-bundle 5.0 and symfony/stopwatch 5.0?

see csarrazi/CsaGuzzleBundle@097cf06#diff-978c02ccc8ccdf14cb8a58dc91ab346c

With Symfony 4 it is possible to autowire services via type hinting.
Please add "GuzzleHttp\ClientInterface" as a service alias for "guzzle.client".
This will allow to autowire the default client without additional service configuration for the own application.

https://blackfire.io/docs/cookbooks/profiling-microservices
$client->getConfig('handler')->push(...)

Would there be a reason to require this component outside of dev environment?

Documentation says require and
new Emoe\GuzzleBundle\EmoeGuzzleBundle(),
implies that it should always be added to bundles array rather than only in test and dev.

When used with Symfony 3.4.X, the following deprecation notice is being returned:

User Deprecated: Implementing "Symfony\Component\HttpKernel\DataCollector\DataCollectorInterface" without the "reset()" method is deprecated since version 3.4 and will be unsupported in 4.0 for class "Emoe\GuzzleBundle\DataCollector\GuzzleDataCollector".

According to the Symfony changelog:

Implementing DataCollectorInterface without a reset() method has been deprecated and will be unsupported in 4.0.

Hello!

What do you think about dropping support of outdated php versions (5.6, 7.0 according to http://php.net/supported-versions.php) and add support for php 7.3?

Thank you

Apparently symfony has deprecated things like this:

    emoe_guzzle.stopwatch:
        class: %emoe_guzzle.stopwatch.class%
        ...

Instead you're supposed to now do it like this (with double or single quotes):

    emoe_guzzle.stopwatch:
        class: "%emoe_guzzle.stopwatch.class%"
        ...

Would you be willing to update your services.yml / monolog.yml files?

Hello.

One question... it's possible to disable self-signed certificates verification? I need to change the GuzzleClient config and set 'verify' option to false.

Like in this post... https://stackoverflow.com/questions/28066409/how-to-ignore-invalid-ssl-certificate-errors-in-guzzle-5

Thx.

If I set in my config.yml

emoe_guzzle:
    log:
        enabled: true

then I can't deserialize response body contents because is empty

$response = $this->guzzleClient->post($this->url, $body);

$object = $this->serializer->deserialize($response->getBody()->getContents());

It's possible to keep both behaviours?

Hi.

Is there any missing setting or config to enable debug profiler? I'm always getting an empty log. Even on right and wrong requests...

Thx.

When a Guzzle DELETE request is captured and displayed in the profiler, it is put in the "OTHER" tab, instead of "DELETE". See screenshot:

This is just a simple typo in the template file that I can submit a PR to address.

Package satooshi/php-coveralls is abandoned, you should avoid using it. Use php-coveralls/php-coveralls instead.

How do I tell the client to use Curl instead of the StreamHandler? I can't really find any config option for that.