`defaultOptions` only invokes _last_ `requestModifier` in chain

[nodediggity]

`defaultOptions` only invokes _last_ `requestModifier` in chain

[onevcat]

Currently the requestModifier option only keeps the last value, it is not linkable and cannot be called in a chain.

Instead of setting multiple . requestModifier, you need to create a "multiple version" of ImageDownloadRequestModifier yourself, like this:

final class ConcatenatedRequestModifier: ImageDownloadRequestModifier {
    let modifiers: [any ImageDownloadRequestModifier]
    init(modifiers: [any ImageDownloadRequestModifier]) {
        self.modifiers = modifiers
    }
    
    func modified(for request: URLRequest) -> URLRequest? {
        var r = request
        for modifier in modifiers {
            if let modified = modifier.modified(for: r) {
                r = modified
            } else {
                return nil
            }
        }
        return r
    }   
}

Indeed it is a good idea to allow the option accepts multiple .requestModifier as a chained behavior. However, it breaks the current behavior and usage (although this should be a rare use case). I will consider it in the following major upgrade!

Thanks for the suggestion.

[nodediggity]

Ah thank you so much, I misunderstood how it works, your suggestion worked perfectly.

Feel free to close this, I really appreciate the speedy response 🙏