]> git.pond.sub.org Git - eow/blobdiff - empire.lisp
Fix retry logic. Remove extraneous newlines in logging.
[eow] / empire.lisp
index 4797ac71e4817e424e26ad250c26ffab57d82ad5..34aea1367f279b6a5c6130f00719a420083b3673 100644 (file)
     :initarg :stream
     :accessor network-stream
     :documentation "Stream used to talk to the empire server.")
-   (logging-stream
-    :initarg :logging-stream
-    :accessor logging-stream
-    :initform t
-    :documentation "Messages coming back from the server is sent to
-this stream.")
    (connection-handler
     :accessor connection-handler
     :initform nil
@@ -46,7 +40,11 @@ this stream.")
    (mode
     :accessor connection-mode
     :initform nil
-    :documentation "TODO(gefla)")))
+    :documentation "TODO(gefla)")
+   (session
+    :accessor session
+    :initarg :session
+    :documentation "web session to which this connection belongs")))
 
 (defun make-connection (&key (user nil)
                        (password nil)
@@ -54,7 +52,7 @@ this stream.")
                        (server-port *default-empire-server-port*)
                        (socket nil)
                        (network-stream nil)
-                       (logging-stream nil))
+                       session)
   (make-instance 'connection
                 :user user
                 :password password
@@ -62,14 +60,14 @@ this stream.")
                 :server-port server-port
                 :socket socket
                 :stream network-stream
-                :logging-stream logging-stream))
+                :session session))
 
 ;;; interface
 (defun connect  (&key (user nil)
                 (password nil)
                 (server-name *default-empire-server*)
                 (server-port *default-empire-server-port*)
-                (logging-stream *standard-output*))
+                session)
   "Connect to server and return a connection object."
   (let* ((socket (usocket:socket-connect server-name server-port))
          (stream (usocket:socket-stream socket))
@@ -77,19 +75,31 @@ this stream.")
                                      :password password
                                      :socket socket
                                       :network-stream stream
-                                      :logging-stream logging-stream
                                       :server-name server-name
-                                     :server-port server-port))
+                                     :server-port server-port
+                                     :session session))
         (handler (spawn-with-name (format nil "empire-handler-~a" user)
                                   #'handle-connection connection)))
     (setf (connection-handler connection) handler)
+    (empire-log:info "empire:connect: ~a" connection)
     connection))
 
-
 (defgeneric quit (connection))
 (defgeneric handle-connection (connection))
 (defgeneric read-message (connection))
 (defgeneric send-message (connection message))
+(defgeneric reconnect (conncetion))
+
+(defmethod reconnect ((c connection))
+  (with-slots (server-name server-port user password socket stream connection-handler) c
+      (let* ((sock (usocket:socket-connect server-name server-port))
+            (s (usocket:socket-stream sock))
+            (handler (spawn-with-name (format nil "empire-handler-~a" user)
+                                      #'handle-connection c)))
+       (setf socket sock
+             stream s
+             connection-handler handler)))
+  (empire-log:info "~a: reconnect" c))
 
 (defparameter +C_CMDOK+ "0")
 (defparameter +C_DATA+ "1")
@@ -121,11 +131,15 @@ this stream.")
 (defmethod set-new-mode ((m base-mode) new-mode)
   (let* ((c (connection m))
         (mode (make-mode c new-mode)))
+    (empire-log:info "~a: set-new-mode ~a -> ~a" c (connection-mode c) new-mode)
     (setf (connection-mode c) mode)))
 
 (defgeneric handle-data (base-mode message))
 (defgeneric handle-init (base-mode message))
+(defgeneric handle-exit (base-mode message))
 (defgeneric handle-prompt (base-mode message))
+(defgeneric handle-flush (base-mode message))
+(defgeneric handle-simple-message (base-mode message))
 
 (defmethod handle-init ((m init-mode) message)
   (declare (ignorable message))
@@ -136,22 +150,41 @@ this stream.")
          (setf play-sent-p t))
        (set-new-mode m 'play-mode))))
 
+(defmethod handle-exit ((m init-mode) message)
+  (declare (ignorable message))
+  (quit (connection m)))
+
+(defmethod handle-exit ((m play-mode) message)
+  (declare (ignorable message))
+  (quit (connection m)))
+
 (defmethod handle-data ((m play-mode) message)
-  (let* ((c (connection m))
-        (log (logging-stream c)))
-    (format log "handle-data ~a~%" message)))
+  (let* ((c (connection m)))
+    (empire-web:data (session c) message)))
+
+(defmethod handle-simple-message ((m play-mode) message)
+  (let* ((c (connection m)))
+    (empire-web:data (session c) message)))
+
+(defmethod handle-flush ((m play-mode) message)
+  (let* ((c (connection m)))
+    (empire-web:prompt (session c) message)))
 
 (defmethod handle-prompt ((m play-mode) message)
-  (let* ((c (connection m))
-        (log (logging-stream c)))
+  (let* ((c (connection m)))
     (with-input-from-string (s message)
-      (let* ((minutes (read s))
+      (let* ((minutes (read s)) ;;FIXME DANGEROUS
             (btus (read s)))
-       (format log "[~a:~a]: ~%" minutes btus)))))
+       (empire-web:prompt (session c)
+                          (format nil "[~a,~a]: " minutes btus))))))
 
 (defparameter *line-type-dispatch*
   (list `(,+C_DATA+ . handle-data)
        `(,+C_INIT+ . handle-init)
+       `(,+C_EXIT+ . handle-exit)
+       `(,+C_FLUSH+ . handle-flush)
+       `(,+C_BADCMD+ . handle-simple-message)
+       `(,+C_FLASH+ . handle-simple-message)
        `(,+C_PROMPT+ . handle-prompt)))
 
 (defun parse-server-line (line)
@@ -160,7 +193,9 @@ this stream.")
             if (eq (char line i) #\Space)
             do (return i)))
         (message-type (subseq line 0 first-space-index))
-        (message (subseq line (+ first-space-index 1))))
+        (message (if first-space-index
+                     (subseq line (+ first-space-index 1))
+                     "")))
     (values message message-type)))
 
 (defmethod handle-connection ((c connection))
@@ -178,26 +213,59 @@ this stream.")
               (format stream "No handler for ~a in mode ~a."
                       type mode)))))
 (defmethod read-message ((c connection))
-  (let* ((s (network-stream c))
-        (log (logging-stream c))
-        (line (read-line s)))
-    (format log "< ~a~%" line)
-    (multiple-value-bind (message type) (parse-server-line line)
-      (let ((handler (lookup-handler type))
-           (mode (connection-mode c)))
-       (if handler
-           (funcall handler mode message)
-           (error 'no-handler :mode mode :type type))))
-    line))
-  
-(defmethod send-message ((c connection) message)
-  (let ((s (network-stream c))
-       (log (logging-stream c)))
-    (format log "< ~a~%" message)
+  (handler-case
+      (let* ((s (network-stream c))
+            (line (read-line s)))
+       (empire-log:info "~a: < ~a" c line)
+       (multiple-value-bind (message type) (parse-server-line line)
+         (let ((handler (lookup-handler type))
+               (mode (connection-mode c)))
+           (if handler
+               (funcall handler mode message)
+               (error 'no-handler :mode mode :type type))))
+       line)
+    (sb-int:closed-stream-error () nil)
+    (end-of-file () nil)))
+
+(defun raw-send-message (s message)
     (write-sequence message s)
     (terpri s)
-    (force-output s)
-    message))
+    (force-output s))
+  
+(defmethod send-message ((c connection) message)
+  (empire-log:info "~a: > ~a" c message)
+  (let ((sent-p nil)
+       (tries 3))
+    (loop
+       while (and (not sent-p)
+                 (> tries 0))
+       do (handler-case
+             (let ((s (network-stream c)))
+               (raw-send-message s message)
+               (setf sent-p t))
+           (sb-int:closed-stream-error ()
+             (progn
+               (decf tries)
+               (empire-log:info "~a: Connection close - retrying (~a tries left)" c tries)
+               (reconnect c))))))
+  message)
+
+(defparameter +special-command-char+ #\;)
+
+(defun special-command-p (line)
+  (char= +special-command-char+ (aref line 0)))
+
+(defmethod special-xup ((c connection))
+  t)
+
+(defmethod special-command ((c connection) line)
+  (cond ((string= line "xup") (special-xup c))
+       (t (empire-web:data (session c) "Unknown special command")))
+  line)
+
+(defmethod command ((c connection) line)
+  (cond ((special-command-p line) (special-command c (subseq line 1)))
+       (t (send-message c line))))
 
 (defgeneric play (connection))
 (defmethod play ((c connection))