From d52fbbb0409df15dd633a81fd871aaac08162321 Mon Sep 17 00:00:00 2001
From: Trenton H <holmes.trenton@gmail.com>
Date: Mon, 24 Oct 2022 13:16:14 -0700
Subject: [PATCH] More smoothly handle the case of a password protected PDF for
 barcodes

---
 src/documents/barcodes.py                     |  49 ++++++++++++++----
 .../tests/samples/password-is-test.pdf        | Bin 0 -> 8398 bytes
 src/documents/tests/test_barcodes.py          |  39 +++++++++++++-
 3 files changed, 76 insertions(+), 12 deletions(-)
 create mode 100755 src/documents/tests/samples/password-is-test.pdf

diff --git a/src/documents/barcodes.py b/src/documents/barcodes.py
index 13e78e1813b..1f5e33d376c 100644
--- a/src/documents/barcodes.py
+++ b/src/documents/barcodes.py
@@ -10,9 +10,12 @@
 import magic
 from django.conf import settings
 from pdf2image import convert_from_path
+from pdf2image.exceptions import PDFPageCountError
 from pikepdf import Page
+from pikepdf import PasswordError
 from pikepdf import Pdf
 from pikepdf import PdfImage
+from pikepdf.models.image import HifiPrintImageNotTranscodableError
 from PIL import Image
 from PIL import ImageSequence
 from pyzbar import pyzbar
@@ -120,7 +123,9 @@ def _pikepdf_barcode_scan(pdf_filepath: str):
                     pdfimage = PdfImage(page.images[image_key])
 
                     if "/CCITTFaxDecode" in pdfimage.filters:
-                        raise BarcodeImageFormatError()
+                        raise BarcodeImageFormatError(
+                            "Unable to decode CCITTFaxDecode images",
+                        )
 
                     # Not all images can be transcoded to a PIL image, which
                     # is what pyzbar expects to receive
@@ -132,7 +137,7 @@ def _pikepdf_barcode_scan(pdf_filepath: str):
                         separator_page_numbers.append(page_num)
 
     def _pdf2image_barcode_scan(pdf_filepath: str):
-        # use a temporary directory in case the file os too big to handle in memory
+        # use a temporary directory in case the file is too big to handle in memory
         with tempfile.TemporaryDirectory() as path:
             pages_from_path = convert_from_path(pdf_filepath, output_folder=path)
             for current_page_number, page in enumerate(pages_from_path):
@@ -150,20 +155,42 @@ def _pdf2image_barcode_scan(pdf_filepath: str):
         if mime_type == "image/tiff":
             pdf_filepath = convert_from_tiff_to_pdf(filepath)
 
+        # Chose the scanner
         if settings.CONSUMER_USE_LEGACY_DETECTION:
-            _pdf2image_barcode_scan(pdf_filepath)
+            logger.debug("Using pdf2image for barcodes")
+            scanner_function = _pdf2image_barcode_scan
         else:
-            try:
-                _pikepdf_barcode_scan(pdf_filepath)
-            except Exception as e:
+            logger.debug("Using pikepdf for barcodes")
+            scanner_function = _pikepdf_barcode_scan
 
-                logger.warning(
-                    f"Exception using pikepdf for barcodes,"
-                    f" falling back to pdf2image: {e}",
-                )
-                # Reset this incase pikepdf got part way through
+        # Run the scanner
+        try:
+            scanner_function(pdf_filepath)
+        # Neither method can handle password protected PDFs without it being
+        # provided.  Log it and continue
+        except (PasswordError, PDFPageCountError) as e:
+            logger.warning(
+                f"File is likely password protected, not splitting: {e}",
+            )
+        # Handle pikepdf related image decoding issues with a fallback
+        except (BarcodeImageFormatError, HifiPrintImageNotTranscodableError) as e:
+            logger.warning(
+                f"Falling back to pdf2image because: {e}",
+            )
+            try:
                 separator_page_numbers = []
                 _pdf2image_barcode_scan(pdf_filepath)
+            # This file is really borked, allow the consumption to continue
+            # but it may fail further on
+            except Exception as e:  # pragma: no cover
+                logger.warning(
+                    f"Exception during barcode scanning: {e}",
+                )
+        # We're not sure what happened, but allow the consumption to continue
+        except Exception as e:  # pragma: no cover
+            logger.warning(
+                f"Exception during barcode scanning: {e}",
+            )
 
     else:
         logger.warning(
diff --git a/src/documents/tests/samples/password-is-test.pdf b/src/documents/tests/samples/password-is-test.pdf
new file mode 100755
index 0000000000000000000000000000000000000000..b16b023c33cd3caa086db4f624ebd681a3dc62fb
GIT binary patch
literal 8398
zcma)i2UL^G);1jp9i<8y0R<FDFLVMSQl%<L5d#E>bO=q5CLIx}(tD92O+Wz=1p#R)
zNEN9fO*(>f|KL66o^$T~zW@Jctt9jA*|YcTJ<pzZt$A`$2aA({NFo6jTk~6Aw$`;~
z0iZxI(8=NsKvou{NpQ3w*#co?i5>_?v?mc<K{$IniGU?oI$06q<pFLaR|4Jv5Xd7T
z{Wf~wJdVi<XqL-({;QkkS=&BbpIL>E=u_qpx#Eh5Nz$6+mEjw_z7TE}y&EGJY_dNa
zd~{X}m2+NVW0aw?Wzs*N6X#tyoGgvd!1lM#^;)Vomz~-W(-TG~g{M-{ZPy7>%Z{9P
zjOKDKnTUAX6J-<hJ=CuBNHe*<F?cyd`O@Jiw%EapWKOckFTFaa7j$Yw13+-J`eTLU
z-bqk^lUPnV5Cr68^tTN0UlAk!mx%w!2m}ZLgOTXJ;+Of_;4;C`L8+dw6Z1h=UiJLO
zgIBry`m5g+)|l@rEU`_uZM6lx{<SrZ$_jo4xe8yt_aZRyXlS+e{kQ9{*57N+UQH^D
z39l2BtYR=#2(&ow8}fAT*Ed+e?K+o9m#z|9mte(T_>D4dLxzEDBBt|8LUb(XzNoHs
zDPcBN{<8z^M~<AU3TF=%aut(uxSk}OmcM7<`XVg3%Go+qbL(fm81Sb)SN^7&<L-zw
zSG4!9ke4VU_F3de&+Q2Y+pscB!nmiGvX`_BXg1rA=vqOcPfQeT28mNaL&8?*?qUhH
z>8LE}2Ry=*Mq|ysY9<0p&N}0D?(g|v5fd}K%QH4fLnc>Nx0Njk`kB*+AE{~`NLyML
zj49_1n`*nt3M}tIe)2f!%3ALCg0J5%Tx(VXh;(h6*eWRxZksiRO*bJZN|4vL6F^s^
z@;;Pj?|%hZa^}t9@AfvEN<WC^4Koc+$j*oQYR$$IO?(+AH_vcXo+_eYmsJnC>BvbC
zHWK<u8Shu0W<JtRPk^#YzxE*(h6>5^Zr)~Dl@n%euGdKERq6uJ-A*a^Vy3Zu+sWDS
z=b5JCgZ11?AM7Zf?P^jlr7W?v;y0)C_pMLMczPLCwQ%MM8ld;9{RUhER#P^AFgmTC
z=8|TPF^{06IP6_<Q)jemhAvm=wVl!2gWq|aw-sC;9s0ATutg@J@${;{%bjp%<1WRc
z7hk7T66&ZJIFiF@lr`0ci#+MeuclPAEp(he%5~BhxUcOtQdX3egiidrQKcI7#u?y!
zdNqXkLmwEGzgCo~!J%Dtg%kqpJ_cUg!-$k6M3(G78wuv&vK8FSRW>r9?GsQ)_i8%!
z);W79mdjaO_NohIkl7$Dq?jDt*Ed-p&^~M?d*j>I`#2K@c8|2$-4$NM#-~vu7V50-
z5O)qxnGu@)?aU?2gV%;RGfW*8Re5xiKA1HUx6@HgQT1X~444ahUU38Ywt8J_e#m&o
zRI2LpT1Wj-A$9~BPTU<dZ2Z`$?7}J#2@+TtHJfQPu^Ua^)h@E1I<|frqOy0tPitHk
zQagBT<f1F|`V=*%oGruHqpP-(tw7<#5Ieks(Ssjh`A(OmUk76vQ|upl-Ma0v^@#PE
zws#ugU9`DM<q#^7@wqU|YQL#XLlvhZ8&Bb?3N#m6Au;c*swJ_3FkZr4b$lG3t0;u#
zCS8+h@qXZ=Q6oKq&(iLjlr7M|+xqc&;;pl`_wf`##<i)-^mIw*Zq8)~l*Zo7ds58R
zVn#`<auA;E)I8YBnGj!o`8f5VoQdX5IpQZ_nSr67dgCbnq)ftH4GKy(xn$P~<(uIY
z>uL!VR7TtPKCwxEIVgELz6H97?x^%{IVYg=O_Hf9t?8$>Bane*WMbnshk-U_q@PY}
zIWxfdxe`_U@r@VRm#`MJa@R7nllx;=B^)rJ?5&fYp=IjKM|K}kwhnbahUzrT!_3W6
zZcDvIa4v@&#;=_HkO6ujqFz1z2GfhA$7opi^wveMm2$*>i^;OA&(AJ(=>ZbvyFjV;
zRIK$20p$t4^&J|%XFR4h4l^GcLKWN}$?Dxy|MG@@za(Q%|G8k2>J#K$q+qoOHO~#y
zAmmHh`eLc=b1N<L`;<8FhuPcdeBVVpSv_3dNeG0w=nJ>!rfXSHrfwtGcfTpE#-8~Z
zNs}|aRibFrbGr6RRh1d#=k>Q60tv0FIs=%bz4?kj`8_plIKBKMe`gs#@J15+EYA*~
zdp1gP%!^HEp~I?P>0Lp0F&(<*D|p6o=gTW(ujlhoB{Oy7D#(Ul>F{{tqeV&otz@*=
zLopVnh&lF(hh;Yz)RLJ(u;>+lFs^R2?nRRS{5g!WP+;7>Yd6Xf?3%jjR1mqD0AnpN
z@BVYuHfzA|Uq&E3)ZKejpGQ?sd4HzD%=*ety#cwb$9=hZX|k?kTY+%uEF#f-#l3cN
zXFy0;aLVe{NbsVN=u3)6;RYAJ(wkUpjD}OEd~^Gf3+dQAMh<7kIT)(Qegp8cc5(5r
z$H(k&-@ZsM-?Bd?Zdi7nlZA2Lr=Iyy_WNRtYVU`HE8orGGgIy0P>A8!k@HUujnC>Z
zj`GE%yU7XBkK<IA$H!Q$&#zZm9-~iPXY=v89ng6rNZgS1Fy3x(Sk%9SQ!;G3TH1Cg
zw&(Bw_|}{^P`iG#cV$iO@}47Yp^sxo&?&FoThui}(mL`kccgh@?e;not`BB`UED`M
za`s{hFBW>V)y1?3!IQS${902sj=M~g&pUZ7<J%!0_tZ77$tuKikmN&#s)X?25s{a9
zvCR!(e(7;T0a#`o=1a;~w+!{)TWX2U*zj@f031R9bhK?Unyjplx#WZb&!-PC`EA6Y
z;j=U6`R(Nl_}uXQ&(_#ug3SF2eNdlP5ihC)ms>z18!{@G4;jnzeOWP#k^S4!>P?p-
zG-&y+B+i|)l3w|$e*Jmd*s*5~RB8t^I)Y#R;*=CEfA>(4w_EkrftX1?zBtwP%lLEU
z)Z+x^;$tqanDW)wCKfYD0qL9jkBUL@p0dsc!DAJN_+uEL%0uwcV(;_Ek};zZH=5(`
zzr?2h%#4eRWR|STj+ErSuCuFea!yi!_0EO$;MW?oqd`$T{BMd}2&rW{y9buy4&JmZ
z6a1=K%Y#6|Sy_4A)BT8RnvV|r+oSG<ZEU-`e|At^+2)?c@}isHoV#(2#^0bQXR1y$
zQukU*@frn_Eb(z-S=*-BSPs`qxW5|b((xw~Z8wGN;%BN!Z*djod_L#nLslMdFIF6~
zc0*&H<eWNO@G`u!ox^g6LbjOG<B8jN@Tjx6N~BF<DSM-GoXX4e4ATNm6|v#;pkKl(
zncKX3*P}t5fK|TRnu<X;Fsr#L!(t<5vQqUHKOci6#z<r8iH0h+2}dK;wrJO}1K}%#
zS^fHvu*_y4$`|1fI4Z{wo33DBJE+u=k;R)*X{THGann~WlF`6*x!So6p2=Z@--h<C
zJ9*=zc@1^V==x97kE;6SS2HffWMn2tErb_ovk#jbor6~$JaEf~GP$|Q>{JRwId%MM
zjfN}j7t}*xO(i^Fm4Z6YgypxvLlKp&8Asmd*VI!2<@#N5Kc^rs70%)v=*}2lvI;|f
zXmV^jQ?r(xMlHVdOYr4-u^sKekpthR1g#6dApS{hrPmLUo^t6iz%8#=HNowPf~i7;
z%WO4Y#O<t(-(lzAzfzlZ(<vI3QH`4{pASCFmsi@?P_)-QKV|BtbeW3l<1?#+3Y<Sz
zNC5tGA%=a6{#jv`X^7(|q1}DS*Mgc@hJ-Rv`Q^J?bvMw|ylLOJLbU4b=t?O~7AzEP
zBDi$g3xiOin(lQT^z(pPZjb74g)kc9*|8#Sjx}$%{iq|xjA?shzj{W+L%}WVhRXxX
z50Uz>=<Mmh*GjH+O}yG&!oT1D<fRAWtCOeN&CPiKdVv!-kRqtP+!<LSe<>EC@+(AQ
zJS2pDpYh>qii1n}B$q`mo!ym}il?;?YI|OKxt=L^9Z6h%SCVQ9f3TudJXJ5CrsBn8
z?bqa7nRkw|;KnH@s>@kd()t5RD_aiMr*&k?0KGy<J^IAJLQ_71jqczNxcxKcLVLnD
zt0?rxm93(xeF*zTV;N{SH1!FtIn}t++h$9u=W3*7lE$I&PqmzYHe6i)*twc4WYT9e
zZTM24O-8;})6FY<9-BY4TJsivbuU&EuWy^2u?U?V(#?rAk3NicSj7>Nl$v^jBk@P^
zFkKf7z%koomxjkiepD<TZO8x}YI=#jZi2NIF6DT&j1XxBlPVNbGIe`7Ru`Ehh0I0j
zxmBu~dSk{~1B0U9i@bPMq;NGv$!;p(*4ZfHdx_yOW=jPtJ;tghpG{?Mkir7wLJwm@
zc)a<gX~AvBE=Bw~tRLfzcK1PF@7dj3QK%zOW!rA?o7X&(zOumGCmK76uW@aEx+;5X
zOtECT^_zQAdD}-yu>_N!l;9NE@bTUdiQ?l}3||2~wZtyaD0H@Hx6R@<ILCo=u=CPt
zI+e0SW4VCWJz&G>1p_uQ{UEWUO4B4YuAxfGv*}jW<7P+c+N(E2#Lo5YIms)Ba(aek
zY_;Cxz3BTZWXq<AVZK9JX^MG?{at$2+@qnnDre5-F30+GUGTf>9W`&ZEci0d(R7Pb
z`5*6!n12tF8+^9IC@*%k_(<8varDRfK*Ue)yl2w(_wIBK_3&ogx`4dHv;h@MWtjcJ
zA($3U^JPy1`tht$uQbHNU#!H*fAkq&av9{D#Ot_x|1AYdLL-mIWfKiVjf2zr+#Suw
z)*IU1@tc{;V^6}-+JQIjUk#R@<(3{=PStpFSGc)b=#vq19az!nicbP}+x_uN^6?6%
zBXnQGuY#awpwYRmD)Z8^tEMlF8u;lAKPbK*n6P`2ZdXy}7e4KtAh1r2HMXC=J`v>R
z{pQ8Mt*PR!(KL6Y_#67gnHA%$HIACj#|;kCy7B2z##^>?_?zF;01wjB6mIl}9hUCt
z4fMZaW?7gS9TUHlxDEBwzjd_W7Zw8Fkzptw8;&@$Ir1|jWykmcpdj6Pi129kPS_Q(
zx0-%dWVsN>{}`8V!!&6>QEJ>|B#XGJEw-zUOp|nwSWG`E*zB8+5+u>6eP)n(`xTnK
z9=z-#Avtx|z&Mb39`$aS;|*ff3~}*W=fnEWuSzfKQ3_8(+zuYA*5mCK)fKJnw!~sv
z-&YG*f4cPTvQTu*+A$5C60Adr(Jhnel3c0}vswSGj@>l@3e4AU14&s}?*jkzTAr+~
zp;ZwKt6!;=XPR0d^Cu^>JnE+Hx)?9^rO9T&kjW`v>;9$V`0|xU6hY>$VYRW5N}BYW
zRXC32ZWQii!bQW)QTj#aUu8T_n>(}8n-Z;>7g24oLU}3$>et)@Io)Hhdc$BOWd<k9
zfk$uX{6(oVFZxO;>Ysx%#M>?53~50O3Owz8`czyUS{;ct2K7;mdvP6Z^uwfry&;`=
zmlj~*Y7@Kh7ODRLbRp_Gby?YURA`cgW*2Xm=;x{per}C;L<zHEIzOCGA8~*GAz=3k
zhKlc-@IYgDX!qcFlu-?B2P-wmuL(lEOp}`_Zrees_3nfr(~FXRZIw>7WgR}%O;>bW
z(#%6>&wP4Z(dRg|eK$Q%qoV}%G}xW{m!?o;pGA#}Xn|ZqVIwo~wp(a=ZRd_Lm#f0Z
z{_%<7IMMLV*p<7p6}vcr0n?qVEM~+)gD$Bj?M*J)zo17ldVW}|MOV3X5lm6PdS+Bw
z7eXD6QD(RK>ie{3@j|)_q#W?Hr5O9LMUwg6*o)E3A0qfSYgezlO=bGdGs5bI`7o4J
z?e^$mhLFGG!(D17Oqz3iO1w{TNuhY_K3m9IrMiWL;oyXV?z*f=0#np7V32dQ(JNZv
z$*XfK$HRj8od;1b5ys~Jc@JPvTw$>Vr&Fr520wR`-bI>3U;LO;dlj{&dUu3pO-FMl
zbbI7l-AwwR4uLiAd27K@DQyj-bok_^@HvWy25y09*6QrzBg(VeEhgh5Slol_eXM9(
z<B<$phMh-z<2I!X;Q@xd_KW26d21NbwNgZGgv&R~r8}Ut4U1dz?aEk6u2VDn9(YS;
z(a%|@(&r4wLi+GRc+uqS`3|Dmt@P(%oO6jGpJ?1lzcc>gCADjaWqB6dvvheuaW<Gb
z`LZi<{KeDDr*CfWH=7!W-;=l-L%sya&d6czGMM3%B?|2)Sva-Ra@=x<DQS$-KVOr@
zD0#w&gF-cTz(Sbm_q?Boukfc#)t6Q~;59Ht2T$ueuZ-?_d2hD8@6^Y-^Ru8rpti~j
z0LUXm3SC_&0C1YI66-y6-Xs2YI3?vnYYLCLPm5NlC}KX^{@k=x70_Tky{s#3u<H@t
zFeeErQC96zN~MtMuNbgmJj1<)rsB(SOw7w4^igk(52x%7RW0ml@H`)~SC~8Nf;wIE
zgO~PMS=-%b8djmn8L2`~DCm+qRG$;)Vpc=$u{6454dBe}xGJg>>3n5YP6Y<pR_wRs
zCW|Mk8Gi1Hr?!lZ=+M^aB<eG^nP>}WF*Gadr?O^xS;c*DSr#n`teNNcTDv?&`K_Rn
zPc)QDV?x4KcHDxtvN^+{R;IjcMsD2!{xWDmccadzs#Ix9+y`K-P@>#gLHi*t5`MWQ
zQ}ao%cx9a`R~8R^r_d%TtV7A*+zX$I*KJY(N5y7-@5_^sWdZgmj_>SHi74DT-vGv$
z@9A}to=&ydc7dVe38KMlyZ8#aHws(1R!?6$Z^~B0D6nzT#&}-RjaK`j^Q$&<X~MLj
z^U2j+aWNMLtK~a)&xJ+xO~%A6{+jg`<S=`v_@qa~Q#NvVWd<-X|1fd_qQ-qezgk0l
zl~vwiAW;c%&3m}(mWW0ZeaARX)*>z1I-Ud6+^n9dd&AXTdU-l7dwr0Cf`(mGwD!eQ
zNB%JGuUD$A^&;#|M;TW{Un5dCj~<8SN||ew<XrmsoO%62l2vy`QXNyXag7t;H2*-<
z%s~&;z1vd@cdir{Y8buVq7V~Mi%3GyqL@Y}>MR>ty!SSyq#2!e?=O^_67CQzqrT0u
zRDyE%(vM4GB!%+|%&0HspBoa)zc~f$Dv{5;4+s$5du7vdIZN2hnEGkcMmKY^{do*2
zcVME=IJ^(OyWTYzzZI!y?p?3FQRt_)bX%)nw=3k`4Deo~<y~|S4b~aQeRn!QiZ`)A
zQFrO8w-(D@<_|6dCeFF-zT9U%hFJ#wV~-1Tv-@8_$7OM8gDEs{q5Gj=0Xfa;u)yS~
zJD())jDgj-;FQ-cxo;jAXJ;>;#Y_bn>yPOA=e$ZkQ!BkyN37odv0X8yBA-d~J;?lv
zkm!YuwHve@b0k89V@otoNkzXD=SMAtXm6YA#i{QfJXXrcfBE+K!#n)ZtpTyA6(i4Q
z67Sc%fc%a{d~comYAnY(D_H=ZCe7l3{UW634l)0Ll{pIM?uvISwz8dh4KLNIxh@PQ
z_wF!#vj3^&UZ7Q0gjf_*y6oZi39rRAHC8Nac=>y}ZN>Miho_UZy~ZI=O=q%MQjTn!
z!i8JxsfyKGA$}CEk%_9mri8LIx@M1z)zQ(SnA@=3E3H_|+xb2Y{N)Oirw#k&WORY^
zF8E`ywU8bACy!#v?^&l9##_g{(jWUqB*4y=k%k|s^Lj)(SI|jpa*ei|uJ^5gQdU?1
zZtp6gmp|2QQb^PJutFq{wa-Z9Vvq9VubC#9LVPPF){A*c=hFexB@7?c+=acmH`1kq
zG-r)^{3qTFtRL~Wcl2A8u@Bt2pTU!o`d0L2!*pd1xH_`FwP21kl<m>GK&z+SGZ^Vz
zxaVCRQT(ZZz4qF}B7qvI4A)__{>~#Yl9B%Vd#29$e9wzWm}+87(S+B9TNU;_ZUHYd
zBW6!CU#1``$}k~BwO(BW#%YA|d3MPdduz<bL=@fp%o*#9e|KeGI3jFI(Y{`8jmf|9
zj3xIIjpVue99Jlo93|EC!?%}IKYGC_8+u_j-MJ4oig7kX;N?r0SId_JZtCt7(k)P1
zhhwqOIQj%m_nH2sr9IJCRX;Y-clmZU-&{<9^d~H=^D+yC$qX!x$XKo}U4&hb&9ub(
z;V$%EdTV;6HmFo*jq=59ZSy3y$vN~M7jf%xKUKR-I!{KEVaQqF)$pB#{G`6iXJy>o
z4#tvf6s)F8b)M)7WlpEv0?kT^^dvXlRq2Us`W_tIUm$hVSiX0eHL{(aH@m<RYre-)
zzV^`|m^#r@?>RH@yGECGD{yjsvcHcmy~OY|^A}~#pwDA#t_mZ?RdzmrrNymKY3C^x
zBtihlMltq=Cj0^mn3cK#y)ovsi0X$;dw8j5PI@RGFFLwWoNsu@+?R9n9)I;tY_357
zLsitr`c49{mp}y*Zq2xE2ywc1lb&z-k<Z=N>2h0eHYWInfUsPbw;$m=A)%D!;uvjB
zgnGFV0c3RMbW8K1enwwtI!)%Sjaua-Oi80Ng)iE2>iivnnV-w2hHvg_r(jfv7wG?o
z+x?S!`;#kuBp8hN_w_#Nuj_pSZ)XAs=j2Gj65K3ZiOwV^GND0!yoPrmfRr#N?TRMR
zg5ZiL5uF_M39dwIzzO4F;|7G2J7i5IB_}Tv2?QJllz@R@KnN6t0!pDFW*{{Z-kxZQ
zakQ}~0Kotd#?6x8NIL1AxcZ~3jCWQg5N&KpCsP0*eG<Xp1`tiwIk8OUR|rtx2^XOU
zATtYpk`w5^R{sYb14X0%uVhSGu2%cCqTF5OBFaU^W29KZICczMs_CP+j(%#r9r6$r
zTw>UL|Cvl!^tRSIb;GtE6YSz)=s=X)w%y}&>(yaF>E@!~@_?2YSH?k5?S7gU<rzW(
zTk}NKgZV<02ZnoZSW|@J=TT}S+~W_%2DFM*EUCn&^*dySp{+E5FWQ^bayvadc;61n
zrLgz?l<mt`dJ1mdM^EPm{AB5xwDaCpP0<K@F_&9-(A-ynJ1uJAU^mWoWg9^W>@IHC
ztE9{Y#ur6YG6^l$ztMflwIXj$JVkNT_xQ8V53ki!U-5$(#-S`3%`7;L$NE*zA~3K&
ztGr#68=krvQ^p36Kh=ryJpM9ORO%I*hZR@B!ClS6kP$OO8+o#Q-G`#z8hwlZ2i!>g
zmE?az1f=h7K{~;UfvY>=w^Rx5MmX82{~akrS2vQfE#CEH-!<`n^dV&E7!s{Wwr(b1
zAOe8|!pZ1?p{0N*D0#oZU?2<z`~3?=fPp7?IvGFt{2nLkpUBPr1(LsT3p|0f9sp$E
zbls77!o~qlkn#`cK)`>6g@XW3g8b(=<Zt8TxX7&5ZxxV^tCOWZfn)+Avx6W5f)~l`
z?}+@{tlw$T!P^i(I(XL;ki$<5fb<A%PVTOj1UDe$L|G4{MX(~`Pb%lc0YVBPNv<6!
z7zzwVp;2%k8V->}qoq(t7z!nYfTGPn%H-9M&A9<j$Vk1vRl|w>lL9A`lmGVZ2KX<B
zfWJJFC;Wf7Jjt&H(aOyP_<O-8kIGK&Cn@=xbI9MQ(cj*c@#La+vLOd=Lyq-#>;RCq
zGr<vKd2)W3{L2=|=%z8y(cRwuUteIdZB4wRji_}`%Py<fKQj9#J4p^tnM|ybk0)6v
z3`#`>jKsj97#J3-jDf1ak;*6}6sL@Z%L7if%F5l6@TVRM0*5N0$$Aj79{68+DpFvG
ziWCGY1;#;fN+<{z4ncvH;ZU3uLJFy*gn^)yq+n>Wt=}%m>&B89UQu9Uzp1P9*X4MA
zx*hrVj}fn=+FNTehG4P3C**J2@drJuPr^G|;a#mjH-OMT`C|wa4blU`K-!`WpZgZ9
zl!yr;><;o~dM+;ma=p$R-U`^|TwC3hLTH+cfv$@xhFjTSWphNvS)bTyw&)#E{6`Z5
z=>R1lVCV_``sX3(<w~#yfPoMM0Q~O;L?RF{1kf7zhm2fJWGI|8pyQu1Fc?CvivN&7
z;pBb$rwj#ylSBQd3=Bp7mmU&^_#b2_GH(7k7l}arw@m84*MfqOkB0x4i-I7@FNuHJ
zL&1^%6$c7|B)<;-Lk|W2n_rSEp8U>nJt-!A;#~qceIPw2Co<+wa85p2R2(f`y`9O=
ze;`}U(b@?J`TcJI07wlBGy%$zbEO0Z!*EC(42s35U|?vZ5(<Y@mV)Cj2)K%pJmCMW
za#CzqCrf2pf~B3Cy8{rU0#kykz`$h7QYaio38Ms&f=MZ1aS$jL0f&QeCne)X!n=}w
RZ#(%2kpf)2sG^Mn{2zuYu9^S<

literal 0
HcmV?d00001

diff --git a/src/documents/tests/test_barcodes.py b/src/documents/tests/test_barcodes.py
index 1c4ab7cc34f..f4e5fce14f0 100644
--- a/src/documents/tests/test_barcodes.py
+++ b/src/documents/tests/test_barcodes.py
@@ -174,7 +174,7 @@ def test_scan_file_for_separating_barcodes(self):
         self.assertEqual(pdf_file, test_file)
         self.assertListEqual(separator_page_numbers, [0])
 
-    def test_scan_file_for_separating_barcodes2(self):
+    def test_scan_file_for_separating_barcodes_none_present(self):
         test_file = os.path.join(self.SAMPLE_DIR, "simple.pdf")
         pdf_file, separator_page_numbers = barcodes.scan_file_for_separating_barcodes(
             test_file,
@@ -585,3 +585,40 @@ def test_consume_barcode_supported_no_extension_file(self):
 
         with mock.patch("documents.tasks.async_to_sync"):
             self.assertEqual(tasks.consume_file(dst), "File successfully split")
+
+    def test_scan_file_for_separating_barcodes_password_pikepdf(self):
+        """
+        GIVEN:
+            - Password protected PDF
+            - pikepdf based scanning
+        WHEN:
+            - File is scanned for barcode
+        THEN:
+            - Scanning handle the exception without exception
+        """
+        test_file = os.path.join(self.SAMPLE_DIR, "password-is-test.pdf")
+        pdf_file, separator_page_numbers = barcodes.scan_file_for_separating_barcodes(
+            test_file,
+        )
+
+        self.assertEqual(pdf_file, test_file)
+        self.assertListEqual(separator_page_numbers, [])
+
+    @override_settings(CONSUMER_USE_LEGACY_DETECTION=True)
+    def test_scan_file_for_separating_barcodes_password_pdf2image(self):
+        """
+        GIVEN:
+            - Password protected PDF
+            - pdf2image based scanning
+        WHEN:
+            - File is scanned for barcode
+        THEN:
+            - Scanning handle the exception without exception
+        """
+        test_file = os.path.join(self.SAMPLE_DIR, "password-is-test.pdf")
+        pdf_file, separator_page_numbers = barcodes.scan_file_for_separating_barcodes(
+            test_file,
+        )
+
+        self.assertEqual(pdf_file, test_file)
+        self.assertListEqual(separator_page_numbers, [])