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.

Sign up for GitHub

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

Jump to bottom

Add connect_timeout for guzzle #11

Merged
merged 1 commit into from
Jul 7, 2018
Merged

Add connect_timeout for guzzle #11

merged 1 commit into from
Jul 7, 2018

Conversation

brycefranzen
Copy link

Without this functionality, Guzzle causes server timeout when remote cannot be found. This is essential!

@sergeyklay sergeyklay merged commit 7781e3d into pdffiller:master Jul 7, 2018
@sergeyklay
Copy link
Member

Thank you

@sergeyklay
Copy link
Member

@brycefranzen Just release a new version https://github.com/pdffiller/laravel-influx-provider/releases/tag/v1.6.1

@brycefranzen
Copy link
Author

brycefranzen commented Jul 7, 2018 via email

@brycefranzen
Copy link
Author

brycefranzen commented Jul 9, 2018 via email

@brycefranzen
Copy link
Author

brycefranzen commented Jul 9, 2018 •
edited
Loading

Can you please push a release to composer as well? Composer has no reference to 1.6.1 version at this point :/ Thanks!!! @sergeyklay

@sergeyklay
Copy link
Member

sergeyklay commented Jul 9, 2018 •
edited
Loading

@brycefranzen Since Github services are already deprecated, I would like to not register the Packagist integration as a new Github service. FYI the Packagist also plan to build new way to integrate. See composer/packagist#907 which contains the schedule about when it will stop working.

You still can receive any package version using repositories option:

$ cat composer.json

{
    "repositories": [
        {
            "type": "vcs",
            "url": "[email protected]:pdffiller/laravel-influx-provider.git"
        }
    ]
}
$ composer require pdffiller/laravel-influx-provider:1.6.1

./composer.json has been updated                                   
Loading composer repositories with package information
Updating dependencies (including require-dev)                      
Package operations: 13 installs, 0 updates, 0 removals
  - Installing psr/log (1.0.2): Loading from cache
  - Installing monolog/monolog (1.23.0): Loading from cache
  - Installing guzzlehttp/promises (v1.3.1): Loading from cache
  - Installing psr/http-message (1.0.1): Loading from cache
  - Installing guzzlehttp/psr7 (1.4.2): Loading from cache
  - Installing guzzlehttp/guzzle (6.3.3): Loading from cache
  - Installing influxdb/influxdb-php (1.14.7): Downloading (100%)         
  - Installing paragonie/random_compat (v2.0.17): Downloading (100%)         
  - Installing illuminate/contracts (v5.4.36): Loading from cache
  - Installing doctrine/inflector (v1.3.0): Loading from cache
  - Installing illuminate/support (v5.4.36): Loading from cache
  - Installing illuminate/log (v5.4.36): Downloading (100%)         
  - Installing pdffiller/laravel-influx-provider (v1.6.1): Downloading (100%)         
monolog/monolog suggests installing aws/aws-sdk-php (Allow sending log messages to AWS services like DynamoDB)
monolog/monolog suggests installing doctrine/couchdb (Allow sending log messages to a CouchDB server)
monolog/monolog suggests installing ext-amqp (Allow sending log messages to an AMQP server (1.0+ required))
monolog/monolog suggests installing ext-mongo (Allow sending log messages to a MongoDB server)
monolog/monolog suggests installing graylog2/gelf-php (Allow sending log messages to a GrayLog2 server)
monolog/monolog suggests installing mongodb/mongodb (Allow sending log messages to a MongoDB server via PHP Driver)
monolog/monolog suggests installing php-amqplib/php-amqplib (Allow sending log messages to an AMQP server using php-amqplib)
monolog/monolog suggests installing php-console/php-console (Allow sending log messages to Google Chrome)
monolog/monolog suggests installing rollbar/rollbar (Allow sending log messages to Rollbar)
monolog/monolog suggests installing ruflin/elastica (Allow sending log messages to an Elastic Search server)
monolog/monolog suggests installing sentry/sentry (Allow sending log messages to a Sentry server)
influxdb/influxdb-php suggests installing stefanotorresi/influxdb-php-async (An asyncronous client for InfluxDB, implemented via ReactPHP.)
paragonie/random_compat suggests installing ext-libsodium (Provides a modern crypto API that can be used to generate random bytes.)
illuminate/support suggests installing illuminate/filesystem (Required to use the composer class (5.2.*).)
illuminate/support suggests installing symfony/process (Required to use the composer class (~3.2).)
illuminate/support suggests installing symfony/var-dumper (Required to use the dd function (~3.2).)
Writing lock file
Generating autoload files

/cc @apictapx

@brycefranzen
Copy link
Author

brycefranzen commented Jul 9, 2018 via email

@sergeyklay
Copy link
Member

@brycefranzen Just configured the GitHub Packagist application. So you can safely remove the repositories section.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sergeyklay sergeyklay sergeyklay approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@brycefranzen @sergeyklay