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

Discord embed looks dodgy #1049

Closed
qaisjp opened this issue Mar 21, 2020 · 5 comments
Closed

Discord embed looks dodgy #1049

qaisjp opened this issue Mar 21, 2020 · 5 comments
Assignees
Labels

Comments

@qaisjp
Copy link
Collaborator

qaisjp commented Mar 21, 2020

Describe the bug

Sending a series of links on Discord causes output on Slack that seems to miss a leading newline.

To Reproduce
Steps to reproduce the behavior:

  1. Send https://youtube.com/ https://google.com/ https://discordapp.com/ on Discord
  2. Look at Slack
  3. If it doesn't work, send https://youtube.com/ https://google.com/ https://discordapp.com/ on Discord again
  4. Look at slack

You might have to do it a second time because I think Discord unfurling is late.

Note that the first time those link appear in Slack, you will see Slack's unfurling — Slack's unfurling won't appear the second time because they only unfurl once per "recent conversation" (to prevent embeds spamming up your screen). To get around Slack's unfurl "spam protection", you can cache bust by appending something to the query string: https://youtube.com/?1

Expected behavior

Either have a newline, or don't send Discord embeds if they are only unfurl embeds.

Screenshots/debug logs

image

Environment (please complete the following information):

e9edbfc, macOS

Additional context

Nothing of significance other than ShowEmbeds:

[discord]
[discord.mta]
Token="..."
Server="..."
RemoteNickFormat="{NICK}"
WebhookURL="..."
ShowEmbeds=true

[slack]
[slack.mta]
Token="..."
RemoteNickFormat="{NICK}"
@qaisjp qaisjp added the bug label Mar 21, 2020
@42wim
Copy link
Owner

42wim commented Mar 29, 2020

Hm tried to reproduce this, but nothing happens on the slack side, tried sending multiple times.

image

@42wim
Copy link
Owner

42wim commented Apr 19, 2020

@qaisjp still an issue ?

@qaisjp
Copy link
Collaborator Author

qaisjp commented Apr 19, 2020

Um let me try and reproduce on latest master (can't now though)

To confirm you've tried cache busting, right?

@42wim
Copy link
Owner

42wim commented Apr 19, 2020

No I haven't, but I couldn't produce it on the first message.

@qaisjp qaisjp self-assigned this Apr 20, 2020
@qaisjp
Copy link
Collaborator Author

qaisjp commented May 4, 2020

I can definitely reproduce this on 2d8f749

it only works after the first message (because discord has the embed cached)

qaisjp added a commit to qaisjp/matterbridge that referenced this issue May 4, 2020
qaisjp added a commit to qaisjp/matterbridge that referenced this issue May 4, 2020
@42wim 42wim closed this as completed in 52a071e May 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants