Bug 398358 - News widget cannot get news on Windows
Summary: News widget cannot get news on Windows
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: General (show other bugs)
Version: git master (please specify the git hash!)
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Halla Rempt
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-09-07 14:54 UTC by Scott Petrovic
Modified: 2019-04-13 23:40 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Scott Petrovic 2018-09-07 14:54:23 UTC
I just pulled the latest on master with the news widget on Windows. When I tick the checkbox to get the news, nothing is being returned.

When adding this line to the appendFeedData() function, the NetworkReply returns this error:

dbgKrita << "response error: " << QString::number(reply->error() );

- 301 (Protocol not found)


Talking with boud a bit, it seems like we might need to enable SSL on our Qt build
Comment 1 Jaime Torres 2018-09-14 15:09:13 UTC
Unfortunately, it is still not working in:

Krita
  Version: 4.2.0-pre-alpha (git 5db89dc)
Qt
  Version (compiled): 5.11.1
  Version (loaded): 5.11.1

after changing to Qt 5.11.1
Comment 2 Scott Petrovic 2018-09-14 16:10:36 UTC
@Jaime - did you actually build the modified Qt as part of the build instructions in the 3rd party folder. 

Can you see what error it is giving. You can see how I modified the source code to output the error. Maybe I can add a commit so the error code is easier to see by default
Comment 3 Halla Rempt 2018-09-14 17:35:44 UTC
I also need to build openssl now; I tried to do that, but my windows machine had memory problems, so I needed to repair that first... I want to get the deps builds updated before we release 4.1.2
Comment 4 Jaime Torres 2018-09-14 17:57:09 UTC
I didn't built krita myself (windows development is, after so many years, still a hell for me). I just used the nightly build from the CI.
Comment 5 Jaime Torres 2018-09-14 18:34:46 UTC
What I've seen using wireshark is that there is no connection at all when the news enabled check button is checked in the windows build, in linux it works at the first attempt.

I've also noticed that krita is not enabled as an app that can connect to the internet in the default windows firewall, but after enabling it twice (that's windows), it still doesn't work.
Comment 6 Scott Petrovic 2019-04-13 23:40:43 UTC
This was recently fixed by adding OpenSSL. It works now on the latest nightly builds for Windows... closing