Project

General

Profile

Actions

Feature #15770

closed

httpOrigin: Parameter missing in QFQ Config

Added by Carsten Rose about 1 year ago. Updated 10 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Carsten Rose
Target version:
Start date:
09.03.2023
Due date:
12.03.2023
% Done:

0%

Estimated time:
Discuss:
Prio Planung:
No
Vote:

Description

Es sieht so aus als wenn der Parameter in der QFQ Config abhanden gekommen ist (Problem ist aufgetreten in #15767)

Es gibt auch einen Impact zu baseUrl.

Wenn wir die Logik drin lassen, sollten wir es dokumentieren und via Extension Config verfuegbar machen.

Kommentar: https://project.math.uzh.ch/issues/10782#note-22


Files


Related issues

Related to QFQ - Feature #10782: Tiny MCE: Image UploadClosedEnis Nuredini22.06.2020

Actions
Actions #1

Updated by Carsten Rose about 1 year ago

Actions #3

Updated by Carsten Rose about 1 year ago

  • Description updated (diff)
  • Assignee changed from Carsten Rose to Enis Nuredini
Actions #4

Updated by Enis Nuredini about 1 year ago

HTTP_Origin systemvariable sollte nicht gespeichert werden. Wert muss bei jedem Ladevorgang neu geladen werden.

Actions #5

Updated by Enis Nuredini about 1 year ago

  • Status changed from New to In Progress

Das nicht Speichern der Systemvariable ist kritisch, da z.b. über Autocron zugegriffen die SERVER Variablen nicht existieren. Somit sollte dies doch in die Config gespeichert werden. Jedoch sollte bei jedem Ladevorgang zuerst überprüft werden ob sich der HTTP_Origin geändert hat und wenn ja dann soll der Wert in die Config geschrieben werden. Ein Branch mit passender Lösung ist erstellt. Kann erst gemerged werden wenn Unit Test wieder läuft.

HTTP_ORIGIN wurde ursprünglich für tinyMce Image Upload eingeführt.

Manueller Eintrag über QFQ Config unnötig, da benötigte Werte von Server Variablen entnommen werden kann und diese Ausschlaggebend für den TinyMce Picture Upload sind.

Actions #6

Updated by Carsten Rose about 1 year ago

  • Tracker changed from Support to Feature
Actions #7

Updated by Enis Nuredini almost 1 year ago

  • Status changed from In Progress to Ready to sync (develop)
  • Assignee changed from Enis Nuredini to Carsten Rose

Anpassung mit Branch #15767 zusammengefasst. Dortiger Branch ist bereit für Merge. Ticket selbst wurde dort schon geschlossen, da spezifiesch auf ein Fall zugeschnitten war.

Actions #8

Updated by Carsten Rose 10 months ago

  • Status changed from Ready to sync (develop) to Closed
  • Target version changed from 24.8.0 to 23.6.0
Actions

Also available in: Atom PDF