Miroslavsckaya/tg_rss_bot
1
1
Fork 0
Code Issues 7 Pull requests Projects 1 Releases Activity

Filter HTML tags #29

New issue
Closed
opened 2022-07-08 23:47:45 +00:00 by Miroslavsckaya · 3 comments
Miroslavsckaya commented 2022-07-08 23:47:45 +00:00
Owner
Copy link

We need to filter unsupported HTML tags

https://core.telegram.org/bots/api#html-style

We need to filter unsupported HTML tags https://core.telegram.org/bots/api#html-style
skobkin was assigned by Miroslavsckaya 2022-07-08 23:47:52 +00:00
Miroslavsckaya added this to the RSS Bot Kanban Perdoling project 2022-07-08 23:48:02 +00:00
Miroslavsckaya added the
bug
enhancement
research needed
 labels 2022-07-08 23:48:26 +00:00
skobkin added this to the MVP 0.1 milestone 2022-07-09 14:52:33 +00:00
skobkin commented 2022-07-09 15:36:43 +00:00
Collaborator
Copy link

Possible options:

Possible options: - https://lxml.de/api/lxml.html.clean-module.html - BeautifulSoup - [`get_text()`](https://www.crummy.com/software/BeautifulSoup/bs4/doc/#get-text) - [`unwrap()`](https://www.crummy.com/software/BeautifulSoup/bs4/doc/#unwrap) - [`html-sanitizer`](https://pypi.org/project/html-sanitizer/) - [`Bleach.clean()`](https://bleach.readthedocs.io/en/latest/clean.html#using-bleach-sanitizer-cleaner)
skobkin commented 2022-07-09 16:23:41 +00:00
Collaborator
Copy link

It's also possible to sanitize HTML right in the RssReader with proper FeedParser configuration.

But it's better to sanitize it in telegram.Notifier because those are platform-specific restrictions.

It's also possible to sanitize HTML right in the `RssReader` with proper `FeedParser` [configuration](https://pythonhosted.org/feedparser/html-sanitization.html). But it's better to sanitize it in `telegram.Notifier` because those are platform-specific restrictions.
skobkin commented 2022-07-09 19:56:02 +00:00
Collaborator
Copy link
  • lxml didn't work out well because it was constantly wrapping the provided string or document in the div element as a 'container'.
- `lxml` didn't work out well because it was constantly wrapping the provided string or document in the `div` element as a 'container'.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
master
No results found.
Labels
Clear labels   
bug

Something is not working

  
enhancement

New feature

  
epic

   
invalid

Nothing to fix or wrong report

  
new feature

   
refactoring

   
research needed

   
wontfix

No need to resolve

No labels
bug
enhancement
epic
invalid
new feature
refactoring
research needed
wontfix
Milestone
Clear milestone
No items
No milestone
MVP 0.1
Projects
Clear projects
No items
No project
RSS Bot Kanban Perdoling
Assignees
Clear assignees
No assignees
skobkin
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Miroslavsckaya/tg_rss_bot#29
No description provided.
Delete branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?