Fixes #2290

The following error is resolved:

```
[...]
    if CCITT.COLUMNS in parameters:
TypeError: argument of type 'IndirectObject' is not iterable
```

Closes  #2321

Since b"" is not mutable it causes python to allocate and deallocate memory repeatedly in the for loop which cause hang/long runtime when handle very large string.

For example when using add_js(javascript) to  to add a very big `javascript` bytestring.

Bumps [actions/setup-python](https://github.com/actions/setup-python) from 4 to 5.
- [Release notes](https://github.com/actions/setup-python/releases)
- [Commits](actions/setup-python@v4...v5)

---
updated-dependencies:
- dependency-name: actions/setup-python
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

This PR introduces a new `tests/example_files.yaml` in which we have the local filename as well as the URL from which we download the document.

The idea is that we move all download URLs to that CSV document. Then we just reference the local file names. The advantage of this approach are:

* **Detection of name clashes**: If we use the same `name` for different URLs we could have flaky tests as well. That is currently super hard to detect and would be rather easy to prevent in future
* **No tests needs network**: The network is a source for flaky tests. That's hard to notice. If the download fails, it's easier to see.
* **Efficiency**: We can heavily parallelize downloads. That might lead to speed-ups.
* **De-duplication**: We might have the same file with different file-names in the code base. Having all URLs in one place makes it easier to ensure we download everything just once
* **Shorter Python code**: That's a tiny stylistic argument. I prefer if we don't have super long lines / need to make weird line breaks as the URL is just so long.
* **Cross-project use**: Other projects might be interested in that CSV and use it for their tests.
* **Internal re-use**: It's easier to loop over the existing files we have like this. That way, we can potentially use one PDF for multiple types of tests.

When handling flate objects with a lookup table and the image mode `1`, we would previously raise a generic `AssertionError` if the number of lookup values did not match.

Cases where too many values are specified are now considered a warning only.
Additionally, this PR adds a more meaningful error message.

## What's new

### Bug Fixes (BUG)
-  Cope with deflated images with CMYK Black Only (#2322) by @pubpub-zz
-  Handle indirect objects as parameters for CCITTFaxDecode (#2307) by @stefan6419846
-  check words length in _cmap type1_alternative function (#2310) by @Takher

### Robustness (ROB)
-  Relax flate decoding for too many lookup values (#2331) by @stefan6419846
-  Let _build_destination skip in case of missing /D key (#2018) by @nickryand

### Documentation (DOC)
-  Note in reading form data (#2338) by @MartinThoma
-  Pull Request prefixes and size by @MartinThoma
-  Add https://github.com/zuypt for #2325 as a contributor by @MartinThoma
-  Fix docstring for RunLengthDecode.decode (#2302) by @stefan6419846

### Maintenance (MAINT)
-  Enable `disallow_any_generics` and add missing generics (#2278) by @nilehmann

### Testing (TST)
-  Centralize file downloads (#2324) by @MartinThoma

### Code Style (STY)
-  Fix typo "steam" \xe2\x86\x92 "stream" (#2327) by @stefan6419846
-  Run black by @MartinThoma
-  Make Traceback in bug report template uppercase (#2304) by @stefan6419846

[Full Changelog](3.17.1...3.17.2)