Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/api/should not filter #306

Open
wants to merge 3 commits into
base: develop
Choose a base branch
from

Conversation

tthailu
Copy link

@tthailu tthailu commented May 6, 2024

Adding should not filter to base interceptor.

Description

The PR above adds should not filter methods to the base interceptor.
It is provided in a way subclasses can override this method to control what requests should not be filtered.

changes in this Pull request

Added should not filter to

  1. synapse-api-rest-imperative BaseHttpInterceptor
  2. synapse-client-rest BaseHttpInterceptor

@tthailu tthailu marked this pull request as ready for review May 6, 2024 21:02
@tthailu tthailu requested a review from a team as a code owner May 6, 2024 21:02
@tthailu tthailu requested a review from sowmyab9 May 6, 2024 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants