Make the user required in the connection

.github/workflows/pr.yaml → .github/workflows/tests.yaml

@@ -1,11 +1,11 @@
-name: PR checks
+name: Tests
 on:
   pull_request:
     branches: [main]
   push:
     branches: [main]
 jobs:
-  check_pr:
+  functional_tests:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2

src/server.ts

@@ -1,13 +1,15 @@
+import os from 'os'
 import path from 'path'
 import { Host } from './ansible/types.js'
 import * as blocks from './blocks/index.js'
 import { Server, ServerConfig } from './types.js'
 
 export function server(config: ServerConfig): Server {
+  const user = os.userInfo().username
   let connection = config.connection
   if (!connection) {
-    if (config.name === 'localhost') connection = { type: 'local', user: process.env.USER }
-    else connection = { type: 'ssh', address: config.name }
+    if (config.name === 'localhost') connection = { type: 'local', user }
+    else connection = { type: 'ssh', address: config.name, user }
   }
   const hosty_dir = config.hosty_dir || '/srv/hosty'
   const backups_dir = path.join(hosty_dir, 'backups')

src/types.ts

@@ -4,23 +4,23 @@ import { ChildProcess, SpawnOptions } from 'child_process'
 
 export type LocalConnection = {
   type: 'local'
-  user?: string
+  user: string
   password?: string
 }
 
 export type SshConnection = {
   type: 'ssh'
   address: string
   port?: number
-  user?: string
+  user: string
   password?: string
   private_key_path?: string
 }
 
 export type DockerConnection = {
   type: 'docker'
   container: string
-  user?: string
+  user: string
   password?: string
 }