Skip to content

fix wechat notifier response field name#3330

Merged
gotjosh merged 1 commit intoprometheus:mainfrom
r0nn:main
Apr 28, 2023
Merged

fix wechat notifier response field name#3330
gotjosh merged 1 commit intoprometheus:mainfrom
r0nn:main

Conversation

@r0nn
Copy link
Contributor

@r0nn r0nn commented Apr 18, 2023

fix response json field name according to debug log:

ts=2023-04-17T07:49:52.559Z caller=wechat.go:178 level=debug integration=wechat response="{\"errcode\":60020,\"errmsg\":\"not allow to access from your ip, hint: [1681717792365202991262631], from ip: xxx.xxx.xxx.xxx, more info at https://open.work.weixin.qq.com/devtool/query?e=60020\"}" incident="{}:{alertname=\"不可用\"}"

and wechat api doc:
https://developer.work.weixin.qq.com/document/path/90236

Copy link
Member

@gotjosh gotjosh left a comment

Choose a reason for hiding this comment

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

LGTM

Thank you very much for your contribution.

Signed-off-by: gotjosh <josue.abreu@gmail.com>
@gotjosh gotjosh enabled auto-merge (squash) April 28, 2023 11:34
@gotjosh gotjosh merged commit f040f08 into prometheus:main Apr 28, 2023
gotjosh pushed a commit that referenced this pull request Aug 7, 2023
Signed-off-by: gotjosh <josue.abreu@gmail.com>
radek-ryckowski pushed a commit to goldmansachs/alertmanager that referenced this pull request Nov 6, 2023
Signed-off-by: gotjosh <josue.abreu@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants