PATCH: pagination handling and enable nullable subscription IDs

The commit improved the pagination utility to handle situations where the offset exceeds the number of records. It also updated the subscription handler to support nullable follower and following IDs. Both changes improve data consistency and cater to potential edge cases.

deployments/docker-compose.prod.yml

@@ -9,8 +9,6 @@ services:
       - POSTGRES_DB=${DB_NAME}
       - POSTGRES_USER=${DB_USER}
       - POSTGRES_PASSWORD=${DB_PASS}
-    ports:
-      - "${DB_PORT}:5432"
     volumes:
       - db_data:/var/lib/postgresql/data
       - ./server_alpha_db.sql:/docker-entrypoint-initdb.d/initdb.sql
@@ -36,8 +34,6 @@ services:
       - DB_USER=${DB_USER}
       - DB_PASS=${DB_PASS}
       - MAILGUN_API_KEY=${MAILGUN_API_KEY}
-    ports:
-      - "${APP_PORT}:8080"
     volumes:
       - ${PWD}/private_key.pem:/private_key.pem
       - ${PWD}/public_key.pem:/public_key.pem

internal/handlers/post_handler.go

@@ -13,6 +13,7 @@ import (
 	"server-alpha/internal/schemas"
 	"server-alpha/internal/utils"
 	"strconv"
+	"strings"
 	"time"
 
 	"github.com/golang-jwt/jwt/v5"
@@ -323,6 +324,12 @@ func determineFeedType(r *http.Request, w http.ResponseWriter, handler *PostHand
 	if authHeader == "" {
 		publicFeedWanted = true
 	} else {
+		if !strings.HasPrefix(authHeader, "Bearer ") || len(authHeader) <= len("Bearer ") {
+			err = errors.New("invalid authorization header")
+			utils.WriteAndLogError(w, schemas.InvalidToken, http.StatusBadRequest, err)
+			return false, nil, err
+		}
+
 		jwtToken := authHeader[len("Bearer "):]
 		claims, err = handler.JWTManager.ValidateJWT(jwtToken)
 		if err != nil {

internal/handlers/subscription_handler.go

@@ -105,15 +105,12 @@ func (handler *SubscriptionHandler) HandleGetSubscriptions(w http.ResponseWriter
 		}
 
 		if followingId != uuid.Nil {
-			*subscription.FollowingId = followingId.String()
+			followingIdStr := followingId.String()
+			subscription.FollowingId = &followingIdStr
 		} else {
 			subscription.FollowingId = nil
 		}
 
-		if followerId != uuid.Nil {
-			*subscription.FollowerId = followerId.String()
-		}
-
 		results = append(results, subscription)
 	}
 

internal/utils/pagination_utils.go

@@ -41,21 +41,17 @@ func ParsePaginationParams(r *http.Request) (int, int, error) {
 
 func SendPaginatedResponse(w http.ResponseWriter, records interface{}, offset, limit, totalRecords int) {
 	// Get a reflect.Value of records.
+
 	v := reflect.ValueOf(records)
+	if offset > v.Len() {
+		offset = v.Len()
+	}
 
 	end := offset + limit
 	if end > v.Len() {
 		end = v.Len()
 	}
 
-	if v.Len() == 0 {
-		offset = 0
-	}
-
-	if offset > v.Len() {
-		offset = v.Len() - 1
-	}
-
 	// Check if v is not a slice.
 	if v.Kind() != reflect.Slice {
 		WriteAndLogError(w, schemas.BadRequest, http.StatusBadRequest, errors.New("records not a valid list"))