Skip to content

Commit 50116e7

Browse files
authored
Merge pull request #10 from fabrix-app/v1.5
[fix] LTS, 1.5
2 parents e6f7a9f + 4ebbe29 commit 50116e7

File tree

6 files changed

+517
-158
lines changed

6 files changed

+517
-158
lines changed

lib/ExpressSpool.ts

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -48,9 +48,13 @@ export class ExpressSpool extends ServerSpool {
4848
)
4949
)
5050
}
51+
5152
return Promise.all([
5253
Validator.validateWebConfig(this.app.config.get('web'))
5354
])
55+
.catch(err => {
56+
return Promise.reject(err)
57+
})
5458
}
5559

5660
configure () {
@@ -75,6 +79,9 @@ export class ExpressSpool extends ServerSpool {
7579
this.app.emit('webserver:http:ready', Server.nativeServer)
7680
return
7781
})
82+
.catch(err => {
83+
return Promise.reject(err)
84+
})
7885
}
7986

8087
/**

lib/config/web.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,8 @@ export const web = {
2121
/**
2222
* Init method, can be used to customize express instance
2323
*/
24-
init: (fabrixApp: FabrixApp, expressApp: Express) => {
24+
init: (app: FabrixApp, expressApp: Express) => {
25+
return true
2526
},
2627

2728
/**

lib/schemas/webConfig.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,4 +25,4 @@ export const webConfig = joi.object().keys({
2525
path: joi.string().required()
2626
}),
2727
server: joi.string()
28-
})
28+
}).unknown()

0 commit comments

Comments
 (0)