github/hedgedoc
1
0
Fork 0
mirror of https://github.com/hedgedoc/hedgedoc.git synced 2024-10-19 22:00:15 -04:00
Code Issues Projects Releases Packages Wiki Activity

AppConfig: Move default for rendererOrigin to config file

Browse source 
As we only use rendererOrigin in the frontend config service, where domain will be used if it is not defined, it makes more sense to move this default behavior to the app config directly. That makes it easier to understand what this variable contains and that it defaults to domain.

Signed-off-by: Philip Molares <philip.molares@udo.edu>
This commit is contained in:
Philip Molares 2021-04-25 15:20:51 +02:00
parent 0c2900d1c0
commit 8bf5aa77e3
2 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
src/config/app.config.ts
View file

@ -20,7 +20,10 @@ export interface AppConfig {
const schema = Joi.object({ const schema = Joi.object({
domain: Joi.string().label('HD_DOMAIN'), domain: Joi.string().label('HD_DOMAIN'),
rendererOrigin: Joi.string().optional().label('HD_RENDERER_ORIGIN'), rendererOrigin: Joi.string()
.default(Joi.ref('domain'))
.optional()
.label('HD_RENDERER_ORIGIN'),
port: Joi.number().default(3000).optional().label('PORT'), port: Joi.number().default(3000).optional().label('PORT'),
loglevel: Joi.string() loglevel: Joi.string()
.valid(...Object.values(Loglevel)) .valid(...Object.values(Loglevel))

4
src/frontend-config/frontend-config.service.ts
View file

@ -133,9 +133,7 @@ export class FrontendConfigService {
private getIframeCommunication(): IframeCommunicationDto { private getIframeCommunication(): IframeCommunicationDto {
return { return {
editorOrigin: new URL(this.appConfig.domain), editorOrigin: new URL(this.appConfig.domain),
rendererOrigin: this.appConfig.rendererOrigin rendererOrigin: new URL(this.appConfig.rendererOrigin),
? new URL(this.appConfig.rendererOrigin)
: new URL(this.appConfig.domain),
}; };
} }