-
Notifications
You must be signed in to change notification settings - Fork 153
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
Some extension resource url return 404 or 500 #432
Comments
Seems 500 error is related to this issue #426 |
Yes, openvsx/server/src/main/java/org/eclipse/openvsx/adapter/VSCodeAdapter.java Lines 363 to 365 in f475efa
It looks like some extensions have a While for other extensions the I can add |
I replicated the 404 error for https://open-vsx.org/vscode/asset/viper-admin/viper/2.4.0/Microsoft.VisualStudio.Code.WebResources/extension/package.json The viper extension is not a web extension and therefore webresources aren't extracted from the vsix package. openvsx/server/src/main/java/org/eclipse/openvsx/ExtensionProcessor.java Lines 301 to 302 in f475efa
@jeanp413 Did the Open VSX API return this url in a JSON response or was the url constructed by another app (VS Code, Gitpod)? A fail-safe way to get the package.json: https://open-vsx.org/vscode/asset/viper-admin/viper/2.4.0/Microsoft.VisualStudio.Code.Manifest |
@amvanbaren the extension is returned as part of this query but it doesn't have web resources in the response, I searched for in MS marketplace and it also doesn't have web resources, so it seems vscode asumes all the extensions returned by this query are web extensions and openvsx is filtering incorrectly when doing a search
I see the viper extension response from openvsx contains this properties and the response from MS marketplace doesn't, if these properties are used when filtering, then this is causing the issue {
"key": "Microsoft.VisualStudio.Services.Branding.Color",
"value": "",
},
{
"key": "Microsoft.VisualStudio.Services.Branding.Theme",
"value": "",
}, I'm also seeing some missing resources like MS response for viper extension{
"results": [
{
"extensions": [
{
"publisher": {
"publisherId": "40c87fab-912c-4304-b2ee-b6c71e280a3c",
"publisherName": "viper-admin",
"displayName": "Chair of Programming Methodology - ETH Zurich",
"flags": "verified",
"domain": null,
"isDomainVerified": false
},
"extensionId": "20da0370-13e1-4393-8d10-4e90c9c9e6e2",
"extensionName": "viper",
"displayName": "Viper",
"flags": "validated, public",
"lastUpdated": "2022-02-28T12:00:56.653Z",
"publishedDate": "2018-10-14T20:51:27.093Z",
"releaseDate": "2018-10-14T20:51:27.093Z",
"shortDescription": "This extension provides interactive IDE features for verifying programs in Viper (Verification Infrastructure for Permission-based Reasoning).",
"versions": [
{
"version": "2.4.0",
"flags": "validated",
"lastUpdated": "2022-02-28T12:00:56.653Z",
"files": [
{
"assetType": "Microsoft.VisualStudio.Code.Manifest",
"source": "https://viper-admin.gallerycdn.vsassets.io/extensions/viper-admin/viper/2.4.0/1646049517014/Microsoft.VisualStudio.Code.Manifest"
},
{
"assetType": "Microsoft.VisualStudio.Services.Content.Details",
"source": "https://viper-admin.gallerycdn.vsassets.io/extensions/viper-admin/viper/2.4.0/1646049517014/Microsoft.VisualStudio.Services.Content.Details"
},
{
"assetType": "Microsoft.VisualStudio.Services.Content.License",
"source": "https://viper-admin.gallerycdn.vsassets.io/extensions/viper-admin/viper/2.4.0/1646049517014/Microsoft.VisualStudio.Services.Content.License"
},
{
"assetType": "Microsoft.VisualStudio.Services.Icons.Default",
"source": "https://viper-admin.gallerycdn.vsassets.io/extensions/viper-admin/viper/2.4.0/1646049517014/Microsoft.VisualStudio.Services.Icons.Default"
},
{
"assetType": "Microsoft.VisualStudio.Services.Icons.Small",
"source": "https://viper-admin.gallerycdn.vsassets.io/extensions/viper-admin/viper/2.4.0/1646049517014/Microsoft.VisualStudio.Services.Icons.Small"
},
{
"assetType": "Microsoft.VisualStudio.Services.VsixManifest",
"source": "https://viper-admin.gallerycdn.vsassets.io/extensions/viper-admin/viper/2.4.0/1646049517014/Microsoft.VisualStudio.Services.VsixManifest"
},
{
"assetType": "Microsoft.VisualStudio.Services.VSIXPackage",
"source": "https://viper-admin.gallerycdn.vsassets.io/extensions/viper-admin/viper/2.4.0/1646049517014/Microsoft.VisualStudio.Services.VSIXPackage"
}
],
"properties": [
{
"key": "Microsoft.VisualStudio.Services.Links.Getstarted",
"value": "https://github.com/viperproject/viper-ide.git"
},
{
"key": "Microsoft.VisualStudio.Services.Links.Support",
"value": "https://github.com/viperproject/viper-ide/issues"
},
{
"key": "Microsoft.VisualStudio.Services.Links.Learn",
"value": "https://github.com/viperproject/viper-ide#readme"
},
{
"key": "Microsoft.VisualStudio.Services.Links.Source",
"value": "https://github.com/viperproject/viper-ide.git"
},
{
"key": "Microsoft.VisualStudio.Services.Links.GitHub",
"value": "https://github.com/viperproject/viper-ide.git"
},
{
"key": "Microsoft.VisualStudio.Code.Engine",
"value": "^1.45.0"
},
{
"key": "Microsoft.VisualStudio.Services.GitHubFlavoredMarkdown",
"value": "true"
},
{
"key": "Microsoft.VisualStudio.Code.ExtensionDependencies",
"value": ""
},
{
"key": "Microsoft.VisualStudio.Code.ExtensionPack",
"value": ""
},
{
"key": "Microsoft.VisualStudio.Code.LocalizedLanguages",
"value": ""
},
{
"key": "Microsoft.VisualStudio.Code.ExtensionKind",
"value": "workspace"
}
],
"assetUri": "https://viper-admin.gallerycdn.vsassets.io/extensions/viper-admin/viper/2.4.0/1646049517014",
"fallbackAssetUri": "https://viper-admin.gallery.vsassets.io/_apis/public/gallery/publisher/viper-admin/extension/viper/2.4.0/assetbyname"
}
],
"categories": [
"Programming Languages",
"Snippets",
"Themes",
"Education"
],
"tags": [
"__ext_log",
"__ext_sil",
"__ext_vpr",
"automatic deduction",
"color-theme",
"formal methods",
"keybindings",
"log",
"separation logic",
"snippet",
"specification",
"theme",
"verification",
"viper"
],
"statistics": [
{
"statisticName": "install",
"value": 2131.0
},
{
"statisticName": "averagerating",
"value": 5.0
},
{
"statisticName": "ratingcount",
"value": 1.0
},
{
"statisticName": "trendingdaily",
"value": 0.0
},
{
"statisticName": "trendingmonthly",
"value": 4.178403755868545
},
{
"statisticName": "trendingweekly",
"value": 0.61032863849765262
},
{
"statisticName": "updateCount",
"value": 2188.0
},
{
"statisticName": "weightedRating",
"value": 4.4574588448266113
},
{
"statisticName": "downloadCount",
"value": 55.0
}
],
"deploymentType": 0
}
],
"pagingToken": null,
"resultMetadata": [
{
"metadataType": "ResultCount",
"metadataItems": [
{
"name": "TotalCount",
"count": 1
}
]
}
]
}
]
} |
The 404 errors are caused by the The MS VS Code uses |
I see, thanks for the investigation @amvanbaren 🙏. So this should still be fixed in openvsx right? as trying to access |
Yes, I've added a The main challenge is to write a database migration that re-processes all extension versions to get all resources from its vsix package. Flyway offers Java migrations besides SQL scripts, so it is possible. |
This package.json urls return 404
Also this theme json resources return error 500
This is the query to get the above extensions, this query is used in vscode when previewing themes directly from marketplace
Also from this result I found that the
publisher.displayname
is null, this is required for the quickpick list descriptionYou can also repro 404 errors when previewing product icon themes
cc @amvanbaren @filiptronicek
The text was updated successfully, but these errors were encountered: