load_chain_config returns true on empty chain config

[OscarMrZ]

Hello!

I've noticed that the load_chain_config function returns true if there's no chain configuration in the parameters. Consequently, this means that the filter chain is correctly configured, but with an empty list of filters.

As far as my understanding goes, this means that there's no easy way to tell if the chain is operating without any filters, which could be problematic. For example, if I have a controller that may or may not have a filter chain, I don't have a way to tell if the user provided the config for a filter chain.

Is this the intended behaviour? I was wondering if it would just be better to return false is empty and let the caller handle it as needed.

Thanks a lot!

[jonbinney]

I think this is intended behavior, see my comment on your pr: #90

Maybe we could add a method that lets users check the number of filters in the chain, and if they expect it to be greater than 0 they can handle that as needed. That would also avoid changing the existing function in a way that could break existing code.

[OscarMrZ]

Closing as solved in #90. Thanks a lot!