[PHP][RSS] RSS Feed voldoet niet aan de specs ?

[Verwijderd]

Ik ben al dagen bezig mijn rss feed op orde te krijgen. Het probleem is dat een aantal sites mijn feed niet goed kunnen lezen. Steeds bezig om met feedvalidator.org de foutjes eruit te halen, maar hij blijft zeuren over:

Feeds should not be served with the "text/html" media type [help]

Your feed appears to be encoded as "ISO-8859-1", but your server is reporting "US-ASCII" [

Ik heb mijn php bestand al omgezet naar iso-8859-1, en ook het include bestand wat ik hierin gebruik, poging gedaan met htaccess, maar kom geen stap verder. In een aantal feedreaders werkt het goed, maar bv headliner.nl en google hebben ergens moeite mee, maar geven natuurlijk geen fouten door....

Zie: http://feedvalidator.org/....net/rss/rssfcutrecht.php

[crisp]

PHP:

header('Content-type: text/xml; charset=ISO-8859-1');

[Snake]

crisp schreef op dinsdag 14 augustus 2007 @ 20:16:

PHP:

header('Content-type: text/xml; charset=ISO-8859-1');

Moet het voor een rss feed niet

PHP:

    header("Content-Type: application/rss+xml; charset=utf-8");

zijn?

De charset mag natuurlijk anders zijn

[ Voor 6% gewijzigd door Snake op 14-08-2007 20:19 ]

[Japius]

Ten eerste heb je een correct Content-Type nodig:

code:

Date: Tue, 14 Aug 2007 18:30:16 GMT
Server: Apache
X-Powered-By: PHP/4.4.2
Keep-Alive: timeout=15, max=1000
Connection: Keep-Alive
Transfer-Encoding: chunked
Content-Type: text/html

200 OK

Dit wordt ook keurig aangegeven in de help van de pagina die je opgeeft.

Daarnaast bevat je feed fouten, zoals die op regel 21. Deze zul je eerst moeten verbeteren. Post anders eens wat code, dan kunnen we zien wat je doet.

[Verwijderd]

Bij deze de code, vast bedankt voor jullie hulp !

<?
include "../connect.inc.php";

$tijdnu = time();
$result=mysql_query("SELECT * FROM nieuws WHERE datum < '".$tijdnu."'ORDER BY datum DESC LIMIT 0,20")or die(mysql_error());

echo("<?xml version=\"1.0\" encoding=\"ISO-8859-1\" ?>\n");

//
?>
<rss version="2.0">
<channel>
<copyright>Copyright 2007 www.fcutrecht.net</copyright>
<pubDate><? echo date("r", time() ); ?></pubDate>
<lastBuildDate><? echo date("r", time() ); ?></lastBuildDate>
<description>...</description>
<category>sport</category>
<language>nl-nl</language>
<link>http://www.fcutrecht.net/</link>
<title>fcutrecht.net rss nieuwsberichten</title>

<?
function ed($tekst) {
$tekst = stripslashes($tekst);
$tekst = strip_tags($tekst);
$tekst = str_replace("<", "<", $tekst);
$tekst = str_replace(">", ">", $tekst);
return $tekst;
}
while($nieuws=mysql_fetch_array($result)) {

echo "<item>";
echo "<title>\n".ed($nieuws["titel"])."\n</title>";
echo "<link>".$baseurl."?view=".$nieuws["id"]."</link>";
echo "<description>";
if (!empty($nieuws->intro)) {
echo ed($nieuws["intro"]);
} else {
echo ed($nieuws["bericht"]);
}
echo "</description>"; // <br> ".ed($nieuws["bron"])."
echo "<author>\[e-mail]\n</author>";
echo "<pubDate>\n".date("r", $nieuws["datum"]) ."\n</pubDate>";
echo "</item>";

}

?>
</channel>
</rss>

[Japius]

Verwijderd schreef op dinsdag 14 augustus 2007 @ 20:39:
[code]

<?
include "../connect.inc.php";

$tijdnu = time();
$result=mysql_query("SELECT * FROM nieuws WHERE datum < '".$tijdnu."'ORDER BY datum DESC LIMIT 0,20")or die(mysql_error());

Ik zou de $tijdnu er uit knallen, en de query maken met

code:

... from nieuws where datum < now() ..

echo("<?xml version=\"1.0\" encoding=\"ISO-8859-1\" ?>\n");

Hiermee geef je aan dat je ISO-8859-1 gebruikt, maar je spuugt het niet zo uit. Zoals Crisp, of Snake al zei:

Snake schreef op dinsdag 14 augustus 2007 @ 20:19:
[...]

Moet het voor een rss feed niet

PHP:

    header("Content-Type: application/rss+xml; charset=utf-8");

zijn?

De charset mag natuurlijk anders zijn

Ik denk dat je dan al een stapje verder bent.

[Verwijderd]

Hiermee starten dus

<?php
header("Content-Type: application/rss+xml; charset=utf-8");
?>

echo("<?xml version=\"1.0\" encoding=\"ISO-8859-1\" ?>\n");

En deze ISO-8859-1 ook omzetten naar utf-8 ?

[frickY]

Er.. mits je content in je database ook UTF-8 is, en je ook het bestand als UTF-8 opslaat. Anders vooral ISO blijven gebruiken

[Noork]

Is het niet handiger om b.v. een component zoals Feedcreator te gebruiken? Dan ben je van het validatiegebeuren af.

[Verwijderd]

Noork schreef op dinsdag 14 augustus 2007 @ 21:03:
Is het niet handiger om b.v. een component zoals Feedcreator te gebruiken? Dan ben je van het validatiegebeuren af.

Heb ik al aan gedacht, heb alleen nog niet gekeken of dit er makkelijk tussen te krijgen is. Ik heb helaas maar weinig php ervaring...

[Japius]

Tuurlijk niet, gewoon doorprutsen, weet je het ook voor de volgende keer.

[Noork]

Verwijderd schreef op dinsdag 14 augustus 2007 @ 21:06:
[...]


Heb ik al aan gedacht, heb alleen nog niet gekeken of dit er makkelijk tussen te krijgen is. Ik heb helaas maar weinig php ervaring...

Veel makkelijker dan zelf prutsen.

[Verwijderd]

Nu als volgt aangepast, nu alleen nog een probleempje met de characters zoals euro teken e.d.

<?php
header("Content-Type: application/rss+xml; charset=ISO-8859-1");
print '<?xml version="1.0" encoding="iso-8859-1"?>';
?>

<?
include "../connect.inc.php";

$tijdnu = time();
$result=mysql_query("SELECT * FROM nieuws WHERE datum < '".$tijdnu."'ORDER BY datum DESC LIMIT 0,20")or die(mysql_error());
?>

<rss version="2.0">
<channel>
<copyright>Copyright 2007 www.fcutrecht.net</copyright>
<pubDate><? echo date("r", time() ); ?></pubDate>
<lastBuildDate><? echo date("r", time() ); ?></lastBuildDate>
<description>...</description>
<category>sport</category>
<language>nl-nl</language>
<link>http://www.fcutrecht.net/</link>
<title>fcutrecht.net rss nieuwsberichten</title>

<?
function ed($tekst) {
$tekst = stripslashes($tekst);
$tekst = strip_tags($tekst);
$tekst = str_replace("<", "<", $tekst);
$tekst = str_replace(">", ">", $tekst);
return $tekst;
}
while($nieuws=mysql_fetch_array($result)) {

echo "<item>";
echo "<title>\n".ed($nieuws["titel"])."\n</title>";
echo "<link>".$baseurl."?view=".$nieuws["id"]."</link>";
echo "<description>";
if (!empty($nieuws->intro)) {
echo ed($nieuws["intro"]);
} else {
echo ed($nieuws["bericht"]);
}
echo "</description>"; // <br> ".ed($nieuws["bron"])."
echo "<author>\[e-mail]\n</author>";
echo "<pubDate>\n".date("r", $nieuws["datum"]) ."\n</pubDate>";
echo "</item>";

}

?>
</channel>
</rss>

[Verwijderd]

iso-8859-1 omgezet naar windows-1252 en de problemen met de high ascii characters zijn ook verdwenen.

Laatst wat de validator zegt:

This feed is valid, but may cause problems for some users. We recommend fixing these problems.

line 20, column 10: item should contain a guid element (19 occurrences) [help]

</pubDate></item><item><title>

[RaZ]

Je mist dus:

XML:

<guid>
..blaat
</guid>

Nu mag je zelf opzoek naar wat daar in hoort te staan

[crisp]

Snake schreef op dinsdag 14 augustus 2007 @ 20:19:
[...]

Moet het voor een rss feed niet

PHP:

    header("Content-Type: application/rss+xml; charset=utf-8");

zijn?

De charset mag natuurlijk anders zijn

text/xml is inderdaad deprecated, maar is soms noodzakelijk voor oudere UA's. In hoeverre dat voor RSS UA's nog actueel is durf ik echter niet te zeggen...

[Verwijderd]

Hmm, hij is nog niet helemaal goed. Ik open hem net met een IE6.0 browser en die herkend het niet eens als rss feed... vanavond toch maar weer even verder zoeken

[Kalentum]

IE6 ondersteunt geen RSS. Voor IE6 is het gewoon een ander XML bestand.

[Verwijderd]

Denk dat je geijk hebt, was meer het verschil in presentatie. Normaal zie je een scherm met de rss text, nu kreeg ik een melding van opslaan of openen...

Maar als ik de guid elements toegevoegd heb voldoe ik helemaal aan de feedvalidator eisen, dan zou het eindelijk goed moeten zijn.

[crisp]

Verwijderd schreef op woensdag 15 augustus 2007 @ 10:09:
Denk dat je geijk hebt, was meer het verschil in presentatie. Normaal zie je een scherm met de rss text, nu kreeg ik een melding van opslaan of openen...

Maar als ik de guid elements toegevoegd heb voldoe ik helemaal aan de feedvalidator eisen, dan zou het eindelijk goed moeten zijn.

IE6 is dus een voorbeeld van een UA die geen application/rss+xml 'begrijpt' - met text/xml was dat wel goed gegaan

Verder vind ik je keuze voor windows-1252 als charset twijfelachtig. Als het enkel om het euroteken gaat zou je ISO-8859-15 kunnen overwegen, maar indien mogelijk zou ik uiteindelijk toch voor UTF-8 gaan.

[Verwijderd]

Het ging niet alleen om het euro teken, volgens mij valt de feedvalidator over alle high ascii characters. Ik las het voorstel ergens op internet en het leek de oplossing. Maar ik begrijp dat UTF-8 de voorkeur heeft ?

<?php
header("Content-Type: application/rss+xml; charset=ISO-8859-1");
print '<?xml version="1.0" encoding="iso-8859-1"?>';

Moet het zowel bij het content-type staan, als bij de encoding ?

[ Voor 31% gewijzigd door Verwijderd op 15-08-2007 10:37 ]

[Snake]

Verwijderd schreef op woensdag 15 augustus 2007 @ 10:36:
Het ging niet alleen om het euro teken, volgens mij valt de feedvalidator over alle high ascii characters. Ik las het voorstel ergens op internet en het leek de oplossing. Maar ik begrijp dat UTF-8 de voorkeur heeft ?

<?php
header("Content-Type: application/rss+xml; charset=ISO-8859-1");
print '<?xml version="1.0" encoding="iso-8859-1"?>';

Moet het zowel bij het content-type staan, als bij de encoding ?

Yup, maar je feed code is zowiezo een zooitje, en ipv die functie 'ed' te gebruiken, zet alles tussen CDATA tags, daarvoor dienen die

[Verwijderd]

Oke, mijn hele feed een zooitje ?

Kan je een wat uitgebreider voorbeeld geven, ik heb niet de ervaring het geheel te herschrijven zodat het beter wordt. Ik wil er alles aan doen om een perfecte rss feed te krijgen, deze is erg belangrijk voor mijn website maar heb er gewoon hulp bij nodig.

[Snake]

Verwijderd schreef op woensdag 15 augustus 2007 @ 11:44:
Oke, mijn hele feed een zooitje ?

Kan je een wat uitgebreider voorbeeld geven, ik heb niet de ervaring het geheel te herschrijven zodat het beter wordt. Ik wil er alles aan doen om een perfecte rss feed te krijgen, deze is erg belangrijk voor mijn website maar heb er gewoon hulp bij nodig.

Ik zal 'm even herschrijven

[Snake]

PHP:

<?php
    include "../connect.inc.php";
    header("Content-Type: application/rss+xml; charset=utf-8");
    
    print '<?xml version="1.0" encoding="utf-8"?>';
    


    $result = mysql_query("
        SELECT 
            * 
        FROM 
            nieuws 
        WHERE 
            datum < '" . time() . "' 
        ORDER BY 
            datum 
        DESC LIMIT 0,20")
    or die(mysql_error());
    //ik heb m'n twijfels bij die ' ' rond time(), is het opgeslagen als TEXT of als INT?
?>

<rss version="2.0">
    <channel>
        <copyright>Copyright 2007 www.fcutrecht.net</copyright>
        <pubDate><?php echo date("r", time() ); ?></pubDate>
        <lastBuildDate><?php echo date("r", time() ); ?></lastBuildDate>
        <description>...</description>
        <category>sport</category>
        <language>nl-nl</language>
        <link>http://www.fcutrecht.net/</link>
        <title>fcutrecht.net rss nieuwsberichten</title>
        <?php
        while($nieuws = mysql_fetch_array($result))
        {
            echo "<item>";
            //htmlentities rond je titel
            echo "<title>" . htmlentities($nieuws["titel"], ENT_QUOTES, "UTF-8") . "\n</title>";
            echo "<link>" . $baseurl . "?view=" . $nieuws["id"] . "</link>";
            //CDATA tags rond je inhoud, moet je niet al die zever strippen
            echo "<description><![CDATA[";
            echo empty($nieuws["intro"]) ? $nieuws["bericht"] : $nieuws["intro"];
            echo "]]></description>"; // <br> ".ed($nieuws["bron"])."
            echo "<author>[e-mail]</author>";
            echo "<pubDate>" . date("r", $nieuws["datum"]) . "</pubDate>";
            echo "</item>";
        }
        ?>
</channel>
</rss>

En gebruik <?php ipv <?

[ Voor 4% gewijzigd door Snake op 15-08-2007 11:57 ]

[FragFrog]

Doe nou niet dit

PHP:

 or die(mysql_error());

Hackers van de wereld zouden 't een stuk moeilijker hebben als niet iedereen continu gegevens over z'n mislukte SQL queries zou uitpoepen..

_{Voor de rest vind ik je code wel heel mooi hoor}

* FragFrog gebruikt overigens ook meestal feedcreator, werkt toch wel fijn en net

[ Voor 12% gewijzigd door FragFrog op 15-08-2007 12:47 ]

[Verwijderd]

Geweldig, dit had ik niet verwacht. Je hebt me hier enorm mee geholpen, ik ga hem vanavond direct proberen en zal je het resultaat laten weten.

Nogmaals, erg bedankt

[remco_k]

Snake schreef op woensdag 15 augustus 2007 @ 11:55:

PHP:

    header("Content-Type: application/rss+xml; charset=utf-8");

Moet dat niet

PHP:

    header("Content-Type: application/xml; charset=utf-8");

Zijn?
Bij mij komt mozilla in het rss+xml geval met een download prompt, terwijl ik de xml eigenlijk gewoon wil zien... Bij enkel xml is dat het geval.

[Snake]

@ ^^

Ik stuur deze header mee:

PHP:

    header("Content-Type: application/rss+xml; charset=utf-8");

Wat krijg je hier? http://www.kristofmattei.be/rss/ ?

[remco_k]

Snake schreef op woensdag 15 augustus 2007 @ 13:00:
@ ^^

Ik stuur deze header mee:

PHP:

    header("Content-Type: application/rss+xml; charset=utf-8");

Wat krijg je hier? http://www.kristofmattei.be/rss/ ?

Eveneens een download prompt.
(overigens werk het in IE7 wel - maar dat is even geen maatstaaf. )

[ Voor 10% gewijzigd door remco_k op 15-08-2007 13:03 ]

[Snake]

remco_k schreef op woensdag 15 augustus 2007 @ 13:02:
[...]

Eveneens een download prompt.

Dat ligt toch wel aan de browser. Ik gebruik Firefox 2.0.0.6

En het is een RSS feed, met XML opmaak (hoe anders?)

application/xml is nog iets anders, dat is gewoon... XML.

[FragFrog]

Lijkt me eerder een bug (oke, feature ) in mozilla remco, IE pakt 'm volgens mij wel goed wat ik me meen te herinneren. Wat doet die eigenlijk als je 'm een inline content header meegeeft?

PHP:

header('Content-Disposition: inline"');

Zelf trouwens nooit last van, zoals gezegd, voor feeds gebruik ik doorgaans feedcreator, geen nut het wiel opnieuw uit te vinden

[ Voor 21% gewijzigd door FragFrog op 15-08-2007 13:07 ]

[FlorisB]

Bovendien is het misschien zeker handig om je MySQL connectie&tables op je gebruikte charset te zetten, gaat je een hoop gezeik besparen.

[remco_k]

FragFrog schreef op woensdag 15 augustus 2007 @ 13:04:
Lijkt me eerder een bug (oke, feature ) in mozilla remco, IE pakt 'm volgens mij wel goed wat ik me meen te herinneren. Wat doet die eigenlijk als je 'm een inline content header meegeeft?

PHP:

header('Content-Disposition: inline"');

Dan komt mozilla altijd met een download prompt, bij application/xml en application/rss+xml
Edit: zou een bug ofzo kunnen zijn, mijn mozilla dateerd nog uit 1920.

[ Voor 7% gewijzigd door remco_k op 15-08-2007 13:07 ]

[Verwijderd]

Snake schreef op woensdag 15 augustus 2007 @ 13:00:
@ ^^

Ik stuur deze header mee:

PHP:

    header("Content-Type: application/rss+xml; charset=utf-8");

Wat krijg je hier? http://www.kristofmattei.be/rss/ ?

Bij IE6 inderdaad ook een downloadprompt...

[Verwijderd]

Had gehoopt dat ik een flinke stap verder was, maar het resultaat valt tegen:

feedvalidator fcutrecht.net

Misschien is de [CDATA optie voor mij toch niet zo'n geschikte keuze...

[ Voor 14% gewijzigd door Verwijderd op 15-08-2007 20:21 ]

[Snake]

Verwijderd schreef op woensdag 15 augustus 2007 @ 20:05:
Had gehoopt dat ik een flinke stap verder was, maar het resultaat valt tegen:

feedvalidator fcutrecht.net

Misschien is de [CDATA optie voor mij toch niet zo'n geschikte keuze...

Die CData werkt perfect.
Alleen is er iets mis met die UTF-8.

[Verwijderd]

Ik heb hem aangepast naar dit: en hem even gekopieerd naar www.fcutrecht.net/rss/rss.php

<?php
include "../connect.inc.php";
header("Content-Type: application/xml; charset=ISO-8859-15");

print '<?xml version="1.0" encoding="ISO-8859-15"?>';


$result = mysql_query("
SELECT
*
FROM
nieuws
WHERE
datum < " . time() . "
ORDER BY
datum
DESC LIMIT 0,20")
or die(mysql_error());
//ik heb m'n twijfels bij die ' ' rond time(), is het opgeslagen als TEXT of als INT?
?>

<rss version="2.0">
<channel>
<copyright>Copyright 2007 www.fcutrecht.net</copyright>
<pubDate><?php echo date("r", time() ); ?></pubDate>
<lastBuildDate><?php echo date("r", time() ); ?></lastBuildDate>
<description>...</description>
<category>sport</category>
<language>nl-nl</language>
<link>http://www.fcutrecht.net/</link>
<title>fcutrecht.net rss nieuwsberichten</title>
<?php
while($nieuws = mysql_fetch_array($result))
{
echo "<item>";
//htmlentities rond je titel
echo "<title>" . htmlentities($nieuws["titel"], ENT_QUOTES, "UTF-8") . "\n</title>";
echo "<link>" . $baseurl . "?view=" . $nieuws["id"] . "</link>";
//CDATA tags rond je inhoud, moet je niet al die zever strippen
echo "<description><![CDATA[";
echo empty($nieuws["intro"]) ? $nieuws["bericht"] : $nieuws["intro"];
echo "]]></description>"; // <br> ".ed($nieuws["bron"])."
echo "<author>[e-mail]</author>";
echo "<pubDate>" . date("r", $nieuws["datum"]) . "</pubDate>";
echo "</item>";
}
?>
</channel>
</rss>

Maar dit vindt de feedvalidator ook niet fijn !

[Verwijderd]

Wat betekend dit ?

//CDATA tags rond je inhoud, moet je niet al die zever strippen

Zou het niet verstandig zijn die CDATA er toch uit te halen om een betere uitwisselbaarheid te bereiken met alle systemen. Is op zich erg mooi maar als er diverse readers zijn die er moeite mee hebben dan is het misschien niet verstandig...

[Noork]

CDATA is toch juist bedoeld om HTML in te knallen. Voor plain tekst gebruik je deze niet. Je kunt dus ook gewoon alleen de platte tekst meesturen, deze afweging moet je zelf maken.

[ Voor 33% gewijzigd door Noork op 16-08-2007 22:28 ]

[Verwijderd]

Ja helemaal eens, maar ik begrijp nog steeds niet goed waarom bepaalde sites (bv headliner.nl) mijn berichten niet meer lezen. Het is lastig communiceren met die gasten omdat ze niet reageren op mailtjes, dus ik weet niet of ze echt problemen hebben met mijn layout.

Maar de layout en code is er flink op vooruit gegaan, mede door jullie hulp