cmd/tier: accept a pricing.json from a URL

cmd/tier/help.go

@@ -59,10 +59,13 @@ Print the version of the Tier CLI.
 
 	"push": `Usage:
 
-	tier [--live] push <filename | - >
+	tier [--live] push <filename | url | - >
 
-Tier push pushes the pricing JSON in the provided filename to Stripe. If the
-filename is ("-") then stdin is read.
+"tier push" pushes pricing JSON to Stripe. The data may come from a file, url,
+or stdin. If a URL is specified, push will use the response body from a GET
+request as pricing JSON, if the status code is of the 2XX variety and the body
+is valid pricing JSON. If the filename is ("-") then the pricing JSON is read
+from stdin.
 
 To learn more about how this works, please visit: https://tier.run/docs/cli/push
 

cmd/tier/tier.go

@@ -490,6 +490,21 @@ func fileOrStdin(fname string) (io.ReadCloser, error) {
 	if fname == "-" {
 		return io.NopCloser(stdin), nil
 	}
+	u, err := url.Parse(fname)
+	if err == nil {
+		// if no http then bail
+		if u.Scheme != "http" && u.Scheme != "https" {
+			return nil, fmt.Errorf("URL scheme must be http or https. Got %q", u.Scheme)
+		}
+		res, err := http.Get(fname)
+		if err != nil {
+			return nil, err
+		}
+		if res.StatusCode/100 != 2 {
+			return nil, fmt.Errorf("http error fetching pricing.json: %s", res.Status)
+		}
+		return res.Body, nil
+	}
 	return os.Open(fname)
 }
 

cmd/tier/tier_test.go

@@ -206,7 +206,14 @@ func TestSwitchPreallocateTask(t *testing.T) {
 }
 
 func TestPushStdin(t *testing.T) {
-	s := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, _ *http.Request) {
+	s := httptest.NewServer(http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
+		if r.URL.Path == "/pricing.json" {
+			// be remote pricing.json
+			io.WriteString(w, `{}`)
+			return
+		}
+
+		// Be Stripe
 		io.WriteString(w, `{"id": "price_123"}`)
 	}))
 
@@ -226,6 +233,8 @@ func TestPushStdin(t *testing.T) {
 		{"{", "-", "unexpected EOF", false},
 
 		{"{}", "-", "", true},
+
+		{"", s.URL + "/pricing.json", "^$", true},
 	}
 
 	for _, c := range cases {