Project

General

Profile

Actions

Bug #6401

closed

sendmail: backtick muss escaped werden

Added by Elias Villiger almost 6 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Elias Villiger
Target version:
-
Start date:
10.07.2018
Due date:
% Done:

0%

Estimated time:
Discuss:
Prio Planung:
Vote:

Description

Wenn bei einer _sendmail column das Zeichen ` unpaarig vorkommt, kommt anstelle Mailversand die Fehlermeldung im Anhang.

Das Zeichen müsste wohl generell von qfq escaped werden.


Files

image001.png View image001.png 56.8 KB Elias Villiger, 10.07.2018 20:09
Actions #1

Updated by Carsten Rose almost 6 years ago

Hallo Elias

Escapen ganz sicher - eigentlich sollte das schon via escapeshellarg() passiert sein. Koenntest Du ein Beispiel machen und den Stacktrace (mit dem debugger an die stelle bevor es knallt und dann einen screenshot vom stacktrace im debugger machen) im Ticket anhaengen?

Noch besser waere natuerlich wenn im qfq.log der stacktrace waere. Vermutlich ist dein Master Branch nicht aktuell?

Vielen Dank

CU
Carsten

Actions #2

Updated by Elias Villiger almost 6 years ago

  • Assignee changed from Carsten Rose to Elias Villiger

Ist möglich, dass es in der aktuellsten Version bereits gefixt ist. Ich weise das Ticket mal mir zu, damit ich es mir mal genauer anschaue, wenn ich wieder qfq-aktiver bin.

Actions #3

Updated by Elias Villiger over 5 years ago

  • Status changed from New to Closed

escapeshellargs handelt nur single ticks

Eine andere Option wäre escapeshellcmd, diese ist aber zu gründlich (escapet auch Klammern, Fragezeichen, ... -> dadurch würde alles mögliche nicht mehr korrekt gerendert).

Im Merge Request handle ich die backticks jetzt einfach manuell.

Actions

Also available in: Atom PDF