-
Notifications
You must be signed in to change notification settings - Fork 495
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
API: empty datasets can be created resulting in 502 errors #6752
Comments
Thanks @MYF95 for the report. We'd want the behavior to be that the dataset would not be created. |
Hi all, I guess this issue is related to #6739 (Multistore: If no files are uploaded to published dataset, export fails). |
Thanks @akio-sone. @MYF95 and the team usually work against demo.dataverse.org, which is running 4.19 and does not have have the multiple stores changes. I think this is unrelated, but could be wrong. |
|
I just came across this issue and I think this is (at least) related to my IQSS/dataverse-pm#26 |
these datasets for testing purposes have no required fields
Hi!
As written in the title, we can create "empty" datasets which in turn returns 502 when we try to access them via the dataverse page.
How to reproduce
The endpoint used is the one described here http://guides.dataverse.org/en/latest/api/native-api.html#id47
curl -H X-Dataverse-key:$dataverse-key-value https://demo.dataverse.org/api/dataverses/${dataverse}/datasets --upload-file example.json
The contents of
example.json
are as follows:Which is the bare minimum amount of data allowed to create the dataset.
Current behaviour
This results in the following dataset being created:
And consequently when trying to enter the dataset page
Expected behaviour
Dataset is not created in the first place because the required minimum information is not there
or
The dataset page is able to load with no initial information (maybe using some default values if not set?)
We are not really actively using the endpoint, but just wanted to let you guys know since I found this when testing some of our endpoints using dataverse.
The text was updated successfully, but these errors were encountered: