-
-
Notifications
You must be signed in to change notification settings - Fork 794
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SQL error updateTTL when importing tt-rss opml [BUG] #3552
Comments
I will fix. Would you mind sharing the file (or a subset) you are trying to import? |
Thx, clicking on the update button it stalls, each time on a different feed. |
Your current version of FreshRSS is 1.18.0.
I'm quite new to FreshRSS so I don't know how to udate to that branch. In system configuration (as admin) there's an Auto-update server URL pointing to https://update.freshrss.org Is that the option to change to a different value to test the new branch? I guess I can try to manually replace those 4 files you modified. Is that ok or I risk breaking future updates? |
As our version 1.18.0 as been released so recently, it is OK to replace the 4 files. |
Manually updated the 4 files and created a new user i tried to import the opml i sent you yesterday. It's an old file with also a< few non working feeds it stalls without any error https://i.imgur.com/UV3B4Ww.png sometimes at a different point log is full of wanring and errors like: clearing the logs and trying to update (clicking the update button) gives lots more errors: 2021-03-23 11:34:45 |
I will double-check with MariaDB tonight |
Hum, errors like cd ./FreshRSS/
sudo chown -R :www-data . && sudo chmod -R g+r . && sudo chmod -R g+w ./data/ |
* Minor auto-update in version 1.10.0 (3 years old) FreshRSS#1750 * Related to FreshRSS#3552 * Auto-update for temporary table in version 1.7.0 () FreshRSS#1470 (4 years old)
And here is a fix for the dead-lock you see with MariaDB #3559 |
thx i'll do a few tests. Keep in mind that i don't have full access to that server. It's a shared hosting with cPanel and softaculous so to setup FreshRSS I have a menu like this: https://www.softaculous.com/apps/rss/FreshRSS |
Thanks for the fix #3559 - I updated that file too. One small issue remains when there are duplicate feeds (maybe because one is a redirect) 2021-03-24 07:40:54 in my original tt-rss exported opml file the 2 entries are:
(i manually deleted one to fix the issue. I don't know if you can easily detect duplicates like theese) |
@marrco Did you end up with one or two feeds in your FreshRSS? The error in the logs will remain, but the handling of such cases has already been implemented #3347 |
@Alkarex 2 feeds, but i'm trying to keep my error log clean of errors, so I decided to keep just one of those. This it the tail of my php error log : [24-Mar-2021 07:11:54 UTC] PHP Warning: include(): Failed opening '/home/[munged]/reader.[munged]e.biz/app/Models/CategoryDAO.php' for inclusion (include_path='.:/opt/cpanel/ea-php74/root/usr/share/pear') in /home/[munged]/reader.[munged]e.biz/lib/lib_rss.php on line 38
hth |
@marrco It looks like you migth have deleted or changed the access rights of |
well, I guess that happened this morning while I was updating that file. so far I can confirm no residual issues or error in my logs. Fresh RSS is running fine with my imported feeds. thanks a lot Alkarex ! |
Nice to hear :-) |
@Alkarex I purged my old tt-rss setup last week. Sorry, I can't do that test |
Please consider reopening. I am still seeing these on 1.18.1, FYI:
|
@posita Which database are you using? Are you importing on a newly installed FreshRSS instance? Could you please provide an example of OPML? |
I noticed this after trying to delete and re-add the feed https://www.theepochtimes.com/feed . I think I had installed at 1.18.0 and upgraded (in-app) to 1.18.1. SHOW VARIABLES LIKE '%version%'
admin_tls_version TLSv1,TLSv1.1,TLSv1.2,TLSv1.3
immediate_server_version 999999
innodb_version 8.0.26
original_server_version 999999
protocol_version 10
replica_type_conversions
slave_type_conversions
tls_version TLSv1,TLSv1.1,TLSv1.2,TLSv1.3
version 8.0.26-0ubuntu0.20.04.2
version_comment (Ubuntu)
version_compile_machine x86_64
version_compile_os Linux
version_compile_zlib 1.2.11 |
Describe the bug
Newly installed 1.80 via cPanel, imported my (less than 100) feeds i prevoiusly exported via an outdated (cPanel) tt-rss instance
feeds don't automatically update. Single feeds when forced update do show new items. Trying to force a complete feed refresh doesn't work
looking at logs there is an error:
2021-03-22 10:23:59
SQL error updateTTL 2: Duplicate column name 'ttl' ALTER TABLE
_feed
ADD COLUMN ttl INT NOT NULL DEFAULT 02021-03-22 10:23:59
SQL warning updateTTL 1: Deadlock found when trying to get lock; try restarting transaction UPDATE
_feed
SET ttl=:new_value WHERE ttl=:old_value2021-03-22 10:23:58
SQL error updateTTL 2: Duplicate column name 'ttl' ALTER TABLE
_feed
ADD COLUMN ttl INT NOT NULL DEFAULT 02021-03-22 10:23:58
SQL warning updateTTL 1: Deadlock found when trying to get lock; try restarting transaction UPDATE
_feed
SET ttl=:new_value WHERE ttl=:old_valueEnvironment information (please complete the following information):
cPanel
Additional context
I suspect there's an issue with updating imported tt-rss exported feeds.
The text was updated successfully, but these errors were encountered: