grobidOrg / grobid

39%

master: 39%

LAST BUILD ON BRANCH bugfix/fix-149

branch: bugfix/fix-149

CHANGE BRANCH

x

Reset

Sync Branches

• bugfix/fix-149
• 0.9.0
• bugfix/add-priority-content-type
• bugfix/avoid-IOBE-in-orcid-search
• bugfix/codespell-round-two
• bugfix/correct-git-revision
• bugfix/docker-arm
• bugfix/docker-full-build
• bugfix/fix-15
• bugfix/fix-671
• bugfix/fix-728
• bugfix/fix-849
• bugfix/fix-classification-delft
• bugfix/fix-doi-search
• bugfix/improve-regexes
• bugfix/issue-1024
• bugfix/issue-1355
• bugfix/issue-271
• bugfix/issue-465
• bugfix/issue-512
• bugfix/issue-716
• bugfix/issue-849
• bugfix/issue-920
• bugfix/jep-env-uv
• bugfix/model-selection-flavours
• bugfix/patch-lmdb
• bugfix/reference-segmenter-createTraining
• bugfix/sentence-segmentation-detection
• bugfix/spotless
• bugfix/support-missing-dir
• bugfixes/docker-retag
• bugix/correct-revision
• chore/remove-powermock
• chore/remove-unused-models
• ci/reduce-coveralls
• ci/support-forks
• display-version-in-ui
• feature/add-api-status-in-ui
• feature/add-code-cleaning
• feature/add-ignore-areas-api
• feature/add-pdf-tei-editor
• feature/auto-update-hf-space
• feature/blingfire
• feature/code-formatting
• feature/codeql
• feature/codespell
• feature/coi-ac
• feature/coi-ac-update-light-models
• feature/community-page
• feature/development-api
• feature/docker-summary
• feature/documentation
• feature/extract-any-url-in-fulltext
• feature/fix-bibtex-index
• feature/improve-documentation
• feature/issue-to-documentation
• feature/literature-law-flavour
• feature/literature-law-flavour-eval
• feature/literature-law-flavour-experiment
• feature/literature-law-flavour-experiments
• feature/middle-name-bibtex
• feature/onnx-models
• feature/onnx-models-perf
• feature/onnx-models-perfs
• feature/otlp-metrics-push
• feature/prepare-release
• feature/refactor-tei-formatter
• feature/refactor-trainers
• feature/remove-unused-lexicons
• feature/rename-ui-items
• feature/revise-consolidation
• feature/rewrite-overloaded
• feature/sync-crf-manual-build
• feature/unpin-lmdb-delft
• feature/update-benchmarks
• feature/update-copyright
• feature/update-delft
• feature/update-dependencies
• feature/update-doc-flavor-dev
• feature/update-editorconfig
• feature/update-lingua
• feture/update-documentation
• fix/pdfalto-memory-limit-macos
• hotfix/0.9.0-docker
• improvement/affiliation-author
• improvement/fix-lexicon
• improvements/fails-on-failures
• improvements/speed-up-ci
• master
• multi-arch-docker-image
• publish/0.9.0-central
• release/0.9.0
• rootless
• security/cmdinjection-threadleak
• security/jline-cmdinjection-threadleak
• security/update-dependencies
• tensorflow-2.16
• tensorflow-2.16-docker-full

Build # 27977359529

Build Type

Pull #1459

github

Committed by lfoppiano

Commit Message

fix: guard against out-of-range page index in citation annotation #149

CitationsVisualizer.annotatePage() called getPages().get(pageNum) without
bounds checking. When PDFBox fails to extract the page tree (malformed PDF
or empty page catalog), this threw IndexOutOfBoundsException which surfaced
as an HTTP 500 from the annotation endpoint. Skip the annotation gracefully
and log a warning instead.

Pull Request Pull Request #1459: Fix #149

Coverage Stats

8636 of 24854 branches covered (34.75%)

Branch coverage included in aggregate %.

18386 of 45182 relevant lines covered (40.69%)

4.94 hits per line