vain/server.go

package vain

import (
	"encoding/json"
	"fmt"
	"net/http"
	"strings"
)

// NewServer populates a server, adds the routes, and returns it for use.
func NewServer(sm *http.ServeMux, store Storage) *Server {
	s := &Server{
		storage: store,
	}
	sm.Handle("/", s)
	return s
}

// Server serves up the http.
type Server struct {
	storage Storage
}

func (s *Server) ServeHTTP(w http.ResponseWriter, req *http.Request) {
	switch req.Method {
	case "GET":
		fmt.Fprintf(w, "<!DOCTYPE html>\n<html><head>\n")
		for _, p := range s.storage.All() {
			fmt.Fprintf(w, "%s\n", p)
		}
		fmt.Fprintf(w, "</head>\n</html>\n")
	case "POST":
		if req.URL.Path == "/" {
			http.Error(w, fmt.Sprintf("invalid path %q", req.URL.Path), http.StatusBadRequest)
			return
		}
		p := Package{}
		if err := json.NewDecoder(req.Body).Decode(&p); err != nil {
			http.Error(w, fmt.Sprintf("unable to parse json from body: %v", err), http.StatusBadRequest)
			return
		}
		if p.Repo == "" {
			http.Error(w, fmt.Sprintf("invalid repository %q", req.URL.Path), http.StatusBadRequest)
			return
		}
		p.path = fmt.Sprintf("%s/%s", req.Host, strings.Trim(req.URL.Path, "/"))
		if !Valid(p.path, s.storage.All()) {
			http.Error(w, fmt.Sprintf("invalid path; prefix already taken %q", req.URL.Path), http.StatusConflict)
			return
		}
		if err := s.storage.Add(p); err != nil {
			http.Error(w, fmt.Sprintf("unable to add package: %v", err), http.StatusInternalServerError)
			return
		}
	default:
		http.Error(w, fmt.Sprintf("unsupported method %q; accepted: POST, GET", req.Method), http.StatusMethodNotAllowed)
	}
}
moved ysv -> vain 2016-02-11 11:57:16 -08:00			`package vain`
Added an http server 2016-02-08 00:15:22 -08:00
stubbed out supported methods on / 2016-02-11 12:12:13 -08:00			`import (`
simple in-memory storage working for POST/GET 2016-02-13 01:18:06 -08:00			`"encoding/json"`
stubbed out supported methods on / 2016-02-11 12:12:13 -08:00			`"fmt"`
			`"net/http"`
simple in-memory storage working for POST/GET 2016-02-13 01:18:06 -08:00			`"strings"`
stubbed out supported methods on / 2016-02-11 12:12:13 -08:00			`)`
Added an http server 2016-02-08 00:15:22 -08:00
Addressing go vet / golint 2016-02-15 01:10:14 -08:00			`// NewServer populates a server, adds the routes, and returns it for use.`
Don't require user to set YSV_HOST The original reason for having this required configuration parameter was that I didn't think to read the hostname off the request. Doing it this way has the additional benefit of a single server being able to serve a multitude of hostnames. Derek helped me realize this by repeating "there should be sane defaults" like 9000 times (maybe over 9000) till I got so tired of trying to make sure that he understood why I did it that way to begin with that I looked at the problem from a different perspective and like in a dream the solution came to me. fixes #10. 2016-02-16 23:18:54 -08:00			`func NewServer(sm http.ServeMux, store Storage) Server {`
Addressing go vet / golint 2016-02-15 01:10:14 -08:00			`s := &Server{`
Don't require user to set YSV_HOST The original reason for having this required configuration parameter was that I didn't think to read the hostname off the request. Doing it this way has the additional benefit of a single server being able to serve a multitude of hostnames. Derek helped me realize this by repeating "there should be sane defaults" like 9000 times (maybe over 9000) till I got so tired of trying to make sure that he understood why I did it that way to begin with that I looked at the problem from a different perspective and like in a dream the solution came to me. fixes #10. 2016-02-16 23:18:54 -08:00			`storage: store,`
Addressing go vet / golint 2016-02-15 01:10:14 -08:00			`}`
			`sm.Handle("/", s)`
			`return s`
			`}`

			`// Server serves up the http.`
Added an http server 2016-02-08 00:15:22 -08:00			`type Server struct {`
Don't require user to set YSV_HOST The original reason for having this required configuration parameter was that I didn't think to read the hostname off the request. Doing it this way has the additional benefit of a single server being able to serve a multitude of hostnames. Derek helped me realize this by repeating "there should be sane defaults" like 9000 times (maybe over 9000) till I got so tired of trying to make sure that he understood why I did it that way to begin with that I looked at the problem from a different perspective and like in a dream the solution came to me. fixes #10. 2016-02-16 23:18:54 -08:00			`storage Storage`
Added an http server 2016-02-08 00:15:22 -08:00			`}`

stubbed out supported methods on / 2016-02-11 12:12:13 -08:00			`func (s Server) ServeHTTP(w http.ResponseWriter, req http.Request) {`
			`switch req.Method {`
			`case "GET":`
simple in-memory storage working for POST/GET 2016-02-13 01:18:06 -08:00			`fmt.Fprintf(w, "<!DOCTYPE html>\n<html><head>\n")`
			`for _, p := range s.storage.All() {`
			`fmt.Fprintf(w, "%s\n", p)`
			`}`
			`fmt.Fprintf(w, "</head>\n</html>\n")`
stubbed out supported methods on / 2016-02-11 12:12:13 -08:00			`case "POST":`
simple in-memory storage working for POST/GET 2016-02-13 01:18:06 -08:00			`if req.URL.Path == "/" {`
			`http.Error(w, fmt.Sprintf("invalid path %q", req.URL.Path), http.StatusBadRequest)`
			`return`
			`}`
			`p := Package{}`
			`if err := json.NewDecoder(req.Body).Decode(&p); err != nil {`
Addressing go vet / golint 2016-02-15 01:10:14 -08:00			`http.Error(w, fmt.Sprintf("unable to parse json from body: %v", err), http.StatusBadRequest)`
			`return`
			`}`
			`if p.Repo == "" {`
			`http.Error(w, fmt.Sprintf("invalid repository %q", req.URL.Path), http.StatusBadRequest)`
simple in-memory storage working for POST/GET 2016-02-13 01:18:06 -08:00			`return`
			`}`
Don't require user to set YSV_HOST The original reason for having this required configuration parameter was that I didn't think to read the hostname off the request. Doing it this way has the additional benefit of a single server being able to serve a multitude of hostnames. Derek helped me realize this by repeating "there should be sane defaults" like 9000 times (maybe over 9000) till I got so tired of trying to make sure that he understood why I did it that way to begin with that I looked at the problem from a different perspective and like in a dream the solution came to me. fixes #10. 2016-02-16 23:18:54 -08:00			`p.path = fmt.Sprintf("%s/%s", req.Host, strings.Trim(req.URL.Path, "/"))`
Addressing go vet / golint 2016-02-15 01:10:14 -08:00			`if !Valid(p.path, s.storage.All()) {`
Prevent addition of duplicate path prefixes. I think I understand this now; the failure comes from here: https://golang.org/src/cmd/go/vcs.go#L818 and makes it so one can't both host: - github.com/foo - github.com/foo/bar which would have to clone the latter inside of the former. Fixes #1. 2016-02-13 10:44:41 -08:00			`http.Error(w, fmt.Sprintf("invalid path; prefix already taken %q", req.URL.Path), http.StatusConflict)`
			`return`
			`}`
Added Save/Load to db Fixes #3. Fixes #2 2016-02-14 22:19:41 -08:00			`if err := s.storage.Add(p); err != nil {`
			`http.Error(w, fmt.Sprintf("unable to add package: %v", err), http.StatusInternalServerError)`
			`return`
			`}`
stubbed out supported methods on / 2016-02-11 12:12:13 -08:00			`default:`
updated docs to match reality. 2016-02-14 21:46:12 -08:00			`http.Error(w, fmt.Sprintf("unsupported method %q; accepted: POST, GET", req.Method), http.StatusMethodNotAllowed)`
stubbed out supported methods on / 2016-02-11 12:12:13 -08:00			`}`
			`}`