-
Notifications
You must be signed in to change notification settings - Fork 43
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
403/404 from update cache if mod.io mod deleted #126
Comments
Also need to improve the error reporting when this situation does occur. |
I have been running into this issue for the last several days and I'm wondering if it is an issue with a moved resource or something else, like a rate limit from Mod.io. |
Note that the error message isn't particularly helpful here -- the last request might not necessarily be the one causing the 404 error (which is what I meant by the error message needing to be improved). Could you please send the mod list that triggers the 404 error? |
I opened up each mod's page, by clicking on it from mod integration, and found that one of the mods had been deleted. After removing that mod, the 404's stopped. It looks like I was incorrect, but the error messages weren't helpful in identifying the issue. |
We improved the error reporting on master, which will tell you (in a future release) that
|
So for mod.io-specific caching, we basically need to treat mod.io as unreliable: mods and their transitive closure of dependencies can be renamed, yanked or hidden. We would need to devise a scheme that aims to maintains the integrity of the local cache in a way such that updating the cache can fail, but it should not block installing what's present in the cache. Failing to update the cache for any mod in particular also should not prevent other mods from being updated. The core logic probably looks something like "a mod.io mod should only be updated iff mods in its transitive closure can be updated". |
Same problem here, after updating cache this error appeared:
I had to dig through the mod URLs to find out which mod was removed or hidden. In the end, removing the mod and updating the cache fixed the problem. I would suggest a change to:
Another solution is moving these mods to a new directory where they wouldn't be launched or installed, but they would still be listed in the app. |
There seems to also be an issue that mod.io just will throw a 404 even for mods that exist. Perhaps there is a server side issue, or more rate limiting? ERROR drg_mod_integration::gui::message: modio::Error { |
@MrLint i believe that message shows up with hidden mods. I have seen it before. |
We have a bug in our caching logic that if a mod gets renamed/deleted/hidden on mod.io then the API endpoint will return 403/404 which prevents the cache from being updated, i.e. bricking mint. This is less than ideal, we should properly invalidate the relevant cache entries upon receiving such responses.
Current Workaround
The text was updated successfully, but these errors were encountered: