Skip to content
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

404 error on Hylly map file download #11793

Closed
fm-sys opened this issue Oct 6, 2021 · 12 comments
Closed

404 error on Hylly map file download #11793

fm-sys opened this issue Oct 6, 2021 · 12 comments
Assignees
Labels
Bug Issues classified as a bug

Comments

@fm-sys
Copy link
Member

fm-sys commented Oct 6, 2021

Just a heads up, the internal downloader is returning a 404 error on the Hylly map file.

Originally posted by @Vike91 in #10511 (comment)

Side note by @fm-sys:
I can reproduce the issue

@fm-sys fm-sys added the Bug Issues classified as a bug label Oct 6, 2021
@moving-bits
Copy link
Member

I can reproduce the issue as well. Strange thing is: The URL still gets parsed correctly, and if I fire up a browser with that URL, download works. So why doesn't it work with the download manager?

If I download one of the theme files, coming from the same server, download manager does work as well.

@pailakka: Do you have access to the server logs, and if so, can you check them for some 404 errors while trying to retrieve the mtk_suomi.map file? (One of my tries was at 2021-10-08 21:35:33) Thanks!

@moving-bits
Copy link
Member

Did some retest with current master. Strange thing is: If I start a download of Mtk_suomi.map it will automatically start downloading the theme file Peruskartta.zip as well (which is intended). Downloading the theme file will fail with 404, but the map file download will continue... Strange.

@Vike91
Copy link

Vike91 commented Nov 1, 2021

Tried to reproduce with 2021.10.24-RC, looks like it has been fixed, as both the map file and the theme files were downloaded successfully via the c:geo map download manager.

@Lineflyer
Copy link
Member

@Vike91 @moving-bits Can this be closed then?

@Vike91
Copy link

Vike91 commented Nov 5, 2021

Tested downloading the map file with 2021.11.04-RC and now I'm getting the 404 error again.

@Vike91
Copy link

Vike91 commented Nov 5, 2021

Does @pailakka have a hint what might be going wrong?

@moving-bits
Copy link
Member

@Vike91 @moving-bits Can this be closed then?

As we have not changed anything in the code, neither before opening this issue nor since then, I guess the problem is somehow server-related, maybe a temporarily failing load balancer or some certificate issue (wild guesses). As mentioned in #11793 (comment) I would really like to get some hint from the server logs.

@moving-bits
Copy link
Member

Closing due to lack of feedback regarding the server-side. Feel free to reopen if we have more info.

@Kolli96
Copy link

Kolli96 commented Jan 17, 2024

The same problem still exists in the latest version as well. The phone is One Plus Nord 3 5G. So the problem is not solved! If you want to update the hylly.org map files, you currently have to download them via the browser and move them to the correct folder. If you try this with c:geo, you will get the 404 error message. The strange thing is that the same maps can be downloaded normally if c:geon is reinstalled.

@moving-bits moving-bits self-assigned this Jan 17, 2024
@moving-bits moving-bits reopened this Jan 17, 2024
@moving-bits
Copy link
Member

I've debugged a bit into this: Error comes from the parallel download of theme file "peruskartta.zip", which fails with a 404 error when being done while downloading a map file (but does work, when initiated manually).

Workaround is to download this theme once before downloading the map. (This can be done from the downloader.)
After that, downloading the map does work.

I've reopened the issue as a reminder to fix the underlying problem.

@Kolli96
Copy link

Kolli96 commented Jan 18, 2024

I've debugged a bit into this: Error comes from the parallel download of theme file "peruskartta.zip", which fails with a 404 error when being done while downloading a map file (but does work, when initiated manually).

Workaround is to download this theme once before downloading the map. (This can be done from the downloader.) After that, downloading the map does work.

I've reopened the issue as a reminder to fix the underlying problem.

The map download works if I start the download from the maps settings. (settings->map sources->download offline map->Hylly map.) If I start it from c:geo start screen (update offline data->map data), I will get the error message even if I first manually download the peruskartta.zip file. The problem is somehow related to this home screen map update function.

@moving-bits
Copy link
Member

Thanks @Kolli96 for the additional info, I'll look into this.
(My example was related to the initial download with neither map nor theme file being present yet.)

moving-bits added a commit to moving-bits/cgeo that referenced this issue Jan 20, 2024
moving-bits added a commit to moving-bits/cgeo that referenced this issue Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Issues classified as a bug
Projects
None yet
Development

No branches or pull requests

5 participants