Use the web standard Request/Response types for `installer.handleCallback`

[abhijit-hota]

Currently, the installer.handleCallback method supports the Node specific IncomingMessage and ServerResponse. This is not ideal for apps using Bun and other frameworks like Hono:

Here,

1. c.req.raw is of type Request and is not assignable to IncomingResponse which is a node:http specific type.
2. c.Res (type of Response) is not assignable to ServerResponse<IncomingMessage>.

I understand that this is node-slack-sdk and not bun-slack-sdk but this feature request is more about adopting a more compatible standard hence making it more framework/runtime agnostic. Some kind of backward compatibility might be needed.

If this is out of scope, how can I use this package with my above example (Hono and Bun)?

Packages:

Select all that apply:

• @slack/web-api
• @slack/rtm-api
• @slack/webhooks
• @slack/oauth
• @slack/socket-mode
• @slack/types
• I don't know

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[abhijit-hota]

how can I use this package with my above example (Hono and Bun)

I've resorted to manually validating state and calling oauth.v2.access APIs from @slack/web-api for now.

[WilliamBergamin]

@abhijit-hota Thanks for opening this 💯 it seems like a reasonable feature request!

Until we implement this, your workaround should be a valid solution

[kguzek]

Hi @WilliamBergamin, is there any progress on this? I see @zimeg has included it in the 'breaking changes' issue, but it's been open for quite a while now, and it seems like other features are being prioritized. I'd love to be able to use the callback handler directly without having to manually implement it in my hono route.

[WilliamBergamin]

👋 Since this is considered a breaking change we need to include it in a major release, unfortunately our current short term roadmap does not include releasing a major for this project