dandi / dandi/dandi-archive

`doi_configured` uses incorrect logic

Open
#2,576 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
26
Forks
21
Avg merge
4d 23h
Merged PRs (30d)
15

Description

Currently, we use this `doi_configured` function to check if we should perform DOI operations. However, this function simply checks if _any_ DOI settings are configured, not if they _all_ are. This can lead to errors, since if just a single DOI setting is provided, the application will attempt DOI operations, but will fail, since not all of the settings are configured.

https://github.com/dandi/dandi-archive/blob/f222f03af8ddc1e628fcb0422cabb69ea1c927c7/dandiapi/api/doi.py#L23-L24

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with dandiapi/api/doi.py at lines 23–24 and inspect how doi_configured determines whether DOI operations run. Confirm the behavior for partial and complete DOI settings; done means DOI operations are skipped unless all required settings are configured, preventing failures from partial configuration.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
api, backend
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Clearly specified
Newbie friendliness
55/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.