add time parsing error

Author: Kaya-Sem

api/connection.go

@@ -6,16 +6,29 @@ import (
 	"io"
 	"net/http"
 	"os"
+	"regexp"
 	"strconv"
 
 	"github.com/Kaya-Sem/commandtrein/internal/util"
 )
 
 /*
 GetConnections fetches the connection data from the API and returns the response body as a byte slice.
-
 https://docs.irail.be/#connections
 */
+
+func validateTimeRegex(timeStr string) bool {
+	// Pattern explanation:
+	// ^           - start of string
+	// ([01][0-9]  - 00-19 (first two digits)
+	// |2[0-3])    - OR 20-23 (hours)
+	// [0-5][0-9]  - 00-59 (minutes)
+	// $           - end of string
+	pattern := `^([01][0-9]|2[0-3])[0-5][0-9]$`
+	matched, _ := regexp.MatchString(pattern, timeStr)
+	return matched
+}
+
 func GetConnections(stationFrom string, stationTo string, time string, departure bool, date string) ([]byte, error) {
 
 	timesel := "arrival"
@@ -25,8 +38,14 @@ func GetConnections(stationFrom string, stationTo string, time string, departure
 
 	if time == "" {
 		time = util.GetBelgiumTimeHHMM()
+	} else {
+		if !validateTimeRegex(time) {
+			return nil, fmt.Errorf("Time does not follow hhmm format")
+		}
 	}
 
+	// TODO: do time checking
+
 	url := fmt.Sprintf("https://api.irail.be/connections/?from=%s&to=%s&time=%s&timesel=%s&format=json&lang=nl&typeOfTransport=automatic&alerts=false&results=10",
 		stationFrom,
 		stationTo,

cmd/connection.go

@@ -85,7 +85,7 @@ func getNextValidDay(dayName string) string {
 	return fmt.Sprintf("%02d%02d%02d", nextValidDay.Day(), nextValidDay.Month(), nextValidDay.Year()%100)
 }
 
-func handleConnection(stationFrom string, stationTo string, queryMode string, departure bool, date string) {
+func handleConnection(stationFrom string, stationTo string, timeQuery string, departure bool, date string) {
 	s := util.NewSpinner("", " fetching connections", 1*time.Second)
 	s.Start()
 
@@ -100,7 +100,7 @@ func handleConnection(stationFrom string, stationTo string, queryMode string, de
 		}
 	}
 
-	connectionsJSON, err := api.GetConnections(stationFrom, stationTo, queryMode, departure, parsedDate)
+	connectionsJSON, err := api.GetConnections(stationFrom, stationTo, timeQuery, departure, parsedDate)
 	if err != nil {
 		panic(err)
 	}

cmd/version.go

@@ -6,7 +6,7 @@ import (
 	"github.com/spf13/cobra"
 )
 
-const Version = "2.3.13"
+const Version = "2.3.14"
 
 func NewVersionCommand() *cobra.Command {
 	return &cobra.Command{