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

[PR #5019/3fe9592c backport][stable-4] Slack: Add support for (some) groups #5043

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Jul 31, 2022

This is a backport of PR #5019 as merged into main (3fe9592).

SUMMARY

Some of the older private channels in the workspace I'm working in have channel ID's starting with G0 and GF and this resulted to false positive channel_not_found errors.
I've added these prefixes to the list to maintain as much backwards compatibility as possible.

Ideally the auto-prefix of the channel name with # is dropped entirely, given the Channel ID's have become more dominant in the Slack API over the past years.

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

Slack notification module

ADDITIONAL INFORMATION

This ansible task will trigger the "channel_not_found" error, because this module erroneously prefixes the channel id with #:

- name: Send a message to slack
  community.general.slack:
    token: "{{ slack_token }}"
    channel: GF123456
    msg: my-message

* Slack: Add support for (some) groups

Some of the older private channels in the workspace I'm working in have channel ID's starting with `G0` and `GF` and this resulted to false positive `channel_not_found` errors.
I've added these prefixes to the list to maintain as much backwards compatibility as possible.

Ideally the auto-prefix of the channel name with `#` is dropped entirely, given the Channel ID's have become more dominant in the Slack API over the past years.

* Add changelog fragment for slack channel prefix fix

* Update changelogs/fragments/5019-slack-support-more-groups.yml

Co-authored-by: Felix Fontein <[email protected]>

Co-authored-by: Felix Fontein <[email protected]>
(cherry picked from commit 3fe9592)
@ansibullbot
Copy link
Collaborator

@ansibullbot ansibullbot added backport bug This issue/PR relates to a bug module module needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR new_contributor Help guide this first time contributor notification plugins plugin (any type) and removed needs_revision This PR fails CI tests or a maintainer has requested a review/revision of the PR labels Jul 31, 2022
@felixfontein felixfontein merged commit 9f1e976 into stable-4 Jul 31, 2022
@felixfontein felixfontein deleted the patchback/backports/stable-4/3fe9592cf1505a78b243ed4552aba06423ee948b/pr-5019 branch July 31, 2022 21:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue/PR relates to a bug module module new_contributor Help guide this first time contributor notification plugins plugin (any type)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants