Fix slackapi#36 simple sample app using Chalice doesn't work

seratch · seratch · commit 9b718c9eaa05 · 2020-08-10T18:19:54.000+09:00
diff --git a/samples/aws_chalice/app.py b/samples/aws_chalice/app.py
@@ -4,16 +4,11 @@
 
 from slack_bolt import App
 from slack_bolt.adapter.aws_lambda.chalice_handler import ChaliceSlackRequestHandler
-from slack_bolt.adapter.aws_lambda.lambda_s3_oauth_flow import LambdaS3OAuthFlow
 
 # process_before_response must be True when running on FaaS
 bolt_app = App(
     process_before_response=True,
     authorization_test_enabled=False,
-    oauth_flow=LambdaS3OAuthFlow(
-        install_path="/api/slack/install",
-        redirect_uri_path="/api/slack/oauth_redirect",
-    ),
 )
 
 
@@ -57,9 +52,10 @@ def oauth_redirect() -> Response:
 
 # configure aws credentials properly
 # pip install -r requirements.txt
-# edit .chalice/config.json
+# cp -p .chalice/config.json.simple .chalice/config.json
+# # edit .chalice/config.json
 # rm -rf vendor/slack_* && cp -pr ../../src/* vendor/
 # chalice deploy
 
-# for local dev
+# # for local dev
 # chalice local --stage dev --port 3000
diff --git a/samples/aws_chalice/oauth_app.py b/samples/aws_chalice/oauth_app.py
@@ -57,7 +57,8 @@ def oauth_redirect() -> Response:
 
 # configure aws credentials properly
 # pip install -r requirements.txt
-# edit .chalice/config.json
+# cp -p .chalice/config.json.oauth .chalice/config.json
+# # edit .chalice/config.json
 # rm -rf vendor/latest_slack_bolt && cp -pr ../../src vendor/latest_slack_bolt
 # chalice deploy
 
diff --git a/samples/aws_chalice/simple_app.py b/samples/aws_chalice/simple_app.py
@@ -1,12 +1,15 @@
 import logging
-import time
 
 from chalice import Chalice, Response
+
 from slack_bolt import App
 from slack_bolt.adapter.aws_lambda.chalice_handler import ChaliceSlackRequestHandler
 
 # process_before_response must be True when running on FaaS
-bolt_app = App(process_before_response=True)
+bolt_app = App(
+    process_before_response=True,
+    authorization_test_enabled=False,
+)
 
 
 @bolt_app.event("app_mention")
@@ -15,6 +18,11 @@ def handle_app_mentions(payload, say, logger):
     say("What's up? I'm a Chalice app :wave:")
 
 
+@bolt_app.command("/hello-bolt-python-chalice")
+def respond_to_slack_within_3_seconds(ack):
+    ack("Thanks!")
+
+
 ChaliceSlackRequestHandler.clear_all_log_handlers()
 logging.basicConfig(format="%(asctime)s %(message)s", level=logging.DEBUG)
 
@@ -23,23 +31,6 @@ def handle_app_mentions(payload, say, logger):
 slack_handler = ChaliceSlackRequestHandler(app=bolt_app, chalice=app)
 
 
-def respond_to_slack_within_3_seconds(ack):
-    # This method is for synchronous communication with the Slack API server
-    ack("Thanks!")
-
-
-def can_be_long(say):
-    # This can be executed in a thread, asyncio's Future, a new AWS lambda function
-    # ack() is not allowed here
-    time.sleep(5)
-    say("I'm done!")
-
-
-bolt_app.command("/hello-bolt-python-chalice")(
-    ack=respond_to_slack_within_3_seconds, subsequent=[can_be_long],
-)
-
-
 @app.route(
     "/slack/events",
     methods=["POST"],
@@ -49,11 +40,22 @@ def events() -> Response:
     return slack_handler.handle(app.current_request)
 
 
+@app.route("/slack/install", methods=["GET"])
+def install() -> Response:
+    return slack_handler.handle(app.current_request)
+
+
+@app.route("/slack/oauth_redirect", methods=["GET"])
+def oauth_redirect() -> Response:
+    return slack_handler.handle(app.current_request)
+
+
 # configure aws credentials properly
 # pip install -r requirements.txt
-# edit .chalice/config.json
-# rm -rf vendor/latest_slack_bolt && cp -pr ../../src vendor/latest_slack_bolt
+# cp -p .chalice/config.json.simple .chalice/config.json
+# # edit .chalice/config.json
+# rm -rf vendor/slack_* && cp -pr ../../src/* vendor/
 # chalice deploy
 
-# for local dev
+# # for local dev
 # chalice local --stage dev --port 3000
