GrahamCampbell
Repos
57
Followers
6114
Following
130

Laravel is a web application framework with expressive, elegant syntax. We’ve already laid the foundation for your next big idea — freeing you to create without sweating the small things.

71072
22190

Guzzle, an extensible PHP HTTP client

22104
2281

📛 An open source status page system for everyone.

12897
1453

Loads environment variables from `.env` to `getenv()`, `$_ENV` and `$_SERVER` automagically.

12153
572

A CommonMark wrapper for Laravel

1211
134

A system for auto-decorating models with presenters

754
63

Events

issue comment
let's update the phpoption package

We don't need those annotations because we actually have the real return types.

Created at 4 days ago
issue comment
let's update the phpoption package

This is already solved in the latest release.

Created at 4 days ago
closed issue
let's update the phpoption package

Let's update the phpoption package to solve the #[\ReturnTypeWillChange] temporary suppressor for php 8.1 version

Created at 4 days ago
closed issue
Save to .env

There is a way to edit the variable in .env file?

Created at 6 days ago
issue comment
Save to .env

No, that is not something we have implemented here.

Created at 6 days ago
issue comment
GitHub Workflows security hardening

Can you show an example how this can be exploited? PR builds surely only push to the fork?

Created at 1 week ago

[1.7] Fix case where the timeout in setHtml() isn't respected (#433)

Created at 1 week ago
pull request closed
[1.7] Fix case where the timeout in `setHtml()` isn't respected

The previous implementation ignores the timeout when sending the html to the browser. The timeout was applied only when waiting for a response. Now it will be respected as the maximum time for the whole operation.

Fixes #432

Created at 1 week ago

[1.8] Add helper function to find target (#436)

Co-authored-by: enricodias enricodias@gmail.com Co-authored-by: Graham Campbell GrahamCampbell@users.noreply.github.com

Created at 1 week ago
pull request closed
[1.8] Add helper function to find target

A convenience function that makes it easier to find targets by title.

Created at 1 week ago

Fix command execution in AutoDiscover (#424)

Co-authored-by: Graham Campbell GrahamCampbell@users.noreply.github.com

Upgrade phpstan

Release 1.7.1

Add windows compatibility for unit tests (#435)

Created at 1 week ago
started
Created at 1 week ago
started
Created at 1 week ago
issue comment
when use retry middleware, if the http status code is 4xx or 5xx, it callable fulfilled, not rejected

not stale

Created at 2 weeks ago
closed issue
Guzzle 6.5.5 Providing RequestException instead of ConnectException

Guzzle version(s) affected: 6.5.5 PHP version: 7.2.34 cURL version: 7.29.0 (hint: php -i | grep cURL)

Description

cURL error 0: The cURL request was retried 3 times
and did not succeed. The most likely reason for the failure
is that cURL was unable to rewind the body of the request
and subsequent retries resulted in the same error. Turn on
the debug option to see what went wrong. See
https://bugs.php.net/bug.php?id=47204 for more information.
(see https://curl.haxx.se/libcurl/c/libcurl-errors.html)

This error is delivered as a RequestException, but should it not be delivered as a ConnectException ? I mean, its a 0 http_code.

I'm getting an error from guzzle based on that _curl_retries logic in CurlFactory.

How to reproduce

will find some time

Possible Solution

none yet

Additional context

happens when making requests to headless chrome by localhost:port

Created at 2 weeks ago
issue comment
[9.x] Prevent workflows from running twice

Opening a pull request.

Created at 2 weeks ago
issue comment
[9.x] Prevent workflows from running twice

I think that's actually desirable that they run twice, allowing testing before cutting a PR.

Created at 2 weeks ago
issue comment
Since v1.5.1 I'm getting "This test did not perform any assertions"

I think that is debatable. That should probably be regarded as a breaking change, and delayed till 2.x, and reverted in 1.x.

Created at 2 weeks ago
issue comment
Relative path with authority part present fails

Oh, I see the last part "Paths not starting with a slash ("/") are assumed to be relative to some base path known to the application or consumer.". I'd be happy to review a PR if you'd like to work on this.

Created at 2 weeks ago
reopened issue
Relative path with authority part present fails

Why relative paths forbidden when authority part is present?

https://github.com/guzzle/psr7/blob/69568e4293f4fa993f3b0e51c9723e1e17c41379/tests/UriTest.php#L599-L605

This violates withPath contract. Also there's clear description how to avoid problem described in the aforementioned code comment.

Created at 2 weeks ago
closed issue
Relative path with authority part present fails

Why relative paths forbidden when authority part is present?

https://github.com/guzzle/psr7/blob/69568e4293f4fa993f3b0e51c9723e1e17c41379/tests/UriTest.php#L599-L605

This violates withPath contract. Also there's clear description how to avoid problem described in the aforementioned code comment.

Created at 2 weeks ago