Always remove filter maybe_double_knock http_response filter!

[pfefferle]

This PR ensures that the http_response filter will always be removed, sepecially for non AP requests.

props @akirk !

Proposed changes:

Other information:

• Have you written new tests for your changes, if applicable?

Testing instructions:

• Go to '..'

Changelog entry

• Automatically create a changelog entry from the details below.

Changelog Entry Details

Significance

• Patch
• Minor
• Major

Type

• Added - for new features
• Changed - for changes in existing functionality
• Deprecated - for soon-to-be removed features
• Removed - for now removed features
• Fixed - for any bug fixes
• Security - in case of vulnerabilities

Message

[Copilot]

Pull Request Overview

This PR fixes a potential infinite recursion issue by moving the filter removal to occur before the early return conditions in the maybe_double_knock method. The change ensures that the http_response filter is always removed at the beginning of the method execution, preventing infinite loops even when the method returns early.

• Relocated the remove_filter call to prevent infinite recursion in all code paths
• Added explanatory comment for the filter removal