fix issue with design on n8n page

This commit is contained in:
usmannasir
2025-04-12 13:28:43 +05:00
parent 9fbae661f4
commit b92bd63a26
4 changed files with 431 additions and 25 deletions

View File

@@ -113,7 +113,9 @@ class N8nAPI:
include_executions: Whether to include execution history in the backup
Returns:
Backup data or None if there was an error
Dict containing backup data or error information
- On success: {'success': True, 'data': backup_data}
- On failure: {'success': False, 'error': error_message}
"""
try:
response = self.client.post(
@@ -121,16 +123,43 @@ class N8nAPI:
json={
"includeCredentials": include_credentials,
"includeExecutions": include_executions
}
},
timeout=30 # Add a 30 second timeout
)
if response.status_code == 200:
return response.json()
logging.writeToFile(f"Error creating backup. Status code: {response.status_code}, Response: {response.text}")
return None
if response.status_code == 200:
# Validate the response contains expected data
backup_data = response.json()
if not isinstance(backup_data, dict):
logging.writeToFile(f"Invalid backup data format: {type(backup_data)}")
return {'success': False, 'error': 'Invalid backup data format returned from n8n'}
return {'success': True, 'data': backup_data}
error_msg = f"Error creating backup. Status code: {response.status_code}"
if response.text:
try:
error_data = response.json()
if 'message' in error_data:
error_msg += f", Message: {error_data['message']}"
except:
error_msg += f", Response: {response.text[:200]}"
logging.writeToFile(error_msg)
return {'success': False, 'error': error_msg}
except requests.exceptions.Timeout:
error_msg = "Timeout while connecting to n8n API"
logging.writeToFile(error_msg)
return {'success': False, 'error': error_msg}
except requests.exceptions.ConnectionError:
error_msg = "Connection error while connecting to n8n API"
logging.writeToFile(error_msg)
return {'success': False, 'error': error_msg}
except Exception as e:
logging.writeToFile(f"Error creating backup: {str(e)}")
return None
error_msg = f"Error creating backup: {str(e)}"
logging.writeToFile(error_msg)
return {'success': False, 'error': error_msg}
def restore_backup(self, backup_data):
"""
@@ -302,39 +331,67 @@ def create_n8n_backup(request):
# Get container info
client = docker.from_env()
try:
container = client.containers.get(container_id)
except docker.errors.NotFound:
return HttpResponse(json.dumps({
'status': 0,
'error_message': f'Container with ID {container_id} not found'
}))
except Exception as e:
error_msg = f"Error getting Docker container: {str(e)}"
logging.writeToFile(error_msg)
return HttpResponse(json.dumps({
'status': 0,
'error_message': error_msg
}))
# Extract container metadata
container_info = container.attrs
n8n_port = None
# Find the n8n port
# Find the n8n port - check different formats
ports = container_info.get('NetworkSettings', {}).get('Ports', {})
port_mappings = []
if '5678/tcp' in ports and ports['5678/tcp']:
n8n_port = ports['5678/tcp'][0].get('HostPort')
elif '5678' in ports and ports['5678']:
n8n_port = ports['5678'][0].get('HostPort')
# If still no port found, try to check exposed ports
if not n8n_port:
exposed_ports = container_info.get('Config', {}).get('ExposedPorts', {})
port_mappings = list(exposed_ports.keys())
# Find any port that might be mapped to 5678
for port in ports:
port_mappings.append(f"{port} -> {ports[port]}")
if not n8n_port:
logging.writeToFile(f"Could not find n8n port mapping in {ports}")
port_details = ", ".join(port_mappings) if port_mappings else "No port mappings found"
error_msg = f"Could not find n8n port mapping. Available ports: {port_details}"
logging.writeToFile(error_msg)
return HttpResponse(json.dumps({
'status': 0,
'error_message': 'Could not find n8n port mapping'
'error_message': error_msg
}))
# Create N8nAPI instance
n8n_api = N8nAPI(container, 'localhost', n8n_port)
# Create backup
backup = n8n_api.create_backup(include_credentials, include_executions)
backup_result = n8n_api.create_backup(include_credentials, include_executions)
if backup:
if backup_result['success']:
return HttpResponse(json.dumps({
'status': 1,
'backup': backup
'backup': backup_result['data']
}))
else:
return HttpResponse(json.dumps({
'status': 0,
'error_message': 'Failed to create backup'
'error_message': backup_result['error']
}))
return HttpResponse(json.dumps({
@@ -342,10 +399,11 @@ def create_n8n_backup(request):
'error_message': 'Invalid request method'
}))
except Exception as e:
logging.writeToFile(f"Error in create_n8n_backup: {str(e)}")
error_msg = f"Error in create_n8n_backup: {str(e)}"
logging.writeToFile(error_msg)
return HttpResponse(json.dumps({
'status': 0,
'error_message': str(e)
'error_message': error_msg
}))
@csrf_exempt
@@ -405,3 +463,110 @@ def restore_n8n_backup(request):
'status': 0,
'error_message': str(e)
}))
@csrf_exempt
def diagnose_n8n_api(request):
try:
if request.method == 'POST':
userID = request.session['userID']
currentACL = ACLManager.loadedACL(userID)
admin = Administrator.objects.get(pk=userID)
data = json.loads(request.body)
container_id = data.get('container_id')
# Get container info
client = docker.from_env()
try:
container = client.containers.get(container_id)
except docker.errors.NotFound:
return HttpResponse(json.dumps({
'status': 0,
'error_message': f'Container with ID {container_id} not found',
'diagnostics': {
'container_exists': False
}
}))
# Container exists
diagnostics = {
'container_exists': True,
'container_status': container.status,
'container_running': container.status == 'running',
'container_names': container.name,
'port_mappings': {},
'exposed_ports': {},
'n8n_port_found': False,
'api_accessible': False
}
# Extract port mappings
container_info = container.attrs
ports = container_info.get('NetworkSettings', {}).get('Ports', {})
exposed_ports = container_info.get('Config', {}).get('ExposedPorts', {})
diagnostics['port_mappings'] = ports
diagnostics['exposed_ports'] = exposed_ports
# Find the n8n port
n8n_port = None
if '5678/tcp' in ports and ports['5678/tcp']:
n8n_port = ports['5678/tcp'][0].get('HostPort')
diagnostics['n8n_port'] = n8n_port
diagnostics['n8n_port_found'] = True
elif '5678' in ports and ports['5678']:
n8n_port = ports['5678'][0].get('HostPort')
diagnostics['n8n_port'] = n8n_port
diagnostics['n8n_port_found'] = True
# Only proceed if container is running and port was found
if diagnostics['container_running'] and diagnostics['n8n_port_found']:
# Test n8n API connection
try:
n8n_api = N8nAPI(container, 'localhost', n8n_port)
# Try to access the health endpoint
response = n8n_api.client.get(f"{n8n_api.base_url}/rest/health", timeout=5)
diagnostics['api_response_code'] = response.status_code
diagnostics['api_accessible'] = response.status_code == 200
if diagnostics['api_accessible']:
# Try to get some basic info
try:
# Check if workflows endpoint is accessible
workflow_response = n8n_api.client.get(f"{n8n_api.base_url}/rest/workflows", timeout=5)
diagnostics['workflows_accessible'] = workflow_response.status_code == 200
# Check if credentials endpoint is accessible
cred_response = n8n_api.client.get(f"{n8n_api.base_url}/rest/credentials", timeout=5)
diagnostics['credentials_accessible'] = cred_response.status_code == 200
# Try a simple export operation
export_response = n8n_api.client.post(
f"{n8n_api.base_url}/rest/export",
json={"includeCredentials": False, "includeExecutions": False},
timeout=5
)
diagnostics['export_accessible'] = export_response.status_code == 200
except Exception as e:
diagnostics['additional_error'] = str(e)
except Exception as e:
diagnostics['api_error'] = str(e)
return HttpResponse(json.dumps({
'status': 1,
'diagnostics': diagnostics
}))
return HttpResponse(json.dumps({
'status': 0,
'error_message': 'Invalid request method'
}))
except Exception as e:
error_msg = f"Error in diagnose_n8n_api: {str(e)}"
logging.writeToFile(error_msg)
return HttpResponse(json.dumps({
'status': 0,
'error_message': error_msg
}))

View File

@@ -440,6 +440,112 @@ app.controller('ListDockersitecontainer', function ($scope, $http) {
});
};
// Diagnostic function to troubleshoot n8n connectivity issues
$scope.diagnoseN8n = function(container) {
$scope.cyberpanelLoading = false;
$('#cyberpanelLoading').show();
var url = "/websites/n8n/diagnose";
var data = {
'container_id': container.id
};
var config = {
headers: {
'X-CSRFToken': getCookie('csrftoken')
}
};
$http.post(url, data, config)
.then(function(response) {
$scope.cyberpanelLoading = true;
$('#cyberpanelLoading').hide();
if (response.data.status === 1) {
var diagnostics = response.data.diagnostics;
// Initialize diagnostic results if not exists
if (!container.diagnosticResults) {
container.diagnosticResults = {};
}
// Store diagnostic results
container.diagnosticResults = diagnostics;
container.showDiagnostics = true;
// Show summary notification
var summaryMessage = "";
if (!diagnostics.container_exists) {
summaryMessage = "Container not found";
} else if (!diagnostics.container_running) {
summaryMessage = "Container exists but is not running. Current status: " + diagnostics.container_status;
} else if (!diagnostics.n8n_port_found) {
summaryMessage = "Container is running but n8n port (5678) is not mapped. Available ports: " +
JSON.stringify(diagnostics.port_mappings);
} else if (!diagnostics.api_accessible) {
summaryMessage = "n8n port found but API is not accessible. Check if n8n is running inside the container.";
if (diagnostics.api_error) {
summaryMessage += " Error: " + diagnostics.api_error;
}
} else {
summaryMessage = "n8n API is accessible. Endpoints status:";
if (diagnostics.workflows_accessible) {
summaryMessage += " Workflows: OK";
} else {
summaryMessage += " Workflows: Failed";
}
if (diagnostics.credentials_accessible) {
summaryMessage += " | Credentials: OK";
} else {
summaryMessage += " | Credentials: Failed";
}
if (diagnostics.export_accessible) {
summaryMessage += " | Export: OK";
} else {
summaryMessage += " | Export: Failed";
}
}
new PNotify({
title: 'Diagnostic Results',
text: summaryMessage,
type: 'info',
hide: false
});
} else {
new PNotify({
title: 'Diagnostic Failed',
text: response.data.error_message || 'Failed to diagnose n8n container',
type: 'error'
});
}
})
.catch(function(error) {
$scope.cyberpanelLoading = true;
$('#cyberpanelLoading').hide();
var errorMessage = 'Connection disrupted, refresh the page.';
if (error.data && error.data.error_message) {
errorMessage = error.data.error_message;
} else if (error.statusText) {
errorMessage = 'Server error: ' + error.statusText;
}
new PNotify({
title: 'Diagnostic Failed',
text: errorMessage,
type: 'error'
});
console.error('Error during diagnosis:', error);
});
};
// Restore from a backup
$scope.restoreFromBackup = function(container) {
// Check if a file has been selected

View File

@@ -607,9 +607,14 @@
<button class="btn btn-danger btn-sm" ng-click="handleAction('stop', web)" ng-if="web.status === 'running'" title="Stop Container">
<i class="fa fa-stop"></i> Stop
</button>
<button class="btn btn-primary btn-sm" ng-click="openSettings(web)" title="Container Settings">
<i class="fa fa-cog"></i> Settings
</button>
<div class="nav navbar-right btn-group">
<a href="javascript:void(0)" class="btn btn-sm btn-primary" ng-click="openSettings(container)">
<i class="fas fa-cog"></i> Settings
</a>
<a href="javascript:void(0)" class="btn btn-sm btn-info" ng-click="diagnoseN8n(container)">
<i class="fas fa-stethoscope"></i> Run Diagnostics
</a>
</div>
</div>
</h2>
<p class="text-muted">
@@ -705,6 +710,14 @@
<button class="quick-action-btn btn-danger" ng-click="handleAction('stop', web)" ng-if="web.status === 'running'">
<i class="fa fa-stop"></i> Stop
</button>
<div class="nav navbar-right btn-group">
<a href="javascript:void(0)" class="btn btn-sm btn-primary" ng-click="openSettings(container)">
<i class="fas fa-cog"></i> Settings
</a>
<a href="javascript:void(0)" class="btn btn-sm btn-info" ng-click="diagnoseN8n(container)">
<i class="fas fa-stethoscope"></i> Run Diagnostics
</a>
</div>
<button class="quick-action-btn btn-primary" ng-click="openSettings(web)">
<i class="fa fa-cog"></i> Settings
</button>
@@ -1863,6 +1876,127 @@
</div>
</div>
<div class="tab-pane" id="n8nDiagnostics">
<div class="row">
<div class="col-md-12">
<div class="card">
<div class="card-header">
<h5 class="card-title">N8n API Diagnostics</h5>
<div class="card-tools">
<button class="btn btn-primary" ng-click="diagnoseN8n(container)">
<i class="fas fa-stethoscope"></i> Run Diagnostics
</button>
</div>
</div>
<div class="card-body">
<div ng-if="container.showDiagnostics">
<div class="alert" ng-class="{'alert-success': container.diagnosticResults.api_accessible, 'alert-danger': !container.diagnosticResults.api_accessible}">
<h5><i class="icon" ng-class="{'fas fa-check': container.diagnosticResults.api_accessible, 'fas fa-ban': !container.diagnosticResults.api_accessible}"></i>
N8n API Status: <span ng-if="container.diagnosticResults.api_accessible">Accessible</span><span ng-if="!container.diagnosticResults.api_accessible">Not Accessible</span>
</h5>
</div>
<div class="row">
<div class="col-md-6">
<div class="card">
<div class="card-header">
<h5 class="card-title">Container Status</h5>
</div>
<div class="card-body">
<table class="table table-bordered">
<tr>
<th>Container Exists</th>
<td><i class="fas" ng-class="{'fa-check text-success': container.diagnosticResults.container_exists, 'fa-times text-danger': !container.diagnosticResults.container_exists}"></i></td>
</tr>
<tr>
<th>Container Status</th>
<td>{{container.diagnosticResults.container_status}}</td>
</tr>
<tr>
<th>Container Running</th>
<td><i class="fas" ng-class="{'fa-check text-success': container.diagnosticResults.container_running, 'fa-times text-danger': !container.diagnosticResults.container_running}"></i></td>
</tr>
<tr>
<th>Container Name</th>
<td>{{container.diagnosticResults.container_names}}</td>
</tr>
</table>
</div>
</div>
</div>
<div class="col-md-6">
<div class="card">
<div class="card-header">
<h5 class="card-title">N8n API Connectivity</h5>
</div>
<div class="card-body">
<table class="table table-bordered">
<tr>
<th>N8n Port Found</th>
<td><i class="fas" ng-class="{'fa-check text-success': container.diagnosticResults.n8n_port_found, 'fa-times text-danger': !container.diagnosticResults.n8n_port_found}"></i></td>
</tr>
<tr ng-if="container.diagnosticResults.n8n_port_found">
<th>N8n Port</th>
<td>{{container.diagnosticResults.n8n_port}}</td>
</tr>
<tr>
<th>API Accessible</th>
<td><i class="fas" ng-class="{'fa-check text-success': container.diagnosticResults.api_accessible, 'fa-times text-danger': !container.diagnosticResults.api_accessible}"></i></td>
</tr>
<tr ng-if="container.diagnosticResults.api_accessible">
<th>Workflows API</th>
<td><i class="fas" ng-class="{'fa-check text-success': container.diagnosticResults.workflows_accessible, 'fa-times text-danger': !container.diagnosticResults.workflows_accessible}"></i></td>
</tr>
<tr ng-if="container.diagnosticResults.api_accessible">
<th>Credentials API</th>
<td><i class="fas" ng-class="{'fa-check text-success': container.diagnosticResults.credentials_accessible, 'fa-times text-danger': !container.diagnosticResults.credentials_accessible}"></i></td>
</tr>
<tr ng-if="container.diagnosticResults.api_accessible">
<th>Export API</th>
<td><i class="fas" ng-class="{'fa-check text-success': container.diagnosticResults.export_accessible, 'fa-times text-danger': !container.diagnosticResults.export_accessible}"></i></td>
</tr>
</table>
</div>
</div>
</div>
</div>
<div class="row" ng-if="container.diagnosticResults.port_mappings">
<div class="col-md-12">
<div class="card">
<div class="card-header">
<h5 class="card-title">Port Mappings</h5>
</div>
<div class="card-body">
<pre>{{container.diagnosticResults.port_mappings | json}}</pre>
</div>
</div>
</div>
</div>
<div class="row" ng-if="container.diagnosticResults.api_error">
<div class="col-md-12">
<div class="card">
<div class="card-header">
<h5 class="card-title">API Error</h5>
</div>
<div class="card-body">
<pre class="text-danger">{{container.diagnosticResults.api_error}}</pre>
</div>
</div>
</div>
</div>
</div>
<div ng-if="!container.showDiagnostics">
<p>Click "Run Diagnostics" to check N8n API connectivity.</p>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</div>

View File

@@ -202,8 +202,9 @@ urlpatterns = [
path('<domain>', views.domain, name='domain'),
# N8N API endpoints
path('n8n/get_workflows', views.n8n_api.get_n8n_workflows, name='get_n8n_workflows'),
path('n8n/toggle_workflow', views.n8n_api.toggle_workflow, name='toggle_workflow'),
path('n8n/create_backup', views.n8n_api.create_n8n_backup, name='create_n8n_backup'),
path('n8n/restore_backup', views.n8n_api.restore_n8n_backup, name='restore_n8n_backup'),
path('websites/n8n/get_workflows', views.n8n_api.get_n8n_workflows, name='get_n8n_workflows'),
path('websites/n8n/toggle_workflow', views.n8n_api.toggle_workflow, name='toggle_workflow'),
path('websites/n8n/create_backup', views.n8n_api.create_n8n_backup, name='create_n8n_backup'),
path('websites/n8n/restore_backup', views.n8n_api.restore_n8n_backup, name='restore_n8n_backup'),
path('websites/n8n/diagnose', views.n8n_api.diagnose_n8n_api, name='diagnose_n8n_api'),
]