Merge pull request #16 from nicholas-fedor/15-no-error-checks-for-fmtsscanf-and-templrender-methods

15 no error checks for fmtsscanf and templrender methods

Author: nicholas-fedor

internal/eui64/eui64.go

@@ -64,7 +64,10 @@ func CalculateEUI64(macStr, prefixStr string) (string, string, error) {
 	ip6 := make([]uint16, 8)
 	for i := 0; i < 8; i++ {
 		if i < len(prefixParts) {
-			fmt.Sscanf(prefixParts[i], "%x", &ip6[i])
+			_, err := fmt.Sscanf(prefixParts[i], "%x", &ip6[i])
+			if err != nil {
+				return "", "", fmt.Errorf("failed to parse hextet: %v", err)
+			}
 		} else if i < 4 {
 			ip6[i] = 0 // Fill remaining prefix hextets with zeros
 		} else {

internal/handlers/handlers.go

@@ -1,6 +1,9 @@
 package handlers
 
 import (
+	"log"
+	"net/http"
+
 	"github.com/gin-gonic/gin"
 	"github.com/nicholas-fedor/EUI64-Calculator/internal/eui64"
 	"github.com/nicholas-fedor/EUI64-Calculator/ui"
@@ -14,7 +17,13 @@ func NewHandler() *Handler {
 }
 
 func (h *Handler) Home(c *gin.Context) {
-	ui.Home().Render(c.Request.Context(), c.Writer)
+	if err := ui.Home().Render(c.Request.Context(), c.Writer); err != nil {
+		if err := c.AbortWithError(http.StatusInternalServerError, err); err != nil {
+			// Log the error since there's nothing more we can do here
+			log.Printf("Failed to abort with error: %v", err)
+		}
+		return
+	}
 }
 
 func (h *Handler) Calculate(c *gin.Context) {
@@ -31,5 +40,11 @@ func (h *Handler) Calculate(c *gin.Context) {
 		data.Error = err.Error()
 	}
 
-	ui.Result(data).Render(c.Request.Context(), c.Writer)
+	if err := ui.Result(data).Render(c.Request.Context(), c.Writer); err != nil {
+		if err := c.AbortWithError(http.StatusInternalServerError, err); err != nil {
+			// Log the error for debugging purposes
+			log.Printf("Failed to abort with error: %v", err)
+		}
+		return
+	}
 }