search

web-standards-ru / web-standards.ru

https://web-standards.ru
151 stars 92 forks source link

Лента не читается RSS агрегатором #361

Closed Mendeo closed 2 months ago

Mendeo commented 3 months ago

Лента https://web-standards.ru/articles/feed содержит ошибки и не читается RSS агрегатором (FreshRSS). Это также можно увидеть, если открыть ссылку в Firefox: Ошибка синтаксического анализа XML: некорректно Адрес: https://web-standards.ru/articles/feed/ Строка 13, символ 33:...

Или в Chrome: This page contains the following errors: error on line 13 at column 31:...

pepelsbey commented 3 months ago

Спасибо за подробности! В чём сложность: XML не должен содержать -- внутри блока <![CDATA[…]]>, а у нас в статьях часто встречаются кастомные CSS-свойства. Но если убрать CDATA, как я попытался сделать в этом пулреквесте, то одиночные <img> уже не XML и возникает другая проблема.

В общем, интересная задача. Будем думать. Спасибо!

alexeyten commented 3 months ago

А может заменять в тексте вхождения -- на -]]><![CDATA[- ?

pepelsbey commented 3 months ago

А может заменять в тексте вхождения -- на -]]><![CDATA[- ?

Мне больше нравится заменять <img> на <img/>, честно говоря)

monochromer commented 3 months ago

А json-формат фида не подойдёт? Или есть вероятность, что сломаются подписки, так как будет другой файл?

pepelsbey commented 3 months ago

Не уверен, насколько хорошая поддержка у JSON-фидов. XML-фид для совместимости всё равно нужен.

alexeyten commented 3 months ago

Мне больше нравится заменять <img> на <img/>, честно говоря)

А потом найдётся какой-нибудь <br>, <hr> и прочие <source> :smirk:

pepelsbey commented 3 months ago

А потом найдётся какой-нибудь <br>, <hr> и прочие <source> 😏

Ну да, XML-изация нужна полная 🥲

pepelsbey commented 2 months ago

Должно стать лучше после https://github.com/web-standards-ru/web-standards.ru/pull/363

https://web-standards.ru/articles/feed/

Спасибо за репорт и терпение :)