Merge pull request #465 from Skgland/330-followup

#330 followup

docs/configuration-config-file.md

@@ -140,7 +140,7 @@ these are rarely used for various reasons.
 
 | variables | example values | description |
 | --------- | ------ | ----------- |
-| `ldap` | `{providerName: ..., url: ..., bindDn: ..., bindCredentials: ..., searchBase: ..., searchFilter: ..., searchAttributes: ..., usernameField: ..., useridField: ..., tlsca: ...}` | An object detailing the LDAP connection. Refer to the [LDAP-AD guide](guides/auth/ldap-AD.md) for more details! |
+| `ldap` | `{providerName: ..., url: ..., bindDn: ..., bindCredentials: ..., searchBase: ..., searchFilter: ..., searchAttributes: ..., usernameField: ..., useridField: ..., starttls: ..., tlsca: ...}` | An object detailing the LDAP connection. Refer to the [LDAP-AD guide](guides/auth/ldap-AD.md) for more details! |
 
 ### OAuth2 Login
 

src/lib/config/interfaces.ts

@@ -127,6 +127,7 @@ export interface Config {
     usernameField: string;
     useridField: string;
     tlsca: string;
+    starttls?: boolean;
     tlsOptions: {
       ca: string[];
     };

src/lib/models/index.ts

@@ -35,7 +35,7 @@ const umzug = new Umzug({
   },
   // Required wrapper function required to prevent winstion issue
   // https://github.com/winstonjs/winston/issues/1577
-  logging: message => {
+  logging: (message): void => {
     logger.info(message)
   },
   storage: 'sequelize',
@@ -44,7 +44,7 @@ const umzug = new Umzug({
   }
 })
 
-export async function runMigrations(): Promise<void> {
+export async function runMigrations (): Promise<void> {
   // checks migrations and run them if they are not already applied
   // exit in case of unsuccessful migrations
   await umzug.up().catch(error => {