search

ceskaexpedice / kramerius

System Kramerius
GNU General Public License v3.0
45 stars 27 forks source link

K7 / Pri generovaní PDF dochádza k chybe header is too large #1061

Open r2it-git opened 2 months ago

r2it-git commented 2 months ago

Ahoj,

v MZK sme narazili na bug pri generovaní PDF. Momentálne máme nastavený limit pre generovanie PDF 200 stránok. Keď je užívateľ neprihlásený, tak to funguje ako má. K chybe dochádza až keď sa prihlásený používateľ pokúsi vygenerovať PDF s cca 124 stránkami. Vtedy to padne na chybe: 30-Apr-2024 07:17:23.129 INFO [http-nio-8080-exec-2342] org.apache.coyote.http11.Http11Processor.service Error parsing HTTP request header Note: further occurrences of HTTP request parsing errors will be logged at DEBUG level. java.lang.IllegalArgumentException: Request header is too large Čo samozrejme dáva zmysel, keďže v takom prípade je header väčší o token.

Samozrejme tu potom nastáva otázka koľko stránok chceme maximálne povoliť pre generovanie PDF. @zabak

Róbert, MZK

vlahoda commented 2 months ago

Ahoj, mohl bys prosim odstranit ten komentar u te property s poctem stran? PDF ted nefunguje vubec...

image
honza-rychtar commented 2 months ago

Jo jo, taky na to teď koukám

{
"rightMsg": "Tento dokument není veřejně přístupný.",
"instance": {
"registr": "https://registr.digitalniknihovna.cz/library/mzk",
"acronym": "mzk"
},
"pdfMaxRange": "120 # Set back to 200 after solving Kramerius issue 1061",
"version": "7.0.35-dev",
"indexerVersion": 19,
"hash": "bb35a804941d1f473dcc60a71cd780184271529c"
}

pdfMaxRange musí být číslo.

Jinak úpravy pro generování PDF (těch problému je tam hodně a omezovat počet stran délkou url nebo velikosti hlaviček je fakt blbost) jsou součásti balíku změn týkající se PDF v Krameriovi, který jsme se s @pavelkocourek snažili už několikrát popsat na nabídnout, ...

r2it-git commented 2 months ago

Fixol som to.

Som si vedomý, že generovanie PDF tu má otvorené viacero issues a asi by to chcelo vyriešiť nejak rozumne, konkrétnym riešením pre všetky tieto nedostatky. Viac menej to tu len logujem ako known bug aby sa o tom vedelo.