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.

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

Unable to fetch slack messages due to incorrect timestamp format #3032

Open
abhi1693 opened this issue Dec 14, 2024 · 1 comment
Open

Unable to fetch slack messages due to incorrect timestamp format #3032

abhi1693 opened this issue Dec 14, 2024 · 1 comment
Labels
bug Something isn't working community-driven

Comments

@abhi1693
Copy link

Bug Description

The Slack Conversations History API returns no messages if the latest or oldest timestamps include decimal places. However, the API returns the expected messages if the timestamps are provided without any decimal places.

To Reproduce

Steps to reproduce the behavior:

  1. Call the Slack API using the following URL with decimal places in the timestamps:
https://slack.com/api/conversations.history?channel={channel_id}&limit=200&oldest=1718641354.561659&latest=1734193354.5616593
  1. Observe that no messages are returned in the response.
  2. Call the Slack API using the following URL without decimal places in the timestamps:
https://slack.com/api/conversations.history?channel={channel_id}&limit=200&oldest=1718641354&latest=1734193354
  1. Observe that messages are successfully returned.

Expected behavior

The API should return messages that are then added to the index.

Screenshots

Environment

  • OS: [e.g. iOS] docker.elastic.co/integrations/elastic-connectors:8.17.0
  • Browser [e.g. chrome, safari] NA
  • Version [e.g. 22] 8.17.0

Additional context

@abhi1693 abhi1693 added the bug Something isn't working label Dec 14, 2024
@seanstory
Copy link
Member

Sounds like you've found a bug in the Slack API, not in the connector. Looking at https://api.slack.com/methods/conversations.history#arg_oldest, I see that decimal format is the example, so it should be expected to work.

Image

Have you raised this issue with Slack?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working community-driven
Projects
None yet
Development

No branches or pull requests

2 participants