8000 Submission date extraction failure is not reported well (and date-guessing heuristics are delicate) · Issue #8434 · ietf-tools/datatracker · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Submission date extraction failure is not reported well (and date-guessing heuristics are delicate) #8434

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

Closed
1 task done
jennifer-richards opened this issue Jan 18, 2025 · 0 comments · Fixed by #8576
Closed
1 task done
Assignees
Labels
accepted bug Something isn't working component: submit/ contrib-medium Medium dev difficulty for contribution minor

Comments

@jennifer-richards
Copy link
Member

Describe the issue

If a user submits an XML draft with a missing or incomplete date, the submission tool tries to fill in a reasonable guess. The rule is roughly

  1. Use year if provided, otherwise assume the current year
  2. Use month if provided, otherwise if the year is the current year, assume the current month
  3. ... similar stuff to guess a day ...

There's a poorly handled corner case here: if a user provides a year but not a month / day, the date is filled in but only if the year is the current year. If it's any other year, the month and day are left as None. This leads to an unhandled exception that is logged as a generic "A system error occurred" type message in the submission's history.

This appears to have come up for at least one and maybe a couple users following 2025-01-01. I don't have their draft source, but I'm guessing they were using "2024" as the date in their drafts.

To resolve this, we must at least raise a SubmissionError in the date extraction code for the case that we're unable to extract a date. If we raise this with a message that indicates that the problem was in the date element, then it will be reported to the submitter who will then have a useful nudge toward fixing their problem.

We might also consider improving the heuristics to look at time intervals rather than calendar years/months for deciding whether it's reasonable to assume "today" or "this month" as the date. That's a bigger can of worms, though.

Code of Conduct

@jennifer-richards jennifer-richards added accepted bug Something isn't working component: submit/ contrib-medium Medium dev difficulty for contribution minor labels Jan 18, 2025
@jennifer-richards jennifer-richards self-assigned this Feb 21, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 1, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accepted bug Something isn't working component: submit/ contrib-medium Medium dev difficulty for contribution minor
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant
0