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

[F-Droid Nightly Repo] IndexV1 has faulty field(s) #132

Open
machiav3lli opened this issue Oct 20, 2024 · 7 comments
Open

[F-Droid Nightly Repo] IndexV1 has faulty field(s) #132

machiav3lli opened this issue Oct 20, 2024 · 7 comments

Comments

@machiav3lli
Copy link

https://raw.githubusercontent.com/onionshare/onionshare-android-nightly/master/fdroid/repo/index-v1.json

The published JAR/JSON for the nightly repo seems to mix IndexV1 with IndexV2 Schemas. More specifically repo.mirrors seems to be a array of objects (which is true for IndexV2) instead of strings.

"mirrors": [{"isPrimary": true, "url": "https://raw.githubusercontent.com/onionshare/onionshare-android-nightly/master/fdroid/repo"}]

Expected:

"mirrors": ["https://raw.githubusercontent.com/onionshare/onionshare-android-nightly/master/fdroid/repo"]

@eighthave
Copy link

Looks like a bug in fdroidserver, which version of fdroidserver was used to generate this?

@eighthave
Copy link

I'm not seeing this in other repos:

So I'm guessing you're running on old version of fdroidserver

@grote
Copy link
Collaborator

grote commented Oct 21, 2024

This uses fdroid/docker-executable-fdroidserver:latest which pulls in master branch of fdroidserver: https://gitlab.com/fdroid/docker-executable-fdroidserver/-/blob/fa8033e738261b976d1c0e7839152fe0f1ee839c/Dockerfile#L8

@grote
Copy link
Collaborator

grote commented Oct 21, 2024

@eighthave I rebuilt fdroid/docker-executable-fdroidserver:latest and ran the nightly publish again. Same result.

I don't know how you build the guardianproject repo, but fdroidclient-nightly is built with fdroidserver from debian, so probably not the latest version.

@grote
Copy link
Collaborator

grote commented Oct 21, 2024

@machiav3lli thanks for reporting. I'll leave this open to track the fix for this project, but ultimately, this needs to get fixed in and thus reported for fdroidserver as that's the software creating the repo.

@grote
Copy link
Collaborator

grote commented Oct 21, 2024

After updating to latest fdroidserver deploy fails with obscure error. I'll disable publishing to nightly until someone fixes this.

@machiav3lli
Copy link
Author

If interesting for you (for client/library), here's my custom serializer to cover both formats for now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants