Improve stacktrace to fix tech debts

Signed-off-by: divyansh42 <diagrawa@redhat.com>

.gitignore

@@ -2,7 +2,6 @@
 /.atom-build.yml
 /.project
 /.vagrant
-.vscode/
 cmd/s2i/debug
 *~
 

pkg/docker/docker.go

@@ -2,6 +2,7 @@ package docker
 
 import (
 	"archive/tar"
+	"bufio"
 	"bytes"
 	"encoding/base64"
 	"encoding/json"
@@ -995,7 +996,14 @@ func (d *stiDocker) RunContainer(opts RunContainerOptions) error {
 		if signal == syscall.SIGQUIT {
 			buf := make([]byte, 1<<16)
 			runtime.Stack(buf, true)
-			fmt.Printf("%s", buf)
+			f, err := os.Create("/var/log/s2i_docker_stack_trace.log")
+			if err != nil {
+				return
+			}
+			defer f.Close()
+			w := bufio.NewWriter(f)
+			w.Write(buf)
+			w.Flush()
 		}
 		os.Exit(2)
 	}

pkg/scripts/download.go

@@ -139,7 +139,7 @@ type FileURLReader struct{}
 func (*FileURLReader) Read(url *url.URL) (io.ReadCloser, error) {
 	// for some reason url.Host may contain information about the ./ or ../ when
 	// specifying relative path, thus using that value as well
-	return os.Open(filepath.Join(url.Host, url.Path))
+	return os.Open(filepath.Clean(filepath.Join(url.Host, url.Path)))
 }
 
 // ImageReader just returns information the URL is from inside the image