Skip to content

Scripts to detect use of draft PIDs and other not found errors#26

Merged
pdurbin merged 4 commits intomainfrom
pidchecks
Jul 16, 2025
Merged

Scripts to detect use of draft PIDs and other not found errors#26
pdurbin merged 4 commits intomainfrom
pidchecks

Conversation

@qqmyers
Copy link
Copy Markdown
Member

@qqmyers qqmyers commented Jul 2, 2025

Per the README - these are scripts created at QDR to detect situations where someone has posted a draft DOI that won't resolve or have copied the direct dataset URL for a draft dataset to somewhere public. They also detect situations where a corrupted link may have been posted - with a period appended at the end of the URL or where double encoding of slashes has occurred, etc. The scripts are intended to be run ~monthly via cron and can log the info and send email to specified admin email accounts.

Notes:

@qqmyers qqmyers marked this pull request as ready for review July 2, 2025 20:34
@pdurbin pdurbin moved this to Ready for Review ⏩ in IQSS Dataverse Project Jul 14, 2025
@pdurbin pdurbin moved this from Ready for Review ⏩ to In Review 🔎 in IQSS Dataverse Project Jul 14, 2025
@pdurbin pdurbin self-assigned this Jul 14, 2025
Copy link
Copy Markdown
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't run these scripts but they seem useful. I left a little bit of feedback..

Comment thread python/pid_reports/README.md Outdated

### Configuration

Before using these scripts, you need to configure several variables in each script:
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An obvious future enhancement would be to extract these settings into a config file.

Comment thread python/pid_reports/README.md Outdated
qqmyers and others added 3 commits July 16, 2025 15:37
Co-authored-by: Philip Durbin <philip_durbin@harvard.edu>
@qqmyers qqmyers removed their assignment Jul 16, 2025
Copy link
Copy Markdown
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As discussed with @qqmyers I'm merging this without testing.

We want to get these scripts out to the community. If you notice any problems, please open an issue!

@github-project-automation github-project-automation Bot moved this from In Review 🔎 to Ready for QA ⏩ in IQSS Dataverse Project Jul 16, 2025
@pdurbin pdurbin merged commit 7cc13c1 into main Jul 16, 2025
@github-project-automation github-project-automation Bot moved this from Ready for QA ⏩ to Merged 🚀 in IQSS Dataverse Project Jul 16, 2025
@pdurbin pdurbin deleted the pidchecks branch July 16, 2025 20:02
@pdurbin pdurbin removed their assignment Jul 16, 2025
@scolapasta scolapasta moved this from Merged 🚀 to Done 🧹 in IQSS Dataverse Project Jul 17, 2025
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

Successfully merging this pull request may close these issues.

3 participants