fix(swp): we store urllist as empty string, when we split on newline we will get a slice length of 1 when urllist is empty string

Author: erikvatt

pkg/service/core/storage/postgres/postgres_workstations.go

@@ -98,20 +98,25 @@ func (s *workstationsStorage) GetLastWorkstationsOnpremAllowList(ctx context.Con
 func (s *workstationsStorage) GetLastWorkstationsURLList(ctx context.Context, navIdent string) (*service.WorkstationURLList, error) {
 	const op errs.Op = "workstationsStorage.GetLastWorkstationsURLListChange"
 
+	urlAllowList := []string{}
 	raw, err := s.db.Querier.GetLastWorkstationsURLListChange(ctx, navIdent)
 	if err != nil {
 		if errors.Is(err, sql.ErrNoRows) {
 			return &service.WorkstationURLList{
-				URLAllowList:           []string{},
+				URLAllowList:           urlAllowList,
 				DisableGlobalAllowList: false,
 			}, nil
 		}
 
 		return nil, errs.E(errs.Database, service.CodeDatabase, op, err)
 	}
 
+	if raw.UrlList != "" {
+		urlAllowList = strings.Split(raw.UrlList, "\n")
+	}
+
 	return &service.WorkstationURLList{
-		URLAllowList:           strings.Split(raw.UrlList, "\n"),
+		URLAllowList:           urlAllowList,
 		DisableGlobalAllowList: raw.DisableGlobalUrlList,
 	}, nil
 }