stripe / stripe-ruby
96%
master: 97%

LAST BUILD BRANCH: HEAD
DEFAULT BRANCH: master
Repo Added
Files 262
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH brandur-clearable-connections
branch: brandur-clearable-connections
CHANGE BRANCH
x

3379

push

travis-ci

brandur 
Reset connections when connection-changing configuration changes

Adds a few basic features around connection and connection manager
management:

* `clear` on connection manager, which calls `finish` on each active
  connection and then disposes of it.

* A centralized cross-thread tracking system for connection managers in
  `StripeClient` and `clear_all_connection_managers` which clears all
  known connection managers across all threads in a thread-safe way.

The addition of these allow us to modify the implementation of some of
our configuration on `Stripe` so that it can reset all currently open
connections when its value changes.

This fixes a currently problem with the library whereby certain
configuration must be set before the first request or it remains fixed
on any open connections. For example, if `Stripe.proxy` is set after a
request is made from the library, it has no effect because the proxy
must have been set when the connection was originally being initialized.

The impetus for getting this out is that I noticed that we will need
this internally in a few places when we're upgrading to stripe-ruby V5.
Those spots used to be able to hack around the unavailability of this
feature by just accessing the Faraday connection directly and resetting
state on it, but in V5 `StripeClient#conn` is gone, and that's no longer
possible.

1838 of 1914 relevant lines covered (96.03%)

2450.87 hits per line

Relevant lines Covered
1914 RELEVANT LINES 1838 COVERED LINES
2450.87 HITS PER LINE
Source Files on brandur-clearable-connections
Coverage ∆ File Lines Relevant Covered Missed Hits/Line

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
3379 brandur-clearable-connections Reset connections when connection-changing configuration changes Adds a few basic features around connection and connection manager management: * `clear` on connection manager, which calls `finish` on each active connection and then disposes o... push brandur travis-ci pending completion  
3378 brandur-clearable-connections Reset connections when connection-changing configuration changes Adds a few basic features around connection and connection manager management: * `clear` on connection manager, which calls `finish` on each active connection and then disposes o... Pull #829 web-flow travis-ci pending completion  
3367 brandur-clearable-connections Reset connections when connection-changing configuration changes Adds a few basic features around connection and connection manager management: * `clear` on connection manager, which calls `finish` on each active connection and then disposes o... Pull #829 web-flow travis-ci pending completion  
3366 brandur-clearable-connections Reset connections when connection-changing configuration changes Adds a few basic features around connection and connection manager management: * `clear` on connection manager, which calls `finish` on each active connection and then disposes o... push brandur travis-ci pending completion  
3365 brandur-clearable-connections Reset connections when connection-changing configuration changes Adds a few basic features around connection and connection manager management: * `clear` on connection manager, which calls `finish` on each active connection and then disposes o... Pull #829 web-flow travis-ci pending completion  
3364 brandur-clearable-connections Reset connections when connection-changing configuration changes Adds a few basic features around connection and connection manager management: * `clear` on connection manager, which calls `finish` on each active connection and then disposes o... push brandur travis-ci pending completion  
See All Builds (4151)