Fix: Disable integration credential validation and add nil checks (#579)

## Summary

This PR addresses potential issues with integration credential
validation by:

1. **Disabling active validation** for GitHub PAT, Google, and GitLab
credentials in the status endpoint
2. **Adding nil pointer checks** in git package initialization

## Changes

### Integration Status (integrations_status.go)
- Removed active token validation for GitHub PAT, Google OAuth, and
GitLab tokens
- Status endpoint now assumes stored credentials are valid
- Added clear comments explaining that integrations will fail gracefully
if credentials are invalid

### Git Package Initialization (main.go)
- Added nil checks for GitHub installation, GitHub PAT credentials, and
GitLab credentials
- Prevents potential nil pointer dereferences when these functions
return nil

## Rationale

Active validation of credentials on every status check can cause:
- Unnecessary API calls to external services
- Potential rate limiting issues
- False negatives due to temporary network issues

Since the backend auto-refreshes tokens and integrations fail gracefully
on actual usage, checking for credential existence is sufficient for the
status endpoint.

## Testing

- [ ] Backend builds successfully
- [ ] Integration status endpoints return expected data
- [ ] No nil pointer dereferences occur during git operations

Author: Gkrumbach07

components/backend/handlers/integrations_status.go

@@ -3,7 +3,6 @@ package handlers
 import (
 	"context"
 	"net/http"
-	"time"
 
 	"github.com/gin-gonic/gin"
 )
@@ -63,13 +62,13 @@ func getGitHubStatusForUser(ctx context.Context, userID string) gin.H {
 	// Check GitHub PAT
 	patCreds, err := GetGitHubPATCredentials(ctx, userID)
 	if err == nil && patCreds != nil {
-		// Validate PAT token
-		valid, _ := ValidateGitHubToken(ctx, patCreds.Token)
+		// NOTE: Validation disabled - if credentials are stored, assume they're valid
+		// The integration will fail gracefully if credentials are actually invalid
 
 		status["pat"] = gin.H{
 			"configured": true,
 			"updatedAt":  patCreds.UpdatedAt.Format("2006-01-02T15:04:05Z07:00"),
-			"valid":      valid,
+			"valid":      true,
 		}
 	}
 
@@ -89,19 +88,15 @@ func getGoogleStatusForUser(ctx context.Context, userID string) gin.H {
 		return gin.H{"connected": false}
 	}
 
-	// Check if token is expired
-	isExpired := time.Now().After(creds.ExpiresAt)
-	valid := !isExpired
-
-	// If near expiry, could validate with Google API, but checking expiry is sufficient
-	// since backend auto-refreshes tokens
+	// NOTE: Validation disabled - if credentials are stored, assume they're valid
+	// The backend auto-refreshes tokens and the integration will fail gracefully if invalid
 
 	return gin.H{
 		"connected": true,
 		"email":     creds.Email,
 		"expiresAt": creds.ExpiresAt.Format("2006-01-02T15:04:05Z07:00"),
 		"updatedAt": creds.UpdatedAt.Format("2006-01-02T15:04:05Z07:00"),
-		"valid":     valid,
+		"valid":     true,
 	}
 }
 
@@ -131,13 +126,13 @@ func getGitLabStatusForUser(ctx context.Context, userID string) gin.H {
 		return gin.H{"connected": false}
 	}
 
-	// Validate token
-	valid, _ := ValidateGitLabToken(ctx, creds.Token, creds.InstanceURL)
+	// NOTE: Validation disabled - if credentials are stored, assume they're valid
+	// The integration will fail gracefully if credentials are actually invalid
 
 	return gin.H{
 		"connected":   true,
 		"instanceUrl": creds.InstanceURL,
 		"updatedAt":   creds.UpdatedAt,
-		"valid":       valid,
+		"valid":       true,
 	}
 }

components/backend/main.go

@@ -94,13 +94,25 @@ func main() {
 	// Initialize git package
 	git.GetProjectSettingsResource = k8s.GetProjectSettingsResource
 	git.GetGitHubInstallation = func(ctx context.Context, userID string) (interface{}, error) {
-		return github.GetInstallation(ctx, userID)
+		installation, err := github.GetInstallation(ctx, userID)
+		if installation == nil {
+			return nil, err
+		}
+		return installation, err
 	}
 	git.GetGitHubPATCredentials = func(ctx context.Context, userID string) (interface{}, error) {
-		return handlers.GetGitHubPATCredentials(ctx, userID)
+		creds, err := handlers.GetGitHubPATCredentials(ctx, userID)
+		if creds == nil {
+			return nil, err
+		}
+		return creds, err
 	}
 	git.GetGitLabCredentials = func(ctx context.Context, userID string) (interface{}, error) {
-		return handlers.GetGitLabCredentials(ctx, userID)
+		creds, err := handlers.GetGitLabCredentials(ctx, userID)
+		if creds == nil {
+			return nil, err
+		}
+		return creds, err
 	}
 	git.GitHubTokenManager = github.Manager
 	git.GetBackendNamespace = func() string {