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.

Sign up for GitHub

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

Jump to bottom

Consider adding remoteAddr to Apiman metrics #2448

Open
msavy opened this issue Jan 14, 2023 · 2 comments
Open

Consider adding remoteAddr to Apiman metrics #2448

msavy opened this issue Jan 14, 2023 · 2 comments
Labels
C: Gateway SC: Metrics Subcomponent: Metrics T: Feature Request Something new you want added to Apiman

Comments

@msavy
Copy link
Member

msavy commented Jan 14, 2023 

          Consider adding `remoteAddr` to Apiman metrics

Originally posted by @msavy in #2408 (comment)
@msavy msavy added SC: Metrics Subcomponent: Metrics C: Gateway T: Feature Request Something new you want added to Apiman labels Jan 14, 2023
@sreenath-tm
Copy link

Hi I would like to work on this issue can you let me know how i can solve this issue.

@msavy
Copy link
Member Author

msavy commented Feb 7, 2023

Hi, thank you so much for your interest!

I'm afraid this change will require quite a few changes because it will cascade through to all the metrics implementations including SQL. I need to have a think about how to deal with that. I suspect it might need to wait until Apiman 4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C: Gateway SC: Metrics Subcomponent: Metrics T: Feature Request Something new you want added to Apiman
Projects
Apiman Technical
Status: Backlog
Milestone
No milestone
Development

No branches or pull requests
2 participants
@msavy @sreenath-tm