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

Curseforge: Invalid file reference in manifest #875

Closed
2 tasks done
CorneliusCornbread opened this issue Feb 22, 2024 · 12 comments
Closed
2 tasks done

Curseforge: Invalid file reference in manifest #875

CorneliusCornbread opened this issue Feb 22, 2024 · 12 comments
Labels
bug Confirmed bugs or reports that are very likely to be bugs. high-priority High priority to be addressed.

Comments

@CorneliusCornbread
Copy link

What happened?

For some reason, maybe due to a mod being present on both CF and modrinth, when exporting a pack to curseforge it will contain duplicate references to mods in its manifest. This causes curseforge to get really upset when uploading a modpack and reject the pack.

For example

Invalid file reference in manifest: File ID 4725669 () appears more than once.

Operating System

Linux

Relevant log output

No response

Additional Information

No response

Is this actually a launcher bug?

  • I am sure this is a bug with the launcher and not an issue with running Minecraft

Code of Conduct

  • I agree to follow this project's Code of Conduct
@CorneliusCornbread CorneliusCornbread added bug Confirmed bugs or reports that are very likely to be bugs. triage Issue has not been looked at to classify and validate. labels Feb 22, 2024
@RyanTheAllmighty
Copy link
Member

RyanTheAllmighty commented Feb 22, 2024

Can you please share 2 things, the instance.json for the instance, and the export zip file you're using to upload to CurseForge that's being rejected.

If this is a private pack you don't want to share publicly, please feel free to DM me on Discord.

@RyanTheAllmighty RyanTheAllmighty added the more-information-needed More information needs to be collected about these problems or feature requests. label Feb 22, 2024
@CorneliusCornbread
Copy link
Author

Can you please share 2 things, the instance.json for the instance, and the export zip file you're using to upload to CurseForge that's being rejected.

If this is a private pack you don't want to share publicly, please feel free to DM me on Discord.

The file is unfortunately a bit too big for GH, it's uhh... quite a big pack with mods spread over both curseforge and modrinth. I'll send it over Discord, it's eventually going to be a public pack but for now I can't upload it :P

Here's the instance json
instance.json

@github-actions github-actions bot removed the more-information-needed More information needs to be collected about these problems or feature requests. label Feb 23, 2024
@RyanTheAllmighty
Copy link
Member

Can you please share 2 things, the instance.json for the instance, and the export zip file you're using to upload to CurseForge that's being rejected.
If this is a private pack you don't want to share publicly, please feel free to DM me on Discord.

The file is unfortunately a bit too big for GH, it's uhh... quite a big pack with mods spread over both curseforge and modrinth. I'll send it over Discord, it's eventually going to be a public pack but for now I can't upload it :P

Here's the instance json instance.json

Just the manifest.json that's inside the CurseForge zip should be enough

@CorneliusCornbread
Copy link
Author

Can you please share 2 things, the instance.json for the instance, and the export zip file you're using to upload to CurseForge that's being rejected.
If this is a private pack you don't want to share publicly, please feel free to DM me on Discord.

The file is unfortunately a bit too big for GH, it's uhh... quite a big pack with mods spread over both curseforge and modrinth. I'll send it over Discord, it's eventually going to be a public pack but for now I can't upload it :P
Here's the instance json instance.json

Just the manifest.json that's inside the CurseForge zip should be enough

Happy to oblige, this should be much easier to download then
manifest.json

@RyanTheAllmighty
Copy link
Member

Okay the issue is with Backpacked mod. It's in the instance.json twice which is odd.

But the export process just takes all mods in the json, makes sure it exists in the file system, then puts it in the manifest.json.

So it's not checking for duplicates, which it should be.

So there's 2 issues here to fix:

  • instance.json should never allow mods to be in there twice
  • When exporting to CurseForge (Modrinth probably also has this issue), don't duplicate file ids (maybe also warn if there's duplicate mod ids)

Will take a look and fix up this weekend

@RyanTheAllmighty RyanTheAllmighty added high-priority High priority to be addressed. and removed triage Issue has not been looked at to classify and validate. labels Feb 23, 2024
@RyanTheAllmighty RyanTheAllmighty added the awaiting-release Issues where the fix/feature has been merged and is awaiting a release. label Feb 23, 2024
@atlauncher-bot
Copy link
Contributor

This has been fixed with version 3.4.35.8

@atlauncher-bot atlauncher-bot removed the awaiting-release Issues where the fix/feature has been merged and is awaiting a release. label Feb 23, 2024
@CorneliusCornbread
Copy link
Author

CorneliusCornbread commented Feb 26, 2024

Unfortunately this seems to still be persisting, just without duplicate mods

Invalid manifest.json file:
References invalid fileIDs: fileID: 4632444, projectID 871698 (project-871698).
References file with invalid status: 4998424 (projectID 925913 - Rejected). 

I'll update this with the manifest soon

@CorneliusCornbread
Copy link
Author

Updated manifest
manifest.json

@CorneliusCornbread
Copy link
Author

I wonder if it has something to do with the fact that my preferred mod platform is modrinth? If for some reason that's making things behave oddly?

@CorneliusCornbread
Copy link
Author

@RyanTheAllmighty Should I open another issue for this?

@RyanTheAllmighty
Copy link
Member

RyanTheAllmighty commented Feb 27, 2024

So looking at it:

References invalid fileIDs: fileID: 4632444, projectID 871698 (project-871698).

This is pointing to create-structures-0.1.1-1.20.1-FORGE.jar which is marked as no longer available in CurseForge's api, and the project itself looks like it's been removed from CurseForge entirely.

My understanding is CurseForge should still allow deleted projects to appear in modpacks, so they don't break. It might be only for existing packs and they reject all new submissions.

I think currently we don't handle anything to do with deleted mods in the CurseForge api, because if they get deleted after the time that you've added them into the instance, we won't know they've been removed, so we can simply not export mods that have been deleted and add them to the overrides, but we'd now need to start scanning every mod you have on export.

Alternatively we don't do that, and fallback to getting users to rescan the metadata in the edit mods page for those mods.


References file with invalid status: 4998424 (projectID 925913 - Rejected).

Similar issue as above, but this time the file itself is deleted, not the project. Looks like they deleted the file you now reference from https://www.curseforge.com/minecraft/mc-mods/create-interactive

They have this message on their CurseForge page:

All new releases are on our website / Modrinth, please download them there:

So looks like they've moved to Modrinth

Pretty much same solution as above, just check if the files available before exporting it in the zip

I'll reopen this and get it fixed up tonight

RyanTheAllmighty added a commit that referenced this issue Feb 28, 2024
@RyanTheAllmighty RyanTheAllmighty added the awaiting-release Issues where the fix/feature has been merged and is awaiting a release. label Feb 28, 2024
@atlauncher-bot
Copy link
Contributor

This has been fixed with version 3.4.35.9

@atlauncher-bot atlauncher-bot removed the awaiting-release Issues where the fix/feature has been merged and is awaiting a release. label Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed bugs or reports that are very likely to be bugs. high-priority High priority to be addressed.
Development

No branches or pull requests

3 participants