Indy 10 (HTTP-Server), Firefox 3.5 und jQuery.post

[SubData]

Ich musste vor Kurzem feststellen, dass Parameter, die aus Firefox heraus mittels jQuery.post an
einen Indy HTTP-Server übergeben werden, nicht verwendet werden können.

Code:

 $.post('/test', { p1: '1', p2: '2' }, null);

Die Paramter sind über ARequestInfo.FormParams sehr wohl zu sehen, können aber mit
ARequestInfo.Params.Values nicht angesprochen werden.
Nach einiger Suche ist mir dann aufgefallen, dass Firefox im Gegensatz zu, zum Beispiel Opera,
als Content-Type nicht "application/x-www-form-urlencoded" sondern
"application/x-www-form-urlencoded; charset=UTF-8" sendet, was Indy dazu veranlasst, nicht
LRequestInfo.UnparsedParams sondern LRequestInfo.QueryParams zu verwenden.

IdCustomHTTPServer.pas (Zeile 859 - 866)

Delphi-Quellcode:

if ParseParams then begin
  if TextIsSame(LRequestInfo.ContentType, 'application/x-www-form-urlencoded') then begin    {Do not Localize}
    LRequestInfo.DecodeAndSetParams(LRequestInfo.UnparsedParams);
  end else begin
    // Parse only query params when content type is not 'application/x-www-form-urlencoded'    {Do not Localize}
    LRequestInfo.DecodeAndSetParams(LRequestInfo.QueryParams);
  end;
end;

Ich habe mir an dieser Stelle damit geholfen, dass ich Zeile 860 durch folgende ersetzt habe:

Delphi-Quellcode:

if TextIsSame(LRequestInfo.ContentType, 'application/x-www-form-urlencoded') or TextIsSame(LRequestInfo.ContentType, 'application/x-www-form-urlencoded; charset=UTF-8') then begin    {Do not Localize}

Preisfrage: Habe ich etwas übersehen, oder ist das tatsächlich ein Bug in den Indys?

Nachtrag: jQuery.get funktioniert auch mit Firefox ohne Probleme.

[Assertor]

Hallo,

danke fürs Melden, wird gefixt! Sobald unser Fix steht, poste ich hier ggf.

Gruß,
Assertor

[SubData]

Sehr schön, vielen Dank.

[ele]

Was für eine Indy-Version?

Der Fehler ist mir schon eine Weile bekannt - dachte allerdings, dass er in der aktuellen Indy-Version behoben worden ist (oder zumindest ist mir der Fehler in meinen Delphi 2010 Projekten noch nicht über den Weg gelaufen).

Der Patch bei mir ist deinem sehr ähnlich, mit dem Unterschied, dass es bei mir egal ist, ob jetzt dahinter UTF-8, ISO-8859-1 oder sonstwas steht (nicht dass ich sowas je gesehen hätte, aber man will ja für alle Fälle gewappnet sein):

Delphi-Quellcode:

s := LRequestInfo.ContentType;
if TextIsSame(Fetch(s, ';', false), 'application/x-www-form-urlencoded') then begin    {Do not Localize}

[SubData]

Delphi 7 mit der letzten Indy aus dem Repository (zum Zeitpunkt des Originalbeitrags).

Ich ging ursprünglich nicht davon aus, dass es überhaupt ein Fehler in den Indys ist,
sondern dachte, dass ich wahrscheinlich etwas falsch gemacht habe.
Daher auch nur dieser "provisorische" Fix.

[ele]

Hm... ok. Das sind soviel ich weiss nicht die aktuellsten Indy-Sources. Für die neuesten Sourcen musst du den Tiburon-Branch auschecken. Dort ist der Fehler nicht mehr vorhanden (meine ich zumindest).

Allerdings musst du damit aufpassen, da Tiburon das ganze Unicode und Encoding-Zeugs enthält. Du musst hauptsächlich darauf achten, dass ResponseInfo.CharSet einen sinnvollen Wert erhält. Wenn du z.B. ein Bild über den HTTP-Server lieferst wirst du ziemlich bald merken wenn es nicht geht, denn dann ist das Bild korrupt, da einzelne Zeichen falsch codiert werden.

[SubData]

Wie jetzt?

http://indy.fulgan.com/ZIP/

Die indy10.zip ist nicht aktuell?
Warum hat die dann immer ein aktuelles Datum?!
Na toll -.-

[ele]

Ich glaube, das liegt daran, dass die Automatisch erstellt werden...

IndyTiburon.zip ist die aktuellste Version nach meinem Kenntnisstand.

[SubData]

In der ist der Fehler aber ebenfalls enthalten.

[ele]

Hast du das getestet oder einfach mal kurz den Code überflogen?

Weil in meinen Source steht

Delphi-Quellcode:

              
if ParseParams then begin
  if TextIsSame(LContentType, ContentTypeFormUrlencoded) then begin
    LRequestInfo.DecodeAndSetParams(LRequestInfo.UnparsedParams);
  end else begin
    // Parse only query params when content type is not 'application/x-www-form-urlencoded'    {Do not Localize}
    LRequestInfo.DecodeAndSetParams(LRequestInfo.QueryParams);
  end;

und LRequestInfo ist

Delphi-Quellcode:

LContentType := ExtractHeaderItem(LRequestInfo.ContentType);

und da ExtractHeaderItem folgendes macht

Delphi-Quellcode:

function ExtractHeaderItem(const AHeaderLine: String): String;
var
  s: string;
begin
  // Store in s and not Result because of Fetch semantics
  s := AHeaderLine;
  Result := Trim(Fetch(s, ';')); {do not localize}
end;

müsste es meiner Meinung nach funktionieren.

[Assertor]

Hallo,

da muß ich mich mal kurz einschalten: ele hat recht.

Zitat:

Zitat von SubData

Wie jetzt?

http://indy.fulgan.com/ZIP/

Die indy10.zip ist nicht aktuell?
Warum hat die dann immer ein aktuelles Datum?!

Weil das der Indy Mirror ist, also sowohl Trunk als auch Branch täglich zieht. IndyTiburon.zip ist der aktuelle Entwicklungszweig, über die Gründe hab ich mich im Forum schon mehrfach an anderer Stelle ausgelassen, sollte auch auf der Indy Webseite stehen. In meinem Blog steht es auch, bei StackOverflow, im EMBT Forum und halt mal wieder hier jetzt :angel2:

Macht ja nichts, mein "wird geprüft" oben war auch nicht im Sinne von "kann es bestätigen", sondern "schau ich mir mal an" gemeint.

Jetzt zum Problem: SubData, kannst Du mir bitte ein kleines Zip erstellen mit einem Beispielprojekt (also nur IdHTTPServer, die HTML Dateien und JQuery & Co)? Dann muß ich mir nicht erst ein Demo zusammensuchen und kann direkt an Deinem Beispiel arbeiten...

Viele Grüße,
Assertor

:dp:

[SubData]

Verflucht. Ich wusste zwar, dass dies ein Mirror ist, ich ging aber davon aus,
dass das Dateidatum wenigstens den Aktualitätsgrad zeigt.

Ich schmeiß jetzt Indy komplett raus, spiele die aktuelle Version neu ein
und teste es nochmal durch. Wenn dann der Fehler immer noch vorhanden ist,
lass ich es euch wissen ;-)

Danke jedenfalls für die Mühen!

Edit: Assertor, ich bau dir eine Demo, wenn der Fehler danach immer noch auftritt.

Edit2: Die aktuelle Tiburon lässt sich mit Delphi 7 nicht compilieren.

Code:

Fehler: Ein 'Never-build' Package 'dclIndyCore70' benötigt 'always-build' Package 'IndyCore70'
Fehler: Ein 'Never-build' Package 'dclIndyCore70' benötigt 'always-build' Package 'IndySystem70'

Muss ich mich also bei Gelegenheit mal in Ruhe beisetzen...

[ele]

Sollte eigentlich gehen. Es ist zwar nicht ganz trivial, da man das alte Indy erst manuell deinstallieren muss, aber es sollte machbar sein (zumindest habe ich das auch schon hingekriegt). Eventuell schaust du mal in C:\WINDOWS\system32\ rein, dort befinden sich (eigenartigerweise) bei mir nämlich die Dateien IndyProtocols70.bpl, IndyCore70.bpl und IndySystem70.bpl. Eventuell musst du diese nur noch an den richtigen Ort schieben. Aber ich bin mir nicht mehr ganz sicher, ist schon ein paar Monate her.

Möglicherweise hat jemand hier im Forum eine genauere Ahnnung.

[SubData]

Hab wirklich alles von Indy deinstalliert.
Die FullD7.bat will leider trotzdem nicht...

[ele]

Anscheinend wird Handarbeit benötigt:

http://www.delphipraxis.net/internal...t.php?t=174087

[SubData]

Och wie blöd... Nagut, dann morgen mal in Handarbeit.

[mjustin]

Zitat:

Zitat von ele

Anscheinend wird Handarbeit benötigt:

http://www.delphipraxis.net/internal...t.php?t=174087

Oder einfach nur die Indy Source Verzeichnisse in den Suchpfad aufnehmen - funktioniert bei Delphi 6 und 2009 sowie Free Pascal. Die FullDx.bat Dateien sind seit einigen Tagen / Wochen 'broken'.

[SubData]

Also manuelles compilieren funktioniert für Delphi 7 auch nicht so ganz ohne Probleme.

Es fehlen an 2 Stellen (IdMessageClient.pas, IdMessageParts.pas) Parameter vom Typ "TIdHeaderQuotingType".

Delphi-Quellcode:

  TIdHeaderQuotingType = (QuotePlain, QuoteRFC822, QuoteMIMEContentType, QuoteHTTP);

Soweit ich das aus dem Quelltext heraus lesen konnte, dürfte QuoteMIMEContentType funktionieren.
Getestet hab ich das aber soweit nicht.

Edit: Mit nicht getestet meine ich, dass ich an beiden Stellen Quote MIMEContentType eingesetzt
habe und die compilierten DCUs ohne Funktionstest übernommen habe.

[Assertor]

Hallo SubData,

wir machen jetzt mal folgendes:

1) Du sendest mir trotzdem ein Demo, dann kann ich wenigstens diesen Punkt von der ToDoList streichen. Sobald ich bestätigen kann, das es mit neuem Delphi & Indy hier funktioniert (sollte es):

2) Ich bespreche das weitere Vorgehen bezüglich Delphi 5-7.

Im Moment ist das alles sehr schwierig, da die Neu-Entwicklung vorangehen muß. Die FullDxxx.bat Files sind im Moment ganz weit unten auf der Prioritätsliste, aber Kompilieren und manuelles Installieren bekommen wir kurzfristig wieder hin.

Der eine oder andere hat es ja schon in Blogs gelesen, es tut sich einiges bei Embarcadero (Gerüchte gibt es hier: http://wings-of-wind.com/2010/03/14/...braries-ahead/), ich werde das nicht kommentieren - aber Fakt ist, die Jungs aus Scotts Valley kosten uns im Moment viel Kapazität.

Gruß,
Assertor

[SubData]

Du kannst den Punkt von der ToDo-Liste streichen.

Ich hab die Tiburon unter Delphi 7 zum Laufen bekommen und das Problem
mit Firefox und jQuery ist ebenfalls behoben! :)

[Assertor]

Hallo SubData,

Zitat:

Zitat von SubData

Du kannst den Punkt von der ToDo-Liste streichen.

Ich hab die Tiburon unter Delphi 7 zum Laufen bekommen und das Problem
mit Firefox und jQuery ist ebenfalls behoben! :)

Vielen Dank für das Feedback! Freut mich, das es jetzt bei Dir läuft :thumb:

Ich kann mich für die Probleme mit D5-D7 nur entschuldigen, sobald wieder etwas Zeit dafür ist, werden wir das entsprechend fixen.

Gruß,
Assertor

[SubData]

Zitat:

Zitat von Assertor

Hallo SubData,
Vielen Dank für das Feedback! Freut mich, das es jetzt bei Dir läuft :thumb:
Ich kann mich für die Probleme mit D5-D7 nur entschuldigen, sobald wieder etwas Zeit dafür ist, werden wir das entsprechend fixen.

Kein Problem. Ist absolut nachvollziehbar, dass bei einem so großen Projekt nicht alle Ecken und Enden zu jedem Zeitpunkt auf dem aktuellen Stand sein können. Und wenn jemand Beta- und Testversionen installiert, dann muss er einfach damit leben, dass eventuell Sachen nicht oder nur unzureichend funktionieren.

Ich danke dir und ele jedenfalls für die super Hilfe! :)