Bug: HTML-Format überträgt sich auf Logs

[mic@]

Mir fiel eben auf, daß bestimmte HTML-Formatierungen anscheinend nicht korrekt geschlossen werden,
daß Listing also nicht von den Logs gekapselt ist. Hier mein Ticket dazu:
https://idefix.opencaching.de/rt3/Ticke ... ml?id=5442

[4_Vs]

Hiho,

ich versteh es nicht was es an dem listing auszusetzen gibt
http://www.opencaching.de/viewcache.php?cacheid=124069

Kannst Du das mal bitte genauer erklären?

LG
Micha

[mic@]

[quote="4_V"]ich versteh es nicht was es an dem listing auszusetzen gibt
http://www.opencaching.de/viewcache.php?cacheid=124069[/quote]

Der innere Listing-Teil endet mit einer fetten Schritftart (bold),
und diese Formatierung wird auch in den Logs beibehalten.
Normalerweise sind Logtexte nicht ganz so fett, sehen also so aus:
http://www.opencaching.de/viewcache.php?wp=OC6155

[4_Vs]

Aaah, stibimmt

[flopp]

Der Bug ist wohl eher, dass invalider HTML-Quelltext akzeptiert wird.

In obigem Listing findet man z.B.

Code: Alles auswählen

<p style="text-align: justify; font-style: italic; font-size: 8pt; margin-left: 50px; ">
<b />Wer's genau wissen will</span>: Es handelt sich um Vers 23, allerdings 
fehlt der Kapitelnummer eine 1 als Hunderterstelle, die richtige Kapitelnummer 
der lateinischen Vulgata ist also 100+A. In der heute gebräuchlichen 
Standardübersetzung ist der Vers jedoch in Kapitel 100+A+1 zu finden.
</p>

Das

Code: Alles auswählen

<b />

macht dort gar keinen Sinn und soll wahrscheinlich eher ein

Code: Alles auswählen

<span>

sein. Und genau dieses

Code: Alles auswählen

<b />

ist es, das sämtlichen folgenden Text fett macht...

[4_Vs]

Hiho,

normalerweise würde es doch reichen, wenn man font, img, b, i, u, ol, ul, href, br und p erlaubt, oder?

LG
Micha

[Schrottie]

Wenn man das soweit einschränkt, dann beginnen gleich wieder die Ratehakenjünger mit lautem Wehgeschrei.

[flopp]

Könnte man die Listings (also den vom User eingegebenen Teil + ein bisschen Gerüst außen rum) nicht automatisch durch den W3C-Markup-Validator schicken - da gibt es sogar eine API: http://validator.w3.org/docs/api.html
Ich gehe mal davon aus, dass falsche, fehlende Tags damit zuverlässig gefunden werden können.

[mic@]

Nur eine Frage von einem Newbie-Programmierer:
Was würde denn passieren, wenn wir auf Vorrat einfach alle HTML-Tags schließen.
Dann dürfte doch keines mehr übrigbleiben, was uns in den Rest der Seite hineinpfutscht.

[Schrottie]

[quote="mic@"]
Nur eine Frage von einem Newbie-Programmierer:
Was würde denn passieren, wenn wir auf Vorrat einfach alle HTML-Tags schließen.
Dann dürfte doch keines mehr übrigbleiben, was uns in den Rest der Seite hineinpfutscht.
[/quote]
Dann liefert OC komplett kranken Quelltext aus und wir machen uns zum Gespött der Leute. Mehr sollte nicht passieren.

Aber mal im Ernst: Ich finde es unnötig, auf der einen Seite dem Owner möglichst viel Freiraum zu lassen und ihm dann auf der anderen Seite irgendwelche Tags zu schließen. Wenn sein Listing Scheiße aussieht, dann hat er das selbst in der Hand.

[flopp]

Deswegen vielleicht ein dezente Warnung von einem Validator:

"Achtung: laut HTML-Validator enthält das Cachelisting folgende Fehler: ... Das kann zu Problemen bei der Anzeige führen. Bitte korrigieren."

[mic@]

Wie geht eigentlich gc mit kaputtem Code um?
Meines Wissens schlägt das doch nicht auf die Logs durch,
und eine HTML-Warnung geben sie auch nicht aus.
Ist das vielleicht bei Ihnen so eine Art "eingebetteter"
HTML-Code (iframe), so daß sich Fehler erst gar nicht
nach außen fortpflanzen können?

[flopp]

[quote="mic@"]
Wie geht eigentlich gc mit kaputtem Code um?
[/quote]

Soweit ich weiß, drehen sie den kompletten Quellcode einmal durch die große Bereinigungsmühle, entfernen dabei alles kaputte & unerlaubte, und geben dann bereinigten Code zurück.

[flopp]

Hab's grade ausprobiert: GC schließt offene Tags und schmeißt unerlaubte Tags kommentarlos raus. Somit kann obiger Fehler bei GC nicht auftreten.

Aber sie machen eine klare Ansage, was HTML angeht...

[mic@]

[quote="flopp"]Aber sie machen eine klare Ansage, was HTML angeht...[/quote]

Wir auch: http://www.opencaching.de/articles.php?page=htmltags