stripe / stripe-go / 3686
86%
master: 66%

LAST BUILD BRANCH: latest-codegen-beta
DEFAULT BRANCH: master
Ran
Jobs 4
Files 114
Run time 2min
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
coverage: 85.998%. First build
3686

push

travis-ci

brandur 
Put `/v1/` prefix as part of all paths instead of URL

Traditionally, stripe-go has done something a little weird in that it
considered the `/v1/` in API URLs as part of the URL (e.g., `APIURL` or
`UploadURL`) instead of part of the paths of individual API calls. So
when calling `POST /v1/charges` we'd call `POST /charges` and rely on
the fact that `/v1/` was expected to be present in the URL.

This has always rubbed me the wrong way a little bit, and caused a
little bit of trouble in #733 as we'd been doing something a little
hacky when initializing new configs for backends.

This has some potential backwards compatibility problems in that people
may have been injecting custom URLs (e.g., if they had a testing setup a
little like we do for stripe-mock) that included the `/v1/` prefix. I've
tried to address this by trimming a `/v1/` prefix when creating a
backend with a configured URL. For example:

``` go
switch backendType {
case APIBackend:
        if config.URL == "" {
                config.URL = apiURL
        }

        // For a long time we had the `/v1` prefix as part of a configured URL
        // rather than in the per-package URLs throughout the library. Continue
        // to support this for the time being by stripping one tha's been
        // passed for better backwards compatibility.
        config.URL = strings.TrimPrefix(config.URL, "/v1")
```

I believe that will address the problem.

There's some risk that I missed a URL string in the project because
there are a lot of them. I tried to minimize the risk there by using a
tool-assisted project-wide find + replace, then manually examining every
string in the project that looks like a URL path (e.g., starts with
`"/`).

4287 of 4985 relevant lines covered (86.0%)

74.19 hits per line

Jobs
ID Job ID Ran Files Coverage
1 3686.1 114
86.0
 Travis Job 3686.1
2 3686.2 114
86.0
 Travis Job 3686.2
3 3686.3 114
86.0
 Travis Job 3686.3
4 3686.4 114
86.0
 Travis Job 3686.4
Source Files on build 3686