Skip to content
Snippets Groups Projects

Draft: rss-bridge: add new app

Closed Benedek Nagy requested to merge nbenedek/freedombox:rss-bridge into master

Hi everyone, I created a skeleton for a great tool called rss-bridge. There are two main things that need to be done, for which I'd like to ask for your assistance:

  • A function to allow/disable public access
  • Creating /etc/rss-bridge/whitelist.txt from whitelist.default.txt

Also, I don't know if there should be a systemd file for rss-bridge.


We might want to make this an advanced app, because by default, Firefox downloads the feeds instead of opening them in the browser. On Chromium they open correctly and can be pasted into the feed reader easily.

Merge request reports

Pipeline failed for 5f9e1ebc on nbenedek:rss-bridge

Test coverage 34.98% (-0.10%) from 1 job
Approval is optional

Closed by Sunil Mohan AdapaSunil Mohan Adapa 2 years ago (Jul 16, 2022 3:50am UTC)

Merge details

  • The changes were not merged into .

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • assigned to @nbenedek

  • Also, I don't know if there should be a systemd file for rss-bridge.

    Generally this is not required for PHP apps, they are served directly by the web server (apache). Sometimes they have auxiliary services for regular cleanup, etc, and that will have a systemd service file.

  • Benedek Nagy added 1 commit

    added 1 commit

    • effb5f4d - rssbridge: action: remove .py file extension

    Compare with previous version

  • Benedek Nagy added 1 commit

    added 1 commit

    • 5f9e1ebc - rssbridge: cleanup action file

    Compare with previous version

  • @nbenedek, I took the liberty of making updates to the app and merging it. Hope that is okay. Please review my changes:

    • Update description for simplicity, group info
    • Indentation fixes
    • End all URLs with a slash
    • Update frontpage shortcut to be a simple one
    • Enable single-sign-on for main interface only
    • In copyright file, merge with public-domain section
    • Simplify and vectorify the icon
    • Create whitelist.txt
    • Add functional tests

    Other notes:

    • I have seen the installation documentation and it does not talk about any periodic jobs to be executed. So, I think systemd jobs are not required for this one.
    • I read that using '*' in whitelist.txt enables all the bridges. So, I did that.
    • I think it is not necessary to make this an advance app. The JSON/RSS etc. formats are meant to copied as URLs and pasted into a feed reader like TT-RSS. So, I think it won't matter if Firefox is downloading them instead of showing them.

    We have the following work ahead:

    • Test some more and see if there any serious blockers or bugs. I have tested with TT-RSS but that required using http:// instead of https://
    • Write a wiki page with documentation on how to generate and use RSS feeds with screenshots.
  • Author Maintainer

    @sunilmohan This awesome, thank you for your work!

    I have tested with TT-RSS but that required using http:// instead of https://

    https is possible given a LE certificate is present.

    Write a wiki page with documentation on how to generate and use RSS feeds with screenshots.

    https://wiki.debian.org/FreedomBox/Manual/RSSBridge

Please register or sign in to reply
Loading