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

When use redis store event, backend send webhook error #1616

Closed
AYue-94 opened this issue Sep 6, 2024 · 1 comment · Fixed by #1628
Closed

When use redis store event, backend send webhook error #1616

AYue-94 opened this issue Sep 6, 2024 · 1 comment · Fixed by #1628
Labels
kind/bug Something isn't working
Milestone

Comments

@AYue-94
Copy link

AYue-94 commented Sep 6, 2024

What happened?

When using Redis storage, the incorrect key was used to query the InstanceMeta, resulting in empty QueryResultEventInfoEntity and a null pointer during notification.
image

How can we reproduce it (as minimally and precisely as possible)?

  1. start agent and send event
  2. start backend use redis
  3. config webhook on backend ui

Anything else we need to know?

No response

Sermant version

2.0.0

OS version

MacOS

@AYue-94 AYue-94 added the kind/bug Something isn't working label Sep 6, 2024
@lilai23
Copy link
Collaborator

lilai23 commented Sep 10, 2024

this issue may happens for addInstanceMeta method use set rather than hset. Since hset can not set expire time for filed, I think we should just use set and get with expire time

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants