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

Alerting: Clarify that Threema Gateway Alerts support only Basic IDs #34828

Merged
merged 2 commits into from May 28, 2021

Conversation

dbrgn
Copy link
Contributor

@dbrgn dbrgn commented May 27, 2021

Threema Gateway supports two types of IDs: Basic IDs (where the
encryption is managed by the API server) and End-to-End IDs (where the
keys are managed by the user).

This plugin currently does not support End-to-End IDs (since it's much
more complex to implement, because the encryption needs to happen
locally). Add a few clarifications to the UI.

(Note: There's also an "Info" key in the notifier plugin which contains more information, but I did not find out where in the UI this is being displayed 😕 Maybe it would be good to show this info text more prominently for alert plugins.)

@dbrgn dbrgn requested a review from a team as a code owner May 27, 2021 13:07
@dbrgn dbrgn requested review from aknuds1 and marefr and removed request for a team May 27, 2021 13:07
@CLAassistant
Copy link

CLAassistant commented May 27, 2021

CLA assistant check
All committers have signed the CLA.

@grafanabot grafanabot added area/backend pr/external This PR is from external contributor labels May 27, 2021
Copy link
Contributor

@aknuds1 aknuds1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm

@aknuds1 aknuds1 added area/alerting/notifications Issues when sending alert notifications type/chore labels May 27, 2021
@aknuds1 aknuds1 requested a review from a team May 27, 2021 13:24
Copy link
Contributor

@aknuds1 aknuds1 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh sorry, I just recalled you need to update pkg/services/ngalert/notifier/, since that's our new alerting subsystem starting from v8.

@dbrgn
Copy link
Contributor Author

dbrgn commented May 27, 2021

Ah, thanks, I somehow missed that!

Threema Gateway supports two types of IDs: Basic IDs (where the
encryption is managed by the API server) and End-to-End IDs (where the
keys are managed by the user).

This plugin currently does not support End-to-End IDs (since it's much
more complex to implement, because the encryption needs to happen
locally). Add a few clarifications to the UI.
@dbrgn dbrgn requested review from a team, oscarkilhed, thisisobate and aknuds1 and removed request for a team May 27, 2021 16:13
@aknuds1 aknuds1 added this to the 8.1.0 milestone May 27, 2021
@aknuds1
Copy link
Contributor

aknuds1 commented May 27, 2021

Just closing and re-opening this to get checks unstuck.

@aknuds1 aknuds1 closed this May 27, 2021
@aknuds1 aknuds1 reopened this May 27, 2021
@dbrgn dbrgn requested a review from a team as a code owner May 27, 2021 19:54
@aknuds1 aknuds1 added the old backport v8.0.x Mark PR for automatic backport to v8.0.x label May 28, 2021
@aknuds1 aknuds1 modified the milestones: 8.1.0, 8.0.0-beta3 May 28, 2021
@aknuds1 aknuds1 merged commit 83a83de into grafana:main May 28, 2021
@aknuds1
Copy link
Contributor

aknuds1 commented May 28, 2021

Thanks for contributing to Grafana!

grafanabot pushed a commit that referenced this pull request May 28, 2021
Threema Gateway supports two types of IDs: Basic IDs (where the
encryption is managed by the API server) and End-to-End IDs (where the
keys are managed by the user).

This plugin currently does not support End-to-End IDs (since it's much
more complex to implement, because the encryption needs to happen
locally). Add a few clarifications to the UI.

Co-authored-by: Arve Knudsen <arve.knudsen@gmail.com>
(cherry picked from commit 83a83de)
aknuds1 pushed a commit that referenced this pull request May 28, 2021
…34871)

Threema Gateway supports two types of IDs: Basic IDs (where the
encryption is managed by the API server) and End-to-End IDs (where the
keys are managed by the user).

This plugin currently does not support End-to-End IDs (since it's much
more complex to implement, because the encryption needs to happen
locally). Add a few clarifications to the UI.

Co-authored-by: Arve Knudsen <arve.knudsen@gmail.com>
(cherry picked from commit 83a83de)

Co-authored-by: Danilo Bargen <mail@dbrgn.ch>
@dprokop dprokop changed the title Clarify that Threema Gateway Alerts support only Basic IDs Alerting: Clarify that Threema Gateway Alerts support only Basic IDs Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
add to changelog area/alerting/notifications Issues when sending alert notifications area/backend area/frontend old backport v8.0.x Mark PR for automatic backport to v8.0.x pr/external This PR is from external contributor type/chore
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants