Opencaching Community

Die Plattform opencaching.de => Entwicklung => Thema gestartet von: mic@ am 13. März 2013, 15:14:41

Titel: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: mic@ am 13. März 2013, 15:14:41
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/Ticket/Display.html?id=5442

Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: 4_Vs am 13. März 2013, 15:26:08
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
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: mic@ am 13. März 2013, 15:31:14
Zitat von: 4_V
ich versteh es nicht was es an dem listing auszusetzen gibt
http://www.opencaching.de/viewcache.php?cacheid=124069

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
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: 4_Vs am 13. März 2013, 16:04:37
Aaah, stibimmt :)
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: flopp am 13. März 2013, 16:15:55
Der Bug ist wohl eher, dass invalider HTML-Quelltext akzeptiert wird.

In obigem Listing findet man z.B.

<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 <b /> macht dort gar keinen Sinn und soll wahrscheinlich eher ein <span> sein. Und genau dieses <b /> ist es, das sämtlichen folgenden Text fett macht...
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: 4_Vs am 13. März 2013, 16:42:00
Hiho,

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

LG
Micha
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: Schrottie am 13. März 2013, 17:00:32
Wenn man das soweit einschränkt, dann beginnen gleich wieder die Ratehakenjünger mit lautem Wehgeschrei.
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: flopp am 13. März 2013, 18:21:28
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.
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: mic@ am 13. März 2013, 21:11:25
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.
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: Schrottie am 14. März 2013, 03:09:11
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.
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.
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: flopp am 14. März 2013, 07:45:25
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."
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: mic@ am 14. März 2013, 07:52:11
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?
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: flopp am 14. März 2013, 10:21:15
Wie geht eigentlich gc mit kaputtem Code um?

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.
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: flopp am 14. März 2013, 10:29:52
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...
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: mic@ am 14. März 2013, 10:33:47
Zitat von: flopp
Aber sie machen eine klare Ansage, was HTML angeht...

Wir auch: http://www.opencaching.de/articles.php?page=htmltags
Titel: Re: Bug: HTML-Format überträgt sich auf Logs
Beitrag von: following am 18. März 2013, 14:11:29
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...

OC schmeißt unerlaubte Tags auch weg, siehe http://www.opencaching.de/articles.php?page=htmltags.