Compare commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
aa5383d826 |
+1
-1
@@ -1,6 +1,6 @@
|
||||
{
|
||||
"name": "proxmox-spice-client",
|
||||
"version": "1.0.6",
|
||||
"version": "1.0.7",
|
||||
"description": "VDI SPICE Client f\u00fcr Proxmox",
|
||||
"main": "src/main.js",
|
||||
"scripts": {
|
||||
|
||||
+12
-5
@@ -266,12 +266,19 @@ ipcMain.handle('settings:setAutoConnect', (_e, target) => {
|
||||
function buildVVFile(params, vmid, client) {
|
||||
const lines = ['[virt-viewer]', `type=${params.type || 'spice'}`];
|
||||
|
||||
const host = params.proxy || params.host || client.host.split(':')[0];
|
||||
// params.host = the real SPICE target (often a cluster-internal node
|
||||
// address) and params.proxy = a ready-made "http://<reachable-host>:3128"
|
||||
// URL that virt-viewer tunnels through via HTTP CONNECT. They are two
|
||||
// different .vv fields — writing params.proxy into host= (as before)
|
||||
// handed virt-viewer a full URL as a hostname and never opened the
|
||||
// tunnel, so it tried (and failed) to dial the internal address directly.
|
||||
const host = params.host || client.host.split(':')[0];
|
||||
lines.push(`host=${host}`);
|
||||
|
||||
if (params['tls-port']) lines.push(`tls-port=${params['tls-port']}`);
|
||||
if (params.port) lines.push(`port=${params.port}`);
|
||||
if (params.password) lines.push(`password=${params.password}`);
|
||||
if (params.proxy) lines.push(`proxy=${params.proxy}`);
|
||||
|
||||
if (params.ca) {
|
||||
const caPath = path.join(os.tmpdir(), 'proxmox-spice-ca.pem');
|
||||
@@ -291,11 +298,11 @@ function buildVVFile(params, vmid, client) {
|
||||
const vvPath = path.join(os.tmpdir(), `spice-${vmid}-${Date.now()}.vv`);
|
||||
fs.writeFileSync(vvPath, lines.join('\n') + '\n', { mode: 0o600 });
|
||||
|
||||
// Surfaced to the renderer so a failed connection (e.g. Proxmox handing out
|
||||
// a cluster-internal node address the client can't route to) is diagnosable
|
||||
// without having to catch the .vv file before remote-viewer deletes it.
|
||||
// Surfaced to the renderer so a failed connection is diagnosable without
|
||||
// having to catch the .vv file before remote-viewer deletes it.
|
||||
const port = params['tls-port'] || params.port || '?';
|
||||
return { vvPath, target: `${host}:${port}` };
|
||||
const target = params.proxy ? `${host}:${port} via ${params.proxy}` : `${host}:${port}`;
|
||||
return { vvPath, target };
|
||||
}
|
||||
|
||||
// VirtViewer's Windows installer names its folder after the bundled version
|
||||
|
||||
Reference in New Issue
Block a user