Prevent issues cause by declaring utilsfile as readonly

[PromoFaux]

By submitting this pull request, I confirm the following:

• I have read and understood the contributors guide, as well as this entire template.
• I have made only one major change in my proposed changes.
• I have commented my proposed changes within the code.
• I have tested my proposed changes, and have included unit tests where possible.
• I am willing to help maintain this change if there are issues with it later.
• I give this submission freely and claim no ownership.
• It is compatible with the EUPL 1.2 license
• I have squashed any insignificant commits. (git rebase)

---

What does this PR aim to accomplish?:

Declaring it as a readonly variable, and then sourceing it causes issues when it is declared in multiple scripts.

Simples.

[yubiuser]

I'm not against this change but undecided if simply remove the readonly and source utils.sh whenever we need it would be the easier solution.

[PromoFaux]

That also works, and is much more simple. I was maybe trying to be too clever by only sourcing the utils script if it needs to be. Is there any danger in sourcing a script multiple times? I guess not with this particular script

[yubiuser]

Is there any danger in sourcing a script multiple times?

I don't think so. It's a pure helper script providing only functions and local variables. It does nothing on its own.

[PromoFaux]

Then let us do it the simple way! I'll redo once I'm back from the office

[pralor-bot]

This pull request has been mentioned on Pi-hole Userspace. There might be relevant details there:

https://discourse.pi-hole.net/t/pi-hole-ftl-v5-15-web-v5-12-and-core-v5-10-released/54987/1