Merge branch 'request-id' into request-id

Author: JadeRedworth

common/common.go

@@ -11,8 +11,10 @@ import (
 	"log"
 	"os"
 	"os/exec"
+	"encoding/base32"
 	"os/signal"
 	"path/filepath"
+	"crypto/rand"
 	"strings"
 	"time"
 	"unicode"
@@ -32,6 +34,8 @@ const (
 	FunctionsDockerImage     = "fnproject/fnserver"
 	FuncfileDockerRuntime    = "docker"
 	MinRequiredDockerVersion = "17.5.0"
+	RequestID = "request-id"
+	RequestedLength = 16
 )
 
 // GetWd returns working directory.
@@ -55,6 +59,16 @@ func GetDir(c *cli.Context) string {
 	return dir
 }
 
+func GetRequestID() string {
+	byteArr := make([]byte, RequestedLength)
+	_, err := rand.Read(byteArr)
+	if err != nil {
+		log.Fatalf("failed to generate random number for requestID")
+	}
+
+	return base32.StdEncoding.WithPadding(base32.NoPadding).EncodeToString(byteArr)
+}
+
 // BuildFunc bumps version and builds function.
 func BuildFunc(c *cli.Context, fpath string, funcfile *FuncFile, buildArg []string, noCache bool) (*FuncFile, error) {
 	var err error

main.go

@@ -14,6 +14,7 @@ import (
 	"text/template"
 
 	"github.com/fnproject/cli/commands"
+	"github.com/fnproject/cli/common"
 	"github.com/fnproject/cli/common/color"
 	"github.com/fnproject/cli/config"
 	"github.com/spf13/viper"
@@ -42,8 +43,7 @@ func newFn() *cli.App {
 		if err != nil {
 			return err
 		}
-
-		viper.Set("request-id", getRequestID())
+		viper.Set(common.RequestID, common.GetRequestID())
 		commandArgOverrides(c)
 		return nil
 	}