fix: reverse proxy url fix

ui/src/app/settings/settings.service.ts

@@ -8,6 +8,7 @@ import {SettingModel} from '../shared/model/setting.model';
 import {LocalStorageService} from 'angular-2-local-storage';
 
 const DEFAULT_LANG = 'en';
+const DEFAULT_REVERSE_PROXY_ACTIVE = 'false';
 
 @Injectable({
   providedIn: 'root'
@@ -20,9 +21,11 @@ export class SettingsService {
   load(languages: Array<string>): Observable<SettingModel[]> {
     this.language = languages;
     const isDarkConfig = window.matchMedia && window.matchMedia('(prefers-color-scheme: dark)').matches;
-    let activeThemeValue: string = isDarkConfig ? 'dark' : 'default';
 
+    let activeThemeValue: string = isDarkConfig ? 'dark' : 'default';
     let activeLanguageValue: string = DEFAULT_LANG;
+    let reverseProxyActiveValue: string = DEFAULT_REVERSE_PROXY_ACTIVE;
+
     if (navigator?.language) {
       activeLanguageValue = navigator.language.split('-')[0];
     }
@@ -40,10 +43,15 @@ export class SettingsService {
       }
     }
 
+    if (this.localStorageService.get('reverseProxyActiveValue')) {
+      reverseProxyActiveValue = this.localStorageService.get('reverseProxyActiveValue');
+    }
+
     const settings: SettingModel[] = [
       {name: 'language-active', value: activeLanguageValue},
       {name: 'theme-active', value: activeThemeValue},
-      {name: 'results-per-page', value: '20'}
+      {name: 'results-per-page', value: '20'},
+      {name: 'reverse-proxy-fix-active', value: reverseProxyActiveValue}
     ];
     return of(settings).pipe(tap(sett => this.store.dispatch(loaded({settings: sett}))));
   }
@@ -55,6 +63,9 @@ export class SettingsService {
     if (setting.name === 'language-active') {
       this.localStorageService.set('languageActiveValue', setting.value);
     }
+    if (setting.name === 'reverse-proxy-fix-active') {
+      this.localStorageService.set('reverseProxyActiveValue', setting.value);
+    }
     return from(new Promise<void>(resolve => resolve()));
   }
 

ui/src/app/settings/settings/settings.component.html

@@ -3,11 +3,11 @@
     <h3 class="modal-title">{{ 'settings.title' | translate }}</h3>
     <div class="modal-body">
       <clr-select-container *ngIf="languageActive && languages?.length > 1">
-        <label class="clr-col-md-3">{{ 'settings.language' | translate }}</label>
+        <label class="clr-col-md-4">{{ 'settings.language' | translate }}</label>
         <select
           clrSelect
           name="language"
-          class="clr-col-md-9"
+          class="clr-col-md-8"
           [value]="languageActive.value"
           (change)="languageActiveSettingOnChange($event.target.value)"
         >
@@ -18,11 +18,11 @@ <h3 class="modal-title">{{ 'settings.title' | translate }}</h3>
         <clr-control-helper>{{ 'settings.languageDescription' | translate }}</clr-control-helper>
       </clr-select-container>
       <clr-select-container *ngIf="themeActive">
-        <label class="clr-col-md-3">{{ 'settings.theme' | translate }}</label>
+        <label class="clr-col-md-4">{{ 'settings.theme' | translate }}</label>
         <select
           clrSelect
           name="theme"
-          class="clr-col-md-9"
+          class="clr-col-md-8"
           [value]="themeActive.value"
           (change)="themeActiveSettingOnChange($event.target.value)"
         >
@@ -33,11 +33,11 @@ <h3 class="modal-title">{{ 'settings.title' | translate }}</h3>
       </clr-select-container>
 
       <clr-select-container *ngIf="resultsPerPage">
-        <label class="clr-col-md-3">{{ 'settings.results' | translate }}</label>
+        <label class="clr-col-md-4">{{ 'settings.results' | translate }}</label>
         <select
           clrSelect
           name="theme"
-          class="clr-col-md-9"
+          class="clr-col-md-8"
           [value]="resultsPerPage.value"
           (change)="resultPerPageSettingOnChange($event.target.value)"
         >
@@ -50,6 +50,23 @@ <h3 class="modal-title">{{ 'settings.title' | translate }}</h3>
           {{ 'settings.resultsDescription' | translate }}
         </clr-control-helper>
       </clr-select-container>
+      <clr-toggle-container>
+        <label class="clr-col-md-4">{{ 'settings.reverseProxyFix' | translate }}</label>
+        <clr-toggle-wrapper>
+          <input
+            type="checkbox"
+            clrToggle
+            name="reverseProxyFix"
+            class="clr-col-md-8"
+            [checked]="reverseProxyFix.value === 'true'"
+            (change)="reverseProxyFixOnChange($event.target.checked + '')"
+          />
+          <label>{{ 'settings.reverseProxyFixActive' | translate }}</label>
+        </clr-toggle-wrapper>
+        <clr-control-helper>
+          {{ 'settings.reverseProxyFixDescription' | translate }}
+        </clr-control-helper>
+      </clr-toggle-container>
     </div>
     <div class="modal-footer">
       <button type="button" class="btn btn-outline" (click)="isOpen = false" translate>settings.close</button>

ui/src/app/settings/settings/settings.component.spec.ts

@@ -16,7 +16,10 @@ describe('SettingsComponent', () => {
 
   const initialState = {
     [fromSettings.settingsFeatureKey]: {
-      settings: [{name: fromSettings.themeActiveKey, value: 'default'}]
+      settings: [
+        {name: fromSettings.themeActiveKey, value: 'default'},
+        {name: fromSettings.reverseProxyFixKey, value: 'false'}
+      ]
     }
   };
 

ui/src/app/settings/settings/settings.component.ts

@@ -11,6 +11,7 @@ export class SettingsComponent extends ModalDialog implements OnInit {
   themeActive: SettingModel;
   resultsPerPage: SettingModel;
   languageActive: SettingModel;
+  reverseProxyFix: SettingModel;
   languages: Array<string>;
 
   constructor(private settingsService: SettingsService) {
@@ -23,6 +24,7 @@ export class SettingsComponent extends ModalDialog implements OnInit {
       this.themeActive = settings.find(st => st.name === 'theme-active');
       this.resultsPerPage = settings.find(st => st.name === 'results-per-page');
       this.languageActive = settings.find(st => st.name === 'language-active');
+      this.reverseProxyFix = settings.find(st => st.name === 'reverse-proxy-fix-active');
     });
   }
 
@@ -37,4 +39,8 @@ export class SettingsComponent extends ModalDialog implements OnInit {
   languageActiveSettingOnChange(language: string): void {
     this.settingsService.dispatch({name: 'language-active', value: language});
   }
+
+  reverseProxyFixOnChange(active: string): void {
+    this.settingsService.dispatch({name: 'reverse-proxy-fix-active', value: active});
+  }
 }

ui/src/app/settings/store/settings.reducer.spec.ts

@@ -12,23 +12,26 @@ describe('settings/store/settings.reducer.ts', () => {
     let expectedState: fromSettings.SettingsState = {
       settings: [
         {name: fromSettings.themeActiveKey, value: 'value1'},
-        {name: fromSettings.languageActiveKey, value: 'value1'}
+        {name: fromSettings.languageActiveKey, value: 'value1'},
+        {name: fromSettings.reverseProxyFixKey, value: 'value1'}
       ]
     };
     let newState = fromSettings.reducer(
       undefined,
       SettingsActions.loaded({
         settings: [
           {name: fromSettings.themeActiveKey, value: 'value1'},
-          {name: fromSettings.languageActiveKey, value: 'value1'}
+          {name: fromSettings.languageActiveKey, value: 'value1'},
+          {name: fromSettings.reverseProxyFixKey, value: 'value1'}
         ]
       })
     );
     expect(newState).toEqual(expectedState);
     expectedState = {
       settings: [
         {name: fromSettings.themeActiveKey, value: 'value2'},
-        {name: fromSettings.languageActiveKey, value: 'value1'}
+        {name: fromSettings.languageActiveKey, value: 'value1'},
+        {name: fromSettings.reverseProxyFixKey, value: 'value1'}
       ]
     };
     newState = fromSettings.reducer(

ui/src/app/settings/store/settings.reducer.ts

@@ -6,6 +6,7 @@ import {SettingModel} from '../../shared/model/setting.model';
 export const settingsFeatureKey = 'settings';
 export const themeActiveKey = 'theme-active';
 export const languageActiveKey = 'language-active';
+export const reverseProxyFixKey = 'reverse-proxy-fix-active';
 
 export interface SettingsState {
   settings: SettingModel[];
@@ -23,7 +24,8 @@ export const getSetting = (settings: SettingModel[], name: string): string =>
 export const initialState: SettingsState = {
   settings: [
     {name: themeActiveKey, value: 'default'},
-    {name: languageActiveKey, value: 'en'}
+    {name: languageActiveKey, value: 'en'},
+    {name: reverseProxyFixKey, value: 'false'}
   ]
 };
 

ui/src/app/shared/api/schedule.service.spec.ts

@@ -4,6 +4,7 @@ import {TaskService} from './task.service';
 
 describe('shared/api/schedule.service.ts', () => {
   let mockHttp;
+  let mockLocalStorageService;
   let scheduleService;
   let taskService;
   let jsonData = {};
@@ -15,8 +16,11 @@ describe('shared/api/schedule.service.ts', () => {
       post: jasmine.createSpy('post'),
       put: jasmine.createSpy('put')
     };
+    mockLocalStorageService = {
+      get: jasmine.createSpy('get')
+    };
     jsonData = {};
-    taskService = new TaskService(mockHttp);
+    taskService = new TaskService(mockHttp, mockLocalStorageService);
     scheduleService = new ScheduleService(mockHttp, taskService);
   });
 

ui/src/app/shared/api/task.service.spec.ts

@@ -5,6 +5,7 @@ import {TaskExecution} from '../model/task-execution.model';
 
 describe('shared/api/task.service.ts', () => {
   let mockHttp;
+  let mockLocalStorageService;
   let taskService;
   let jsonData = {};
   beforeEach(() => {
@@ -14,8 +15,11 @@ describe('shared/api/task.service.ts', () => {
       post: jasmine.createSpy('post'),
       put: jasmine.createSpy('put')
     };
+    mockLocalStorageService = {
+      get: jasmine.createSpy('get')
+    };
     jsonData = {};
-    taskService = new TaskService(mockHttp);
+    taskService = new TaskService(mockHttp, mockLocalStorageService);
   });
 
   it('getTasks', () => {
@@ -144,6 +148,7 @@ describe('shared/api/task.service.ts', () => {
 
   it('getExecutionLogs', () => {
     mockHttp.get.and.returnValue(of(jsonData));
+    mockLocalStorageService.get.and.returnValue(of('false'));
     taskService.getExecutionLogs(
       TaskExecution.parse({
         externalExecutionId: 'foo',

ui/src/app/shared/api/task.service.ts

@@ -13,12 +13,13 @@ import {
   ValuedConfigurationMetadataPropertyList
 } from '../model/detailed-app.model';
 import {UrlUtilities} from '../../url-utilities.service';
+import {LocalStorageService} from 'angular-2-local-storage';
 
 @Injectable({
   providedIn: 'root'
 })
 export class TaskService {
-  constructor(protected httpClient: HttpClient) {}
+  constructor(protected httpClient: HttpClient, private localStorageService: LocalStorageService) {}
 
   getTasks(
     page: number,
@@ -264,10 +265,13 @@ export class TaskService {
           `tasks/logs/${taskExecution.externalExecutionId}?platformName=${platformName}&schemaTarget=${taskExecution.schemaTarget}`;
     const params = new HttpParams({encoder: new DataflowEncoder()});
     return this.httpClient
-      .get<any>(url, {
-        headers,
-        params
-      })
+      .get<any>(
+        UrlUtilities.fixReverseProxyUrl(url, this.localStorageService.get('reverseProxyActiveValue') === true),
+        {
+          headers,
+          params
+        }
+      )
       .pipe(catchError(ErrorUtils.catchError));
   }
 

ui/src/app/url-utilities.service.ts

@@ -24,4 +24,20 @@ export class UrlUtilities {
     path += path.endsWith('/') ? '' : '/';
     return path;
   }
+
+  public static fixReverseProxyUrl(url: string, active: boolean) {
+    if (!active) {
+      return url;
+    }
+    try {
+      const urlToFix: URL = new URL(url);
+      const baseUrl: URL = new URL(window.location.href);
+      urlToFix.host = baseUrl.host;
+      urlToFix.protocol = baseUrl.protocol;
+      urlToFix.port = baseUrl.port;
+      return urlToFix.href;
+    } catch (_) {
+      return url;
+    }
+  }
 }

ui/src/assets/i18n/de.json

@@ -244,7 +244,10 @@
     "darkTheme": "Dunkel",
     "defaultTheme": "Standard",
     "results": "Ergebnisse",
-    "resultsDescription": "Sie können die Anzahl pro Seite wählen."
+    "resultsDescription": "Sie können die Anzahl pro Seite wählen.",
+    "reverseProxyFix": "Reverse-Proxy-Fix",
+    "reverseProxyFixActive": "Aktivieren",
+    "reverseProxyFixDescription": "Sie können den Fix für den Reverse-Proxy-Betrieb aktivieren."
   },
   "about": {
     "user": {

ui/src/assets/i18n/en.json

@@ -248,7 +248,10 @@
     "darkTheme": "dark",
     "defaultTheme": "default",
     "results": "Results",
-    "resultsDescription": "You can choose the number of results per page."
+    "resultsDescription": "You can choose the number of results per page.",
+    "reverseProxyFix": "Reverse-Proxy-Fix",
+    "reverseProxyFixActive": "Activate",
+    "reverseProxyFixDescription": "You can activate the fix for the reverse proxy operation."
   },
   "about": {
     "user": {

ui/src/assets/i18n/ru.json

@@ -244,7 +244,10 @@
     "darkTheme": "темная",
     "defaultTheme": "По умолчанию",
     "results": "Резултат",
-    "resultsDescription": "Вы можете выбрать количество результатов на странице."
+    "resultsDescription": "Вы можете выбрать количество результатов на странице.",
+    "reverseProxyFix": "Исправление обратного прокси",
+    "reverseProxyFixActive": "Активировать",
+    "reverseProxyFixDescription": "Вы можете включить исправление для работы обратного прокси."
   },
   "about": {
     "user": {