-
Notifications
You must be signed in to change notification settings - Fork 235
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
Fix #307 Add options to disable the built-in middleware #310
Fix #307 Add options to disable the built-in middleware #310
Conversation
Codecov Report
@@ Coverage Diff @@
## main #310 +/- ##
==========================================
+ Coverage 91.45% 91.56% +0.10%
==========================================
Files 166 166
Lines 5278 5286 +8
==========================================
+ Hits 4827 4840 +13
+ Misses 451 446 -5
Continue to review full report at Codecov.
|
@@ -169,47 +169,6 @@ def handle_app_mention(): | |||
response = app.dispatch(request) | |||
assert response.status == 200 | |||
|
|||
def test_self_events(self): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I moved this to test_events_ignore_self.py
say("What's up?") | ||
|
||
# request including invalid headers | ||
expired = int(time()) - 3600 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if the request verification is enabled, this request is rejected.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Solid change!
Thanks for reviewing this PR! |
This pull request fixes #307 and also adds more options to disable the built-in middleware. Refer to my comment #307 (comment) for more details.
see also:
Category (place an
x
in each of the[ ]
)slack_bolt.App
and/or its core componentsslack_bolt.async_app.AsyncApp
and/or its core componentsslack_bolt.adapter
/docs
Requirements (place an
x
in each[ ]
)Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.
./scripts/install_all_and_run_tests.sh
after making the changes.