search

dropbox / securitybot

Distributed alerting for the masses!
Apache License 2.0
994 stars 112 forks source link

Support pagination of get_users #30

Closed skiptomyliu closed 6 years ago

skiptomyliu commented 6 years ago

This PR will support pagination in get_users.

Currently, the default get_users will return 1000 users, which doesn't work for orgs/companies that have > 1000 users.

Worth also noting that this API endpoint has a ratelimit (https://api.slack.com/methods/users.list) that will easily hit with large Slack workspaces, however this PR doesn't address that.

skiptomyliu commented 6 years ago

Sorry, may have jumped gun -- cursor is not being set properly. Will reopen later.