From 612fc8ac07fd8b9a5af81269bbc0ff9638c6664c Mon Sep 17 00:00:00 2001
From: Denis Zheleztsov <difrex.punk@gmail.com>
Date: Tue, 6 Nov 2018 12:38:38 +0300
Subject: [PATCH] Point message validation

---
 node/api.go   | 7 +++++--
 node/point.go | 3 +++
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/node/api.go b/node/api.go
index 30ab927..bbbfd8d 100644
--- a/node/api.go
+++ b/node/api.go
@@ -83,7 +83,7 @@ func (es ESConf) UMHandler(w http.ResponseWriter, r *http.Request) {
 	vars := mux.Vars(r)
 	e := vars["ids"]
 
-	log.Print("/u/e/ vars: ", e)
+	log.Print("/u/m/ vars: ", e)
 
 	LogRequest(r)
 
@@ -155,6 +155,9 @@ func (es ESConf) UPointHandler(w http.ResponseWriter, r *http.Request) {
 	// Get plain POST variables
 	if err := r.ParseForm(); err != nil {
 		log.Error("Fail to parse POST args: ", err.Error())
+		w.WriteHeader(500)
+		w.Write([]byte(fmt.Sprintf("error: %s", err.Error())))
+		return
 	}
 	pauth := r.Form.Get("pauth")
 	tmsg := r.Form.Get("tmsg")
@@ -183,7 +186,7 @@ func (es ESConf) UPointHandler(w http.ResponseWriter, r *http.Request) {
 
 	// Proccess point message
 	if err := es.PointMessage(req, user); err != nil {
-		log.Error(err.Error())
+		log.Error("Fail to parse point message: ", err.Error())
 		w.WriteHeader(500)
 		w.Write([]byte(fmt.Sprintf("error: %s", err.Error())))
 		return
diff --git a/node/point.go b/node/point.go
index ee5afb8..71b8241 100644
--- a/node/point.go
+++ b/node/point.go
@@ -31,6 +31,9 @@ func (es ESConf) PointMessage(req PointRequest, user User) error {
 	if err != nil {
 		return err
 	}
+	if err := pmsg.Validate(); err != nil {
+		return err
+	}
 
 	bmsg, err := idec.MakeBundledMessage(pmsg)
 	if err != nil {