mirror of
https://github.com/healthchecks/healthchecks.git
synced 2026-04-25 15:05:49 +03:00
[GH-ISSUE #771] improvement of Zulip integration docs #539
Labels
No labels
bug
bug
bug
feature
good-first-issue
new integration
pull-request
question
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/healthchecks#539
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @tiltX on GitHub (Dec 28, 2022).
Original GitHub issue: https://github.com/healthchecks/healthchecks/issues/771
I've had issues setting up a Zulip integration. Sending a notification to a stream worked fine but sending it to a private user didn't work:
The problem was a setting in Zulip's permission settings: "Who can access user email addresses" was set to "Administrators and Moderators" which excluded regular members from accessing a user's email address. However, the HC bot is set up to be a regular member. Because of that, the integration couldn't send the notification to the given user.
It might be helpful to add this to the the page where you set up the Zulip integration.
@cuu508 commented on GitHub (Jan 12, 2023):
Thanks for the suggestion.
Did you resolve the issue by changing the "Who can access user email addresses" setting?
Looks like there's a possible workaround that would require only a small code change on the Healthchecks side: specify the user by User ID, not by their email.
In Zulip, you can look up user's id in their profile (when you click on an user, or on a three dots button next to an user, there's usually a "View Profile" option):
Zulip REST API accepts user ids for sending private messages, and they work even when "Who can access user email addresses" is set to "Administrators and Moderators".
Healthchecks would require a small code change to support user ids. And docs would indeed need to be updated to explain when to use user emails, and when to use user ids, and how to look the user ids up.