Fix UnicodeEncodeError for non-Latin characters

[dvandonkelaar]

What type of PR is this?

• Refactor
• Feature
• Bug Fix
• New Query Runner (Data Source)
• New Alert Destination
• Other

Description

The Slack Notification gives an error on non-Latin characters (for example '한글'). This fix encodes the message to UTF-8 to prevent the error.

How is this tested?

• Unit tests (pytest, jest)
• E2E Tests (Cypress)
• Manually
• N/A

Related Tickets & Documents

The issue is described in #6710

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

N/A

[justinclift]

Thanks @dvandonkelaar, that looks like a pretty straight forward fix. 😄

[justinclift]

I've enabled "auto-merge" for this, so if it passes the CI tests then it should get automatically merged. 😄

[codecov]

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (a54171f) 63.38% compared to head (633b4c8) 63.38%.

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #6715   +/-   ##
=======================================
  Coverage   63.38%   63.38%           
=======================================
  Files         162      162           
  Lines       13176    13176           
  Branches     1819     1819           
=======================================
  Hits         8351     8351           
  Misses       4534     4534           
  Partials      291      291           

Files	Coverage Δ
redash/destinations/slack.py	41.93% <0.00%> (ø)

[unpo88]

@dvandonkelaar

Thanks a lot! I noticed that you already posted an issue that I was about to raise and modify. Really appreciate your proactiveness! Looking forward to seeing this get merged soon :)

[justinclift]

@unpo88 In theory, the preview build on Docker Hub should have this change if you want to verify things. 😄