diff --git a/site-modules/profile/manifests/phabricator.pp b/site-modules/profile/manifests/phabricator.pp
index a71fda97..c133463c 100644
--- a/site-modules/profile/manifests/phabricator.pp
+++ b/site-modules/profile/manifests/phabricator.pp
@@ -1,314 +1,317 @@
 # Setup an instance of phabricator
 class profile::phabricator {
   $phabricator_basepath = lookup('phabricator::basepath')
   $phabricator_user = lookup('phabricator::user')
   $phabricator_vcs_user = lookup('phabricator::vcs_user')
 
   $phabricator_db_root_password = lookup('phabricator::mysql::root_password')
   $phabricator_db_basename = lookup('phabricator::mysql::database_prefix')
   $phabricator_db_user = lookup('phabricator::mysql::username')
   $phabricator_db_password = lookup('phabricator::mysql::password')
 
   $phabricator_db_max_allowed_packet = lookup('phabricator::mysql::conf::max_allowed_packet')
   $phabricator_db_sql_mode = lookup('phabricator::mysql::conf::sql_mode')
   $phabricator_db_ft_stopword_file = lookup('phabricator::mysql::conf::ft_stopword_file')
   $phabricator_db_ft_min_word_len = lookup('phabricator::mysql::conf::ft_min_word_len')
   $phabricator_db_ft_boolean_syntax = lookup('phabricator::mysql::conf::ft_boolean_syntax')
   $phabricator_db_innodb_buffer_pool_size = lookup('phabricator::mysql::conf::innodb_buffer_pool_size')
   $phabricator_db_innodb_file_per_table = lookup('phabricator::mysql::conf::innodb_file_per_table')
   $phabricator_db_innodb_flush_method = lookup('phabricator::mysql::conf::innodb_flush_method')
   $phabricator_db_innodb_log_file_size = lookup('phabricator::mysql::conf::innodb_log_file_size')
 
   $phabricator_fpm_listen = lookup('phabricator::php::fpm_listen')
   $phabricator_max_size = lookup('phabricator::php::max_file_size')
   $phabricator_opcache_validate_timestamps = lookup('phabricator::php::opcache_validate_timestamps')
 
   $phabricator_notification_listen = lookup('phabricator::notification::listen')
   $phabricator_notification_client_host = lookup('phabricator::notification::client_host')
   $phabricator_notification_client_port = lookup('phabricator::notification::client_port')
 
   $phabricator_vhost_name = lookup('phabricator::vhost::name')
   $phabricator_vhost_docroot = lookup('phabricator::vhost::docroot')
   $phabricator_vhost_basic_auth_file = "${phabricator_basepath}/http_auth"
   $phabricator_vhost_basic_auth_content = lookup('phabricator::vhost::basic_auth_content')
   $phabricator_vhost_ssl_protocol = lookup('phabricator::vhost::ssl_protocol')
   $phabricator_vhost_ssl_honorcipherorder = lookup('phabricator::vhost::ssl_honorcipherorder')
   $phabricator_vhost_ssl_cipher = lookup('phabricator::vhost::ssl_cipher')
   $phabricator_vhost_hsts_header = lookup('phabricator::vhost::hsts_header')
 
   $homedirs = {
     $phabricator_user     => $phabricator_basepath,
     $phabricator_vcs_user => "${phabricator_basepath}/vcshome",
   }
 
   $homedir_modes = {
     $phabricator_user     => '0644',
     $phabricator_vcs_user => '0640',
   }
 
   each([$phabricator_user, $phabricator_vcs_user]) |$name| {
     user {$name:
       ensure => present,
       system => true,
       shell  => '/bin/bash',
       home   => $homedirs[$name],
     }
 
     file {$homedirs[$name]:
       ensure => directory,
       owner  => $name,
       group  => $name,
       mode   => $homedir_modes[$name],
     }
   }
 
   ::sudo::conf {'phabricator-ssh':
     ensure  => present,
     content => "${phabricator_vcs_user} ALL=(${phabricator_user}) SETENV: NOPASSWD: /usr/bin/git-upload-pack, /usr/bin/git-receive-pack, /usr/bin/hg",
   }
 
   ::sudo::conf {'phabricator-http':
     ensure  => present,
     content => "www-data ALL=(${phabricator_user}) SETENV: NOPASSWD: /usr/bin/git-http-backend, /usr/bin/hg",
     require => File['/usr/bin/git-http-backend'],
   }
 
   file {'/usr/bin/git-http-backend':
     ensure => link,
     target => '/usr/lib/git-core/git-http-backend',
   }
 
   $phabricator_ssh_hook = '/usr/bin/phabricator-ssh-hook.sh'
   $phabricator_ssh_config = '/etc/ssh/ssh_config.phabricator'
 
   file {$phabricator_ssh_hook:
     ensure  => present,
     owner   => 'root',
     group   => 'root',
     mode    => '0755',
     content => template('profile/phabricator/phabricator-ssh-hook.sh.erb'),
   }
 
   file {$phabricator_ssh_config:
     ensure  => present,
     owner   => 'root',
     group   => 'root',
     mode    => '0600',
     content => template('profile/phabricator/sshd_config.phabricator.erb'),
     require => File[$phabricator_ssh_hook],
   }
 
   ::systemd::unit_file {'phabricator-sshd.service':
     ensure  => present,
     content => template('profile/phabricator/phabricator-sshd.service.erb'),
     require => File[$phabricator_ssh_config],
   } ~> service {'phabricator-sshd':
     ensure  => 'running',
     enable  => true,
     require => [
       File['/etc/systemd/system/phabricator-sshd.service'],
     ],
   }
 
   include ::mysql::client
 
   class {'::mysql::server':
     root_password    => $phabricator_db_root_password,
     override_options => {
       mysqld => {
         max_allowed_packet      => $phabricator_db_max_allowed_packet,
         sql_mode                => $phabricator_db_sql_mode,
         ft_stopword_file        => $phabricator_db_ft_stopword_file,
         ft_min_word_len         => $phabricator_db_ft_min_word_len,
         ft_boolean_syntax       => $phabricator_db_ft_boolean_syntax,
         innodb_buffer_pool_size => $phabricator_db_innodb_buffer_pool_size,
         innodb_file_per_table   => $phabricator_db_innodb_file_per_table,
         innodb_flush_method     => $phabricator_db_innodb_flush_method,
         innodb_log_file_size    => $phabricator_db_innodb_log_file_size,
       }
     }
   }
 
   $mysql_username = "${phabricator_db_user}@localhost"
   $mysql_tables = "${phabricator_db_basename}_%.*"
 
   mysql_user {$mysql_username:
     ensure        => present,
     password_hash => mysql_password($phabricator_db_password),
   }
 
   mysql_grant {"${mysql_username}/${mysql_tables}":
     user       => $mysql_username,
     table      => $mysql_tables,
     privileges => ['ALL'],
     require    => Mysql_user[$mysql_username],
   }
 
   include ::profile::php
 
   ::php::fpm::pool {'phabricator':
     listen          => $phabricator_fpm_listen,
     user            => 'www-data',
     php_admin_value => {
       post_max_size                 => $phabricator_max_size,
       upload_max_filesize           => $phabricator_max_size,
       'opcache.validate_timestamps' => $phabricator_opcache_validate_timestamps,
     },
   }
 
   ::php::extension {[
     'apcu',
     'mailparse',
   ]:
     provider       => 'apt',
     package_prefix => 'php-',
   }
 
   ::php::extension {[
     'curl',
     'gd',
   ]:
     provider => 'apt',
   }
 
   include ::profile::ssl
   include ::profile::apache::common
   include ::apache::mod::proxy
   include ::profile::apache::mod_proxy_fcgi
 
   ::apache::mod {'proxy_wstunnel':}
 
   ::apache::vhost {"${phabricator_vhost_name}_non-ssl":
     servername      => $phabricator_vhost_name,
     port            => '80',
     docroot         => $phabricator_vhost_docroot,
     docroot_owner   => $phabricator_user,
     docroot_group   => $phabricator_user,
     redirect_status => 'permanent',
     redirect_dest   => "https://${phabricator_vhost_name}/",
   }
 
   $ssl_cert_name = 'star_softwareheritage_org'
   $ssl_cert = $::profile::ssl::certificate_paths[$ssl_cert_name]
   $ssl_chain   = $::profile::ssl::chain_paths[$ssl_cert_name]
   $ssl_key  = $::profile::ssl::private_key_paths[$ssl_cert_name]
 
   ::apache::vhost {"${phabricator_vhost_name}_ssl":
     servername           => $phabricator_vhost_name,
     port                 => '443',
     ssl                  => true,
     ssl_protocol         => $phabricator_vhost_ssl_protocol,
     ssl_honorcipherorder => $phabricator_vhost_ssl_honorcipherorder,
     ssl_cipher           => $phabricator_vhost_ssl_cipher,
     ssl_cert             => $ssl_cert,
     ssl_chain            => $ssl_chain,
     ssl_key              => $ssl_key,
     headers              => [$phabricator_vhost_hsts_header],
     docroot              => $phabricator_vhost_docroot,
     docroot_owner        => $phabricator_user,
     docroot_group        => $phabricator_user,
     rewrites             => [
       { rewrite_rule => '^/rsrc/(.*) - [L,QSA]' },
       { rewrite_rule => '^/favicon.ico - [L,QSA]' },
       { rewrite_rule => "^/ws/(.*)$ ws://${phabricator_notification_listen}/\$1 [L,P]" },
       { rewrite_rule => "^(.*)$ fcgi://${phabricator_fpm_listen}${phabricator_vhost_docroot}/index.php?__path__=\$1 [B,L,P,QSA]" },
     ],
     setenvif             => [
       "Authorization \"(.*)\" HTTP_AUTHORIZATION=\$1",
     ],
     require              => [
         File[$ssl_cert],
         File[$ssl_chain],
         File[$ssl_key],
     ],
   }
 
   file {$phabricator_vhost_basic_auth_file:
     ensure  => absent,
   }
 
   # Uses:
     # $phabricator_basepath
     # $phabricator_user
   ::systemd::unit_file {'phabricator-phd.service':
     ensure  => present,
     content => template('profile/phabricator/phabricator-phd.service.erb'),
   } ~> service {'phabricator-phd':
     ensure => 'running',
     enable => true,
   }
 
   # Uses:
     # $phabricator_basepath
     # $phabricator_user
     # $phabricator_notification_*
   ::systemd::unit_file {'phabricator-aphlict.service':
     ensure  => present,
     content => template('profile/phabricator/phabricator-aphlict.service.erb'),
   } ~> service {'phabricator-aphlict':
       ensure => 'running',
       enable => true,
   }
 
   package {'python-pygments':
     ensure => installed,
   }
 
   $icinga_checks_file = '/etc/icinga2/conf.d/exported-checks.conf'
 
   @@::icinga2::object::service {"phabricator http redirect on ${::fqdn}":
     service_name  => 'phabricator http redirect',
     import        => ['generic-service'],
     host_name     => $::fqdn,
     check_command => 'http',
     vars          => {
       http_address => $phabricator_vhost_name,
       http_vhost   => $phabricator_vhost_name,
       http_uri     => '/',
     },
     target        => $icinga_checks_file,
     tag           => 'icinga2::exported',
   }
 
   @@::icinga2::object::service {"phabricator https on ${::fqdn}":
     service_name  => 'phabricator',
     import        => ['generic-service'],
     host_name     => $::fqdn,
     check_command => 'http',
     vars          => {
       http_address    => $phabricator_vhost_name,
       http_vhost      => $phabricator_vhost_name,
       http_ssl        => true,
       http_sni        => true,
       http_uri        => '/',
       http_onredirect => sticky
     },
     target        => $icinga_checks_file,
     tag           => 'icinga2::exported',
   }
 
   @@::icinga2::object::service {"phabricator https certificate ${::fqdn}":
     service_name  => 'phabricator https certificate',
     import        => ['generic-service'],
     host_name     => $::fqdn,
     check_command => 'http',
     vars          => {
       http_address     => $phabricator_vhost_name,
       http_vhost       => $phabricator_vhost_name,
       http_ssl         => true,
       http_sni         => true,
       http_certificate => 60,
     },
     target        => $icinga_checks_file,
     tag           => 'icinga2::exported',
   }
 
   each($::ssh) |$algo, $data| {
-    if $algo == 'ecdsa' { $algo = 'ecdsa-sha2-nistp256' }
-    @@sshkey {"phabricator-${::fqdn}-${algo}":
+    $real_algo = $algo ? {
+      'ecdsa' => 'ecdsa-sha2-nistp256',
+      default => $algo,
+    }
+    @@sshkey {"phabricator-${::fqdn}-${real_algo}":
       host_aliases => [$phabricator_vhost_name],
-      type         => $algo,
+      type         => $real_algo,
       key          => $data['key'],
     }
   }
 }
diff --git a/site-modules/profile/manifests/ssh/server.pp b/site-modules/profile/manifests/ssh/server.pp
index b8279326..72cdf569 100644
--- a/site-modules/profile/manifests/ssh/server.pp
+++ b/site-modules/profile/manifests/ssh/server.pp
@@ -1,58 +1,61 @@
 # Configure the SSH server
 
 class profile::ssh::server {
   $sshd_port = lookup('ssh::port')
   $sshd_permitrootlogin = lookup('ssh::permitrootlogin')
 
   class { '::ssh::server':
     storeconfigs_enabled => false,
     options              => {
       'PermitRootLogin' => $sshd_permitrootlogin,
       'Port'            => $sshd_port,
     },
   }
 
   $users = lookup('users', Hash, 'deep')
 
   each($users) |$name, $data| {
     if $name == 'root' {
       $home = '/root'
     } else {
       $home = "/home/${name}"
     }
 
     file { "${home}/.ssh":
       ensure  => directory,
       owner   => $name,
       group   => $name,
       mode    => '0600',
       require => [
         User[$name],
         File[$home],
       ],
     }
 
     if $data['authorized_keys'] {
       each($data['authorized_keys']) |$nick, $key| {
         ssh_authorized_key { "${name} ${nick}":
           ensure  => 'present',
           user    => $name,
           key     => $key['key'],
           type    => $key['type'],
           require => File["${home}/.ssh"],
         }
       }
     }
   }
 
   each($::ssh) |$algo, $data| {
-    if $algo == 'ecdsa' { $algo = 'ecdsa-sha2-nistp256' }
-    @@sshkey {"ssh-${::fqdn}-${algo}":
+    $real_algo = $algo ? {
+      'ecdsa' => 'ecdsa-sha2-nistp256',
+      default => $algo,
+    }
+    @@sshkey {"ssh-${::fqdn}-${real_algo}":
       host_aliases => unique(values($::swh_hostname)),
-      type         => $algo,
+      type         => $real_algo,
       key          => $data['key'],
     }
   }
 
   Sshkey <<| |>>
 }