-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[lostfilm] (testing) Exception (lostfilm): The string '12 февраля 2023' was not recognized as a valid DateTime. There is an unknown word starting at index '3'.: Parse error #14008
Comments
curious. I do not get this error. [edit] I'm running win10. [endedit] The indexer is set up to process the ru-RU language in the dates so this is confusing. |
found a matching ticket #9441 |
I can reproduce the error, but only on my Docker Jackett, not when trying the same release on Windows sandbox (using the same sitelink). Performing a keyword search results in row parsing errors: lostfilm: Error while parsing row '<tr>
<td class="placeholder"></td>
<td class="alpha">
<div class="haveseen-btn" title="Пометить серию как просмотренную" onclick="markEpisodeAsWatched(this);" data-episode="548002003" data-season="548002999" data-code="548-2-3"></div>
</td>
<td class="beta" onclick="goTo('/series/Your_Honor/season_2/episode_3/',false)" title="Перейти к серии">2 сезон 3 серия</td>
<td class="gamma" onclick="goTo('/series/Your_Honor/season_2/episode_3/',false)" title="Перейти к серии">
<div>
Часть 13<br>
<span class="gray-color2 small-text">Part Thirteen</span>
</div>
</td>
<td class="delta" onclick="goTo('/series/Your_Honor/season_2/episode_3/',false)" title="Перейти к серии">Ru: <span data-released="29.01.2023">29.01.2023</span><br><span class="gray-color2 small-text">Eng: 29.01.2023</span></td>
<td class="epsilon" onclick="goTo('/series/Your_Honor/season_2/episode_3/comments/',false)" title="Перейти к обсуждению серии">
<div class="comment-blue-box">
<div class="tail"></div>
36
</div>
</td>
<td class="epsilon" onclick="goTo('/series/Your_Honor/season_2/episode_3/',false)" title="Оценка серии пользователями Lostfilm.TV">
<div class="mark-green-box">8.9</div>
</td>
<td class="zeta">
<!-- <div class="external-btn" title="" onClick="PlayEpisode('548','2','3')"></div>-->
<div class="external-btn" title="" onclick="PlayEpisode('548002003')"></div>
</td>
<td class="placeholder"></td>
</tr>':
System.FormatException: String '29.01.2023' was not recognized as a valid DateTime.
at System.DateTime.Parse(String s, IFormatProvider provider)
at Jackett.Common.Indexers.LostFilm.DateFromEpisodeColumn(IElement dateColumn) in /home/vsts/work/1/s/src/Jackett.Common/Indexers/LostFilm.cs:line 821
at Jackett.Common.Indexers.LostFilm.FetchSeriesReleases(String url, TorznabQuery query, String filter) in /home/vsts/work/1/s/src/Jackett.Common/Indexers/LostFilm.cs:line 609 |
@mynameisbogdan maybe pinned down the issue to Docker or Linux? ^ |
@ilike2burnthing try running this: $ docker exec -it jackett apk add --no-cache --update musl musl-utils musl-locales tzdata |
Unfortunately it didn't work, but reverting to the last Ubuntu build (v0.20.3035-ls91) from 2 days ago did. I'll let them know their move to Alpine is causing an issue. |
Looks like it may be related to this dotnet/runtime#80542 (not the dotnet version per se but a change in Alpine with the ICU libs related to Russian language datetimes specifically). We'll have a look at what the options are for a fix, might require downgrading the base to 3.16. |
I also have this issue with Linuxserver/Jackett in Docker |
Many thanks. Please let us know what you decide. |
Looks like it's not a bug, it's an intentional change to ICU in version 72, see: https://unicode-org.atlassian.net/browse/CLDR-14032. So anything we do in terms of reverting to an older base is a temporary bandaid only, parsing will probably need fixing in Jackett to handle it going forward. That said I'll do a PR to drop Jackett down to 3.16 as a mitigation until something proper can be put in place. |
Could you test with:
And see if they work as expected. |
@thespad Neither one worked for me. |
Beat me by a few seconds. Same. |
How annoying. Could you try doing |
@thespad Worked for me. |
Both with the PR builds and the live ones? |
Tested live dev, working fine. Will give the PR builds a shot as well. |
Don't worry about the PR builds, if it works with live we can stick with 3.17 and just add the full ICU package. |
PRs are merged so give it 15-20 minutes for the builds and you should be good. |
Working well, thanks again. |
Have you checked our Troubleshooting page for your issue?
Is there already an issue for your problem?
Have you read our Contributing Guidelines?
Environment
Description
Press 'Test' button on lostfilm indexers, there is an error.
'12 февраля 2023' means '12 february 2023'. Seems date parsing is broken.
Logged Error Messages
Screenshots
No response
The text was updated successfully, but these errors were encountered: