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.

Sign up for GitHub

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

Jump to bottom

Add built-in StateStore implementations using server-side database #1439

Closed
1 of 6 tasks
seratch opened this issue Feb 28, 2022 · 0 comments · Fixed by #1441
Closed
1 of 6 tasks

Add built-in StateStore implementations using server-side database #1439

seratch opened this issue Feb 28, 2022 · 0 comments · Fixed by #1441
Assignees
@seratch
Labels
enhancement M-T: A feature request for new functionality pkg:oauth applies to `@slack/oauth-helper`
Milestone
oauth@2.5.0

Comments

@seratch
Copy link
Member

seratch commented Feb 28, 2022

Ad mentioned at #1436 (comment), @slack/oauth library provides only ClearStateStore as the built-in implementations of StateStore. ClearStateStore does not rely on any server-side database.

In some use case, more secure solutions would be necessary for complying with an organization's security standard. We can add more StateStore as built-in implementation (or as examples) in the future:

  • FileStateStore - local file system based; this can be part of this library
  • MongooseStateStore - Mongoose (MongoDB client) based; this should be part of bolt-js-extensions (we'll publish this in the future)
  • PrismaStateStore - Prisma (RDB / MongoDB client) based; this should be part of bolt-js-extensions (we'll publish this in the future)
  • DynamoDBStateStore - AWS DynamoDB based one; this should be part of bolt-js-extensions (we'll publish this in the future)
  • AmazonS3StateStore - Amazon S3 based one; this should be part of bolt-js-extensions (we'll publish this in the future)
  • Google Cloud and many more ...

Packages:

Select all that apply:

  • @slack/web-api
  • @slack/rtm-api
  • @slack/webhooks
  • @slack/oauth
  • @slack/socket-mode
  • 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.
@seratch seratch added enhancement M-T: A feature request for new functionality pkg:oauth applies to `@slack/oauth-helper` labels Feb 28, 2022
@seratch seratch added this to the oauth@2.5.0 milestone Feb 28, 2022
@seratch seratch self-assigned this Feb 28, 2022
seratch added a commit to seratch/node-slack-sdk that referenced this issue Feb 28, 2022
@seratch
Fix slackapi#1439 Add built-in StateStore implementations using serve…
56d5085 
…r-side database
@seratch seratch mentioned this issue Feb 28, 2022
Merged
2 tasks
seratch added a commit that referenced this issue Mar 1, 2022
@seratch
Fix #1439 Add built-in StateStore implementations using server-side d…
f0fc6d8 
…atabase (#1441)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@seratch seratch

Labels
enhancement M-T: A feature request for new functionality pkg:oauth applies to `@slack/oauth-helper`
Projects
None yet
Milestone
oauth@2.5.0
Development

Successfully merging a pull request may close this issue.

Fix #1439 Add built-in StateStore implementations using server-side database seratch/node-slack-sdk
1 participant
@seratch