str_replace inside app/Utils/Traits/Pdf/PdfMaker.php break PDFs

[Striffly]

Setup

• Version: v5.8.35
• Environment: ZIP

Interface

• Flutter: [x]
• React: []
• Both: []

Describe the bug

Source of problem : db37b50

When generating a PDF (invoice/quotation), some resources hosted on localhost are not accessible. For example, I have some fonts and images that are only accessible on .localhost domains : they are displayed in the HTML preview but not when generating the PDF.

Also, this commit also affects the CSS: certain tags (iframe, etc.) are removed from the CSS that I include in a style tag in my templates, which breaks the design of my PDFs.

I'm not exactly sure in which situations this commit is useful, which is why I'm not suggesting deleting it via a PR. However, it seems important to me that the cases I mentioned be taken into account.

Steps To Reproduce

• Have a custom PDF template with some strings inside (iframe, localhost)
• Display bill or quote
• Generate a PDF

Expected Behavior

Keep all the tags inside my PDF : see my fonts & images, see all of my CSS

Additional context

Invoiceninja & my resources are hosted on a .localhost domain