Hacktoberfest 2026: le issue che i maintainer hanno segnato per ottobre, aperte e adatte ai principianti. Sfoglia le issue Hacktoberfest

Update smalot/pdfparser for PDFs with indirect /Encoding objects

Aperta
  • Qualcosa non va?
#7,706 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
2/5
Tempo stimato
1-3 ore
Idoneità per principianti
35/100
Tipo di issue
Bug
Chiarezza
Abbastanza chiara
Stato di attività
Ferma
Stack tecnologico
php
Ambito
backend

Direzione di ricerca

Inizia da 3rdparty/composer/smalot/pdfparser/src/Smalot/PdfParser/Font.php e verifica la versione o il commit di smalot/pdfparser vendorizzato. Confrontalo con l’issue upstream #822 e la pull request #823, quindi verifica l’aggiornamento della dipendenza con il caso fornito EncodingAsIndirectPDFObject.pdf. Il lavoro è completato quando la versione vendorizzata include il fix upstream e il caricamento del PDF non genera più un PHP 8 TypeError.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Bug Report

Note: This is not a bug in LibreSign itself, but a heads-up about a bug in a vendored dependency that affects LibreSign users.

Summary

When uploading certain PDF files (e.g. PDFs with Type1 fonts where is an indirect object reference), LibreSign throws a fatal error:

This originates in the vendored library at:

Root Cause

The PDF spec (Table 5.11) allows a font's entry to be either a name or an indirect reference to an encoding dictionary (). When it is an indirect reference, tried to cast the to string via , which throws a in PHP 8+ because has no .

The affected PDF is not corrupt — it is valid per the PDF specification.

Upstream Fix

This has been reported and fixed upstream in :

The fix adds a type-safe branch in that handles encoding dictionaries by reading the header entry instead of casting to string.

Request

Please update the vendored in to a version that includes the upstream fix (once merged), so that LibreSign users are no longer affected by this error.

Environment
  • Nextcloud version: 33.0.5.1
  • LibreSign vendored pdfparser commit:
  • PHP version: 8.x
  • Triggered by: Uploading a EncodingAsIndirectPDFObject.pdf PDF with Type1 fonts using indirect references
Nextcloud Log
Workaround

Until the vendored library is updated, the fix can be applied manually to:
3rdparty/composer/smalot/pdfparser/src/Smalot/PdfParser/Font.php

Replace the Encoding line in getDetails() with:

\$encoding = \$this->has('Encoding') ? \$this->get('Encoding') : null;
if (\$encoding instanceof PDFObject) {
    \$baseEncoding = \$encoding->getHeader()->get('BaseEncoding');
    \$details['Encoding'] = (\$baseEncoding instanceof Element && (string) \$baseEncoding !== '')
        ? (string) \$baseEncoding
        : 'Ansi';
} elseif (\$encoding instanceof Element) {
    \$details['Encoding'] = (string) \$encoding;
} else {
    \$details['Encoding'] = 'Ansi';
}
Lingua principale
PHP
Stelle
818
Fork
146
Merge medio
7h 38m
PR unite (30g)
490

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di LibreSign/libresign

Tutte le issue di LibreSign/libresign

Issue simili

Altre issue su PHP

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.