v2/package endpoint does not match specification
See original GitHub issueDo you want to request a feature or report a bug?
Bug / question
What did you do?
I have started porting e2e tests from nuget.org. One of first things that fails is upload into v2/package
endpoint. I am a bit confused since in 3.0 specification it says that upload should be a multipart form. But actual nuget.org tests just stream the nupkg directly to endpoint.
@loic-sharma what do think, should BaGet accept multipart/form-data
or application/octet-stream
at v2/package
?
Currently when client from e2e tests is uploading, on server side we get null package and 400:
public async Task Upload(IFormFile package)
{
if (package == null)
{
HttpContext.Response.StatusCode = 400;
return;
}
Issue Analytics
- State:
- Created 5 years ago
- Reactions:1
- Comments:8 (6 by maintainers)
Top Results From Across the Web
Mention that nuget.org supports non-form-data for package ...
However, nuget.org doesn't require this (see the end-to-end tests) and allows ... v2/package endpoint does not match specification loic-sharma/BaGet#106.
Read more >Unofficial NuGet HTTP Protocol Reference
Introduction. This is unofficial documentation for both NuGet V2 and V3 HTTP package source protocol. A package source is either V2 or V3....
Read more >JR29492: NO DOCUMENTATION FOR MATCH ...
No documentation for match specification use The Release Notes on the publib.boulder site talks about a 'results database' for QualityStage yet nowhere in ......
Read more >v2
CiliumExternalWorkload is a Kubernetes Custom Resource that contains a specification for an external workload that can join the cluster. The ...
Read more >v2
Only endpoints in the upstream // cluster with metadata matching that set in metadata_match will be // considered. The filter name should be...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
I got this working with paket now. I haven’t opened a PR yet, but if you like to cherry-pick https://github.com/ai-traders/BaGet/commit/1826580843e1d0ba585627affed6dfc72e15a7d3
That’s not the only thing for paket to work though. You’ll need #104 too
Will do that. Same as nuget.org.