-
Notifications
You must be signed in to change notification settings - Fork 10
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
Rewrite For New API #158
Comments
Are you referring to the Events API? I hadn't really thought about this before but given how it wraps and reuses the same Is that what you had in mind? Also, I'm curious about your firewall issues but I assume it's related to how you need to open up http access to your slack app for the http handlers? Also, thanks for being an active supporter of the project! |
Yup
Exactly
This issue is related to security in different parts of our infra. Nothing I can go into here. |
I haven't looked at the details yet but looking at the Events API at a high-level, I could almost just imagine adding new I can't commit to a timeline for the addition but this is definitely interesting and I'd be up for exploring that addition. A few random thoughts:
Any more thoughts to share about this? The way I'm thinking about this right now is that this would be just about adding that compatibility with the Events API via server or serverless modes while preserving the existing functionality. |
I just realized I had completely forgotten about this issue. The main reason I haven't taken this on myself is that my main deployments have the opposite problem: it's easier to use the RTM api because it's an outgoing web socket connection rather than having a publicly accessible http endpoint. I might still be able to work on this and test this out at some point but if you'd like to try and add the feature using your setup for testing and iterating, I'd be happy to look over those changes and iterate on the implementation with you. |
Your Question
Any thoughts about creating a new framework based on the new slack API style using inbound HTTPS calls from slack?
We're needing to move in that direction to support some of the newer features like modals. We already have a hack to get around that and some firewall issues but I'm wondering if you'd be interested in help with any clean-slate work here.
The text was updated successfully, but these errors were encountered: