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

bug: datadog plugin failed to send metric when consumer is used with route for authentication #9269

Closed
0prodigy opened this issue Apr 7, 2023 · 8 comments · Fixed by #11354
Assignees
Labels
bug Something isn't working

Comments

@0prodigy
Copy link

0prodigy commented Apr 7, 2023

Current Behavior

datadog plugin failed to send metric

Expected Behavior

plugin should send metric to datadog agent

Error Logs

lua entry thread aborted: runtime error: /usr/local/apisix/apisix/plugins/datadog.lua:92: attempt to concatenate field 'consumer' (a table value)

Steps to Reproduce

  1. Enable datadog plugin in routes
  2. Check log

Environment

  • APISIX version (run apisix version): 3.0.0
  • Operating system (run uname -a): Linux (x86_64 GNU/Linux)
  • OpenResty / Nginx version (run openresty -V or nginx -V): openresty/1.21.4.1
  • etcd version, if relevant (run curl http://127.0.0.1:9090/v1/server_info):
  • APISIX Dashboard version, if relevant:
  • Plugin runner version, for issues related to plugin runners:
  • LuaRocks version, for installation issues (run luarocks --version):
@0prodigy 0prodigy changed the title bug: bug: datadog plugin failed to send metric Apr 7, 2023
@Sn0rt
Copy link
Contributor

Sn0rt commented Apr 10, 2023

can you share your config of the router ?

@leslie-tsang leslie-tsang added the bug Something isn't working label Jul 27, 2023
@shreemaan-abhishek
Copy link
Contributor

@0prodigy please share steps to reproduce, I have configured the dd plugin before and it worked.

@shreemaan-abhishek shreemaan-abhishek added the wait for update wait for the author's response in this issue/PR label Dec 8, 2023
@monkeyDluffy6017
Copy link
Contributor

I will close this due to long period of inactivity, please feel free to reopen it

@zhuoyang
Copy link

I am facing this issue as well, it only happens when there's authentication enabled on the route

@kayx23 kayx23 reopened this Dec 21, 2023
@kayx23 kayx23 removed the wait for update wait for the author's response in this issue/PR label Jan 19, 2024
@kayx23 kayx23 changed the title bug: datadog plugin failed to send metric bug: datadog plugin failed to send metric when consumer is used with route for authentication Jan 19, 2024
@kayx23
Copy link
Member

kayx23 commented Jan 19, 2024

I was able to reproduce the issue with the following steps:

  1. configure datadog on a route
  2. configure a consumer with key-auth
  3. patch the route with key-auth
  4. send a request to the route and check logs

image

cc: @Vacant2333

@shreemaan-abhishek
Copy link
Contributor

looks like a good first issue to me

@Vacant2333
Copy link
Contributor

I was able to reproduce the issue with the following steps:

  1. configure datadog on a route
  2. configure a consumer with key-auth
  3. patch the route with key-auth
  4. send a request to the route and check logs

image

cc: @Vacant2333

u can assign me this, i will take a look when im free~~

@ecoupal-believe
Copy link

#10934

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
10 participants