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

Resource level filter logging feature added #11961

Closed
wants to merge 26 commits into from
Closed

Resource level filter logging feature added #11961

wants to merge 26 commits into from

Conversation

shnrndk
Copy link
Contributor

@shnrndk shnrndk commented Apr 10, 2023

shnrndk added 26 commits March 13, 2023 15:19
@shnrndk
ResourcePath and ResourceMethod added to loggingApiInput Schema
e1b423e
@shnrndk
Moved log_level field from AM_API to AM_API_URL_MAPPING.
d57654e
@shnrndk
log level field added to am_api also to work with both tables.
1de9212
@shnrndk
changed the get apis related api logging to support with the two tables.
d69f1b1
@shnrndk
Unit test testGetApiLogging testcase failure fixed.
2fadd3d
@shnrndk
/api-logging-configs GET rest api changed to support the resourceMeth…
d3f2e0b 
…od and resourcePath properties.
@shnrndk
getAPILogLevel() method changed to compare the log level from api lev…
2c446ab 
…el and resource level.
@shnrndk
logResourceProperties inner class changed to properties hash map.
7eb9ec2
@shnrndk
Changed RETRIEVE_ALL_PER_API_RESOURCE_LOGGING_SQL sql query to return…
b71e155 
… non revision apis only.
@shnrndk
resourceMethod and resourcePath added APIEvent class
a29e40d
@shnrndk
resourceMethod and resourcePath added APIEvent class
2529c77
@shnrndk
Merge remote-tracking branch 'origin/ResoureLevelFilterLogging' into …
8f19598 
…ResoureLevelFilterLogging
@shnrndk
Fixed some bugs in the logic.
dacb9dd
@shnrndk
Log properties multiple key adding bug fixed
557a1d5
@shnrndk
regex matching added to resource path checking
391d5ea
@shnrndk
Code reformatted
5f63a96
@shnrndk
Merge branch 'master' into ResourceLevelLogfilteringCopy
fc24c50
@shnrndk
Added validation to check api resource is valid or not when adding th…
93ac087 
…e resource log level
@shnrndk
If condition logic simplified
24ade92
@shnrndk
NullPointerException fixed when passing without resourcePath and reso…
8e7f197 
…urceMethod
@shnrndk
Null pointer issue fixed when the resourcePath is null.
4f0eac8
@shnrndk
resourcePath regex matching changed to support - and alphabets and no…
ec5a893 
…t only digits.
@shnrndk
added log level off comparison to the filter
62c4bdc
@shnrndk
Regex pattern match updated
44c75bf
@shnrndk
Array index out of bound error fixed.
0eca0a3
@shnrndk
Get log details of all apis not returning resource methods and paths …
529c806 
…fixed.
@chamikasudusinghe chamikasudusinghe mentioned this pull request May 24, 2023
Closed
@shnrndk shnrndk closed this Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Need to support per API Log Filtering based on API Resource
1 participant
@shnrndk