Working dockerfile

README.md

@@ -14,6 +14,3 @@ Currently stores no state and has no user authentication, so using it should jus
   POST /game/{id}/move
   GET /game/{id}/move/{movenum}
 ```
-
-## Authorization headers
-  Requests should contain header "Player-id"="red|blue"

api.go

@@ -49,7 +49,7 @@ func (a *API) NewGame(res http.ResponseWriter, req *http.Request) {
 		}
 	}
 	log.Printf("creating new game %v", a.nextInt)
-	a.games[a.nextInt] = NewSession(8)
+	a.games[a.nextInt] = NewSession()
 	a.games[a.nextInt].redPlayer.Ready = true
 	respondWithJSON(res, http.StatusOK, newGameResp{a.nextInt, "red"})
 	a.nextInt = a.nextInt + 1
@@ -63,7 +63,18 @@ func (a *API) GetGame(res http.ResponseWriter, req *http.Request) {
 		respondWithError(res, http.StatusBadRequest, "Invalid game ID")
 		return
 	}
-	pid := req.Header.Get("Player-id")
+	var gr gameReq
+	decoder := json.NewDecoder(req.Body)
+	if err := decoder.Decode(&gr); err != nil {
+		log.Println(err)
+		respondWithError(res, http.StatusBadRequest, "Invalid request payload")
+		return
+	}
+	defer req.Body.Close()
+	if gr.PlayerID != "red" && gr.PlayerID != "blue" {
+		respondWithError(res, http.StatusBadRequest, "Bad player ID")
+		return
+	}
 	var p *Player
 
 	s, isset := a.games[id]
@@ -71,26 +82,14 @@ func (a *API) GetGame(res http.ResponseWriter, req *http.Request) {
 		respondWithError(res, http.StatusBadRequest, "No such game")
 		return
 	}
-	if pid == "red" {
+	if gr.PlayerID == "red" {
 		p = s.redPlayer
-	} else if pid == "blue" {
-		p = s.bluePlayer
 	} else {
-		respondWithError(res, http.StatusBadRequest, "Bad player ID")
-		return
+		p = s.bluePlayer
 	}
 
 	log.Println("sending game state")
-	board := s.getBoard(p)
-	rotate := req.Header.Get("Rotate")
-	if rotate == "true" {
-		log.Println("rotating output")
-		//rotateBoard(board)
-		//rotateBoard(board)
-		//rotateBoard(board)
-
-	}
-	respondWithJSON(res, http.StatusOK, gameResp{board})
+	respondWithJSON(res, http.StatusOK, gameResp{s.getBoard(p)})
 	return
 }
 
@@ -102,22 +101,23 @@ func (a *API) GetGameStatus(res http.ResponseWriter, req *http.Request) {
 		respondWithError(res, http.StatusBadRequest, "Invalid game ID")
 		return
 	}
+	var gr gameReq
+	decoder := json.NewDecoder(req.Body)
+	if err := decoder.Decode(&gr); err != nil {
+		respondWithError(res, http.StatusBadRequest, "Invalid resquest payload")
+		return
+	}
+	defer req.Body.Close()
+	if gr.PlayerID != "red" && gr.PlayerID != "blue" {
+		respondWithError(res, http.StatusBadRequest, "Bad player ID")
+		return
+	}
 	s, isset := a.games[id]
 	if !isset {
 		respondWithError(res, http.StatusBadRequest, "No such game")
 		return
 	}
-	pid := req.Header.Get("Player-id")
-	var p *Player
-	if pid == "red" {
-		p = s.redPlayer
-	} else if pid == "blue" {
-		p = s.bluePlayer
-	} else {
-		respondWithError(res, http.StatusBadRequest, "Bad player ID")
-		return
-	}
-	log.Printf("sending game status to player %v", p.Team)
+	log.Println("sending game status")
 	respondWithJSON(res, http.StatusOK, gameStatusResp{s.simulator.State, s.moveNum})
 }
 
@@ -132,11 +132,14 @@ func (a *API) PostMove(res http.ResponseWriter, req *http.Request) {
 	var gr gameMovePostReq
 	decoder := json.NewDecoder(req.Body)
 	if err := decoder.Decode(&gr); err != nil {
-		respondWithError(res, http.StatusBadRequest, "Invalid request payload")
+		respondWithError(res, http.StatusBadRequest, "Invalid resquest payload")
 		return
 	}
 	defer req.Body.Close()
-	pid := req.Header.Get("Player-id")
+	if gr.PlayerID != "red" && gr.PlayerID != "blue" {
+		respondWithError(res, http.StatusBadRequest, "Bad player ID")
+		return
+	}
 	var p *Player
 
 	s, isset := a.games[id]
@@ -144,13 +147,10 @@ func (a *API) PostMove(res http.ResponseWriter, req *http.Request) {
 		respondWithError(res, http.StatusBadRequest, "No such game")
 		return
 	}
-	if pid == "red" {
+	if gr.PlayerID == "red" {
 		p = s.redPlayer
-	} else if pid == "blue" {
-		p = s.bluePlayer
 	} else {
-		respondWithError(res, http.StatusBadRequest, "Bad player ID")
-		return
+		p = s.bluePlayer
 	}
 	parsed, err := s.tryMove(p, gr.Move)
 	if err != nil {
@@ -180,6 +180,17 @@ func (a *API) GetMove(res http.ResponseWriter, req *http.Request) {
 		respondWithError(res, http.StatusBadRequest, "Invalid move number")
 		return
 	}
+	var gr gameMoveReq
+	decoder := json.NewDecoder(req.Body)
+	if err := decoder.Decode(&gr); err != nil {
+		respondWithError(res, http.StatusBadRequest, "Invalid resquest payload")
+		return
+	}
+	defer req.Body.Close()
+	if gr.PlayerID != "red" && gr.PlayerID != "blue" {
+		respondWithError(res, http.StatusBadRequest, "Bad player ID")
+		return
+	}
 	var p *Player
 
 	s, isset := a.games[id]
@@ -187,14 +198,10 @@ func (a *API) GetMove(res http.ResponseWriter, req *http.Request) {
 		respondWithError(res, http.StatusBadRequest, "No such game")
 		return
 	}
-	pid := req.Header.Get("Player-id")
-	if pid == "red" {
+	if gr.PlayerID == "red" {
 		p = s.redPlayer
-	} else if pid == "blue" {
-		p = s.bluePlayer
 	} else {
-		respondWithError(res, http.StatusBadRequest, "Bad player ID")
-		return
+		p = s.bluePlayer
 	}
 	moveRes, err := s.getMove(p, move)
 	if err != nil {

api_test.go

@@ -1,122 +0,0 @@
-package main
-
-import (
-	"encoding/json"
-	"fmt"
-	"net/http"
-	"net/http/httptest"
-	"strconv"
-	"testing"
-
-	"github.com/gorilla/mux"
-)
-
-func TestNewAPI(t *testing.T) {
-	a := NewAPI()
-	if len(a.games) != 0 {
-		t.Fatalf("games list not empty")
-	}
-	if a.nextInt != 1 {
-		t.Fatalf("nextInt somehow already implemented")
-	}
-}
-
-func dummyGame(a *API) int {
-	i := a.nextInt
-	a.games[i] = NewSession(8)
-	a.games[i].redPlayer.Ready = true
-	a.games[i].bluePlayer.Ready = true
-	a.games[i].simulator.Setup()
-	initDummy(a.games[i].simulator)
-	return i
-}
-
-func TestNewGame(t *testing.T) {
-	t.Parallel()
-	a := NewAPI()
-	var tests = []struct {
-		gid int
-		pid string
-	}{
-		{1, "red"},
-		{1, "blue"},
-		{2, "red"},
-	}
-	for i, tt := range tests {
-		tname := fmt.Sprintf("/game %v", i)
-		t.Run(tname, func(t *testing.T) {
-			r, _ := http.NewRequest("POST", "/game", nil)
-			w := httptest.NewRecorder()
-			a.NewGame(w, r)
-			resp := w.Result()
-			if resp.StatusCode != http.StatusOK {
-				t.Fatal("failed to create new game")
-			}
-			_, ok := a.games[tt.gid]
-			if !ok {
-				t.Fatalf("API thinks it created a game but it didn't")
-			}
-			var respStruct newGameResp
-			err := json.NewDecoder(resp.Body).Decode(&respStruct)
-			if err != nil {
-				t.Errorf("/game returned bad response body: %v", err)
-			}
-			if respStruct.GameID != tt.gid {
-				t.Errorf("Expected game %v, got %v", tt.gid, respStruct.GameID)
-			}
-			if respStruct.PlayerID != tt.pid {
-				t.Errorf("wrong playerID returned")
-			}
-		})
-	}
-}
-
-func TestGetGame(t *testing.T) {
-	t.Parallel()
-	a := NewAPI()
-	gid := dummyGame(a)
-	var tests = []struct {
-		pid  string
-		code int
-	}{
-		{"red", http.StatusOK},
-		{"blue", http.StatusOK},
-		{"green", http.StatusBadRequest},
-	}
-	for _, tt := range tests {
-		tname := fmt.Sprintf("/game from player %v", tt.pid)
-		t.Run(tname, func(t *testing.T) {
-			r, _ := http.NewRequest("GET", "/game", nil)
-			w := httptest.NewRecorder()
-			r = mux.SetURLVars(r, map[string]string{"id": strconv.Itoa(gid)})
-			r.Header.Add("Player-id", tt.pid)
-			r.Header.Add("Rotate", "false")
-			a.GetGame(w, r)
-			resp := w.Result()
-			if resp.StatusCode != tt.code {
-				t.Fatalf("failed to get game: %v", resp.Status)
-			}
-			if resp.StatusCode == http.StatusOK {
-				var respStruct gameResp
-				err := json.NewDecoder(resp.Body).Decode(&respStruct)
-				if err != nil {
-					t.Fatalf("/game returned bad response body: %v", err)
-				}
-				if len(respStruct.GameBoard) == 0 {
-					t.Errorf("bad game board returned")
-				}
-				for j := range respStruct.GameBoard {
-					for i, vt := range respStruct.GameBoard[j] {
-						curr, err := a.games[gid].simulator.Board.GetPiece(i, j)
-						if err != nil {
-							t.Fatalf("Strange board position: %v", err)
-						}
-						if curr != nil && !vt.Hidden && curr.Owner.String() != tt.pid && curr.Hidden {
-							t.Errorf("/game returned a piece that should be hidden but isn't")
-						}
-					}
-				}
-			}
-		})
-	}
-}

api_types.go

@@ -9,8 +9,16 @@ type newGameResp struct {
 	PlayerID string `json:"player_id"`
 }
 
+type gameReq struct {
+	PlayerID string `json:"player_id"`
+}
+
 type gameResp struct {
-	GameBoard [][]*ViewTile `json:"board"`
+	GameBoard [8][8]*ViewTile `json:"board"`
+}
+
+type gameStatusReq struct {
+	PlayerID string `json:"player_id"`
 }
 
 type gameStatusResp struct {
@@ -19,7 +27,8 @@ type gameStatusResp struct {
 }
 
 type gameMovePostReq struct {
-	Move string `json:"move"`
+	PlayerID string `json:"player_id"`
+	Move     string `json:"move"`
 }
 
 type gameMovePostRes struct {
@@ -29,6 +38,10 @@ type gameMovePostRes struct {
 	Error  error  `json:"error"`
 }
 
+type gameMoveReq struct {
+	PlayerID string `json:"player_id"`
+}
+
 type gameMoveRes struct {
 	Move string `json:"move"`
 }

docker-compose.yml

@@ -6,8 +6,4 @@ services:
   api:
     build: .
     ports:
-      - 1379:1379
-  web:
-    build: nginx
-    ports:
-      - 80:80
+      - 1379:1379

nginx/Dockerfile

@@ -1,3 +0,0 @@
-FROM nginx:latest AS base
-
-COPY ./nginx.conf /etc/nginx/conf.d/default.conf

nginx/nginx.conf

@@ -1,17 +0,0 @@
-server {
-  listen 80;
-  server_name steve;
-
-  location / {
-    proxy_set_header Host $host;
-    proxy_set_header X-Real-IP $remote_addr;
-    proxy_pass http://ui:3000;
-  }
-
-  location /api/ {
-    rewrite ^/api(/.*)$ $1 break;
-    proxy_set_header Host $host;
-    proxy_set_header X-Real-IP $remote_addr;
-    proxy_pass http://api:1379;
-  }
-}

session.go

@@ -15,7 +15,6 @@ type Session struct {
 	bluePlayer *Player
 	moveNum    int
 	moveList   []freego.ParsedCommand
-	boardSize  int
 }
 
 //Player is a player in a match
@@ -35,7 +34,7 @@ func (p *Player) Colour() freego.Colour {
 }
 
 //NewSession creates a new game session
-func NewSession(size int) *Session {
+func NewSession() *Session {
 	sim := freego.NewGame()
 	return &Session{
 		simulator:  sim,
@@ -43,7 +42,6 @@ func NewSession(size int) *Session {
 		bluePlayer: &Player{false, freego.Blue},
 		moveNum:    1,
 		moveList:   make([]freego.ParsedCommand, 20),
-		boardSize:  size,
 	}
 }
 
@@ -80,13 +78,10 @@ func (s *Session) getMove(p *Player, num int) (string, error) {
 	return fmt.Sprintf("%v %v", num, s.moveList[num].String()), nil
 }
 
-func (s *Session) getBoard(p *Player) [][]*ViewTile {
-	res := make([][]*ViewTile, s.boardSize)
-	for i := range res {
-		res[i] = make([]*ViewTile, s.boardSize)
-	}
-	for i := 0; i < s.boardSize; i++ {
-		for j := 0; j < s.boardSize; j++ {
+func (s *Session) getBoard(p *Player) [8][8]*ViewTile {
+	var res [8][8]*ViewTile
+	for i := 0; i < 8; i++ {
+		for j := 0; j < 8; j++ {
 			cur := NewViewTile()
 			terrain, err := s.simulator.Board.IsTerrain(i, j)
 			if err != nil {
@@ -114,7 +109,7 @@ func (s *Session) getBoard(p *Player) [][]*ViewTile {
 					cur.Empty = true
 				}
 			}
-			res[j][i] = cur
+			res[i][j] = cur
 		}
 	}
 	return res

ui/.prettierrc

@@ -1,5 +0,0 @@
-{
-  "tabWidth": 2,
-  "useTabs": false,
-  "semi": true
-}

ui/Dockerfile

@@ -8,11 +8,9 @@ FROM base as dependencies
 COPY pages /pages
 COPY public /public
 COPY styles /styles
-COPY api /api
 COPY components /components
 COPY next* /
 COPY tsconfig.json /
-COPY types.ts /
 
 EXPOSE 3000
 RUN npm run build

ui/api/game.api.ts

@@ -1,20 +0,0 @@
-import wretch from "wretch";
-import { Cell } from "../types";
-
-const USER_ID_HEADER = "Player-id";
-
-export const fetchGameState = async (
-  playerId: string,
-  gameId: number
-): Promise<{ cells: Cell[]; cellWidth: number }> => {
-  const response: { board: Cell[][] } = await wretch(`/api/game/${gameId}`)
-    .headers({
-      [USER_ID_HEADER]: playerId,
-    })
-    .get()
-    .json();
-  return {
-    cells: response.board.flat(),
-    cellWidth: response.board[0].length,
-  };
-};

ui/components/board.tsx

@@ -1,25 +1,7 @@
-import React from "react";
-import { Cell } from "../types";
-import styles from "../styles/board.module.css";
+import React from 'react'
 
-interface BoardProps {
-  cells: Cell[];
-  cellWidth: number;
-}
+const Board = () => (
+  <input/>
+)
 
-const Board = (props: BoardProps) => {
-  return (
-    <div
-      className={styles.gridContainer}
-      style={{
-        gridTemplateColumns: `repeat(${props.cellWidth}, 1fr)`,
-      }}
-    >
-      {props.cells.map((cell) => (
-        <div className={styles.gridCell}>{cell.piece}</div>
-      ))}
-    </div>
-  );
-};
-
-export default Board;
+export default Board;

ui/components/game.tsx

@@ -1,28 +0,0 @@
-import React, { useState, useEffect } from "react";
-import { fetchGameState } from "../api/game.api";
-import { Cell } from "../types";
-import Board from "./board";
-
-interface GameProps {
-  gameId: number;
-}
-
-const Game = (props: GameProps) => {
-  const [isLoading, setLoading] = useState(false);
-  const [cellWidth, setCellWidth] = useState(4);
-  const [cells, setCells] = useState([] as Cell[]);
-  useEffect(() => {
-    setLoading(true);
-    fetchGameState("red", props.gameId).then(
-      ({ cells: cellList, cellWidth: width }) => {
-        setCellWidth(width);
-        setCells(cellList);
-        setLoading(false);
-      }
-    );
-  }, [props.gameId]);
-
-  return <Board cellWidth={cellWidth} cells={cells} />;
-};
-
-export default Game;

ui/package-lock.json

@@ -10,8 +10,7 @@
       "dependencies": {
         "next": "12.1.0",
         "react": "17.0.2",
-        "react-dom": "17.0.2",
-        "wretch": "^1.7.9"
+        "react-dom": "17.0.2"
       },
       "devDependencies": {
         "@types/node": "17.0.21",
@@ -2528,11 +2527,6 @@
       "dev": true,
       "license": "ISC"
     },
-    "node_modules/wretch": {
-      "version": "1.7.9",
-      "resolved": "https://registry.npmjs.org/wretch/-/wretch-1.7.9.tgz",
-      "integrity": "sha512-uUSze1Z72RiQjyoqr7r1KW+05WDNeqqKOeyJDPhw6EVEaOgp9RQNrr8AQt3OF7qylQbh2iVtT9r0nXIHlbJgqQ=="
-    },
     "node_modules/yallist": {
       "version": "4.0.0",
       "dev": true,
@@ -4066,11 +4060,6 @@
       "version": "1.0.2",
       "dev": true
     },
-    "wretch": {
-      "version": "1.7.9",
-      "resolved": "https://registry.npmjs.org/wretch/-/wretch-1.7.9.tgz",
-      "integrity": "sha512-uUSze1Z72RiQjyoqr7r1KW+05WDNeqqKOeyJDPhw6EVEaOgp9RQNrr8AQt3OF7qylQbh2iVtT9r0nXIHlbJgqQ=="
-    },
     "yallist": {
       "version": "4.0.0",
       "dev": true

ui/package.json

@@ -11,8 +11,7 @@
   "dependencies": {
     "next": "12.1.0",
     "react": "17.0.2",
-    "react-dom": "17.0.2",
-    "wretch": "^1.7.9"
+    "react-dom": "17.0.2"
   },
   "devDependencies": {
     "@types/node": "17.0.21",

ui/pages/index.tsx

@@ -1,19 +1,19 @@
-import type { NextPage } from "next";
-import Head from "next/head";
-import Image from "next/image";
-import Game from "../components/game";
-import styles from "../styles/BoardPage.module.css";
+import type { NextPage } from 'next'
+import Head from 'next/head'
+import Image from 'next/image'
+import Board from '../components/board'
+import styles from '../styles/Home.module.css'
 
 const Home: NextPage = () => (
-  <div className={styles.main}>
+  <>
     <Head>
       <title>Free Go Game</title>
       <link rel="icon" href="/favicon.ico" />
     </Head>
     <main>
-      <Game gameId={1} />
+      <Board />
     </main>
-  </div>
-);
+  </>
+)
 
-export default Home;
+export default Home

ui/styles/BoardPage.module.css

@@ -1,4 +0,0 @@
-.main {
-  max-width: 900px;
-  margin: auto;
-}

ui/styles/board.module.css

@@ -1,8 +0,0 @@
-.gridContainer {
-  display: grid;
-}
-
-.gridCell {
-  border: 1px solid black;
-  padding-top: 100%; /* 100% is supposed to be 1:1 aspect ratio but it doesn't look like it */ 
-}

ui/types.ts

@@ -1,6 +0,0 @@
-export interface Cell {
-  piece?: string;
-  terrain: boolean;
-  hidden: boolean;
-  empty: boolean;
-}

util.go

@@ -21,16 +21,6 @@ func respondWithJSON(res http.ResponseWriter, code int, payload interface{}) {
 	res.Write(response)
 }
 
-func rotateBoard(board [][]*ViewTile) {
-	i := 0
-	temp := board[0]
-	for ; i < len(board)-1; i++ {
-		board[i] = board[i+1]
-	}
-
-	board[i] = temp
-}
-
 //TODO remove this when you can actually setup a game
 func initDummy(g *freego.Game) {
 	//Setup terrain

view_tile.go

@@ -12,16 +12,3 @@ type ViewTile struct {
 func NewViewTile() *ViewTile {
 	return &ViewTile{}
 }
-
-func (vt *ViewTile) String() string {
-	if vt.Piece != "" {
-		return vt.Piece
-	}
-	if vt.Hidden {
-		return "?"
-	}
-	if vt.Terrain {
-		return "X"
-	}
-	return " "
-}