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

API Log Filtering based on API Resource Feature with Resource Matching #12046

Closed
wants to merge 42 commits into from
Closed

API Log Filtering based on API Resource Feature with Resource Matching #12046

wants to merge 42 commits into from

Conversation

chamikasudusinghe
Copy link
Contributor

shnrndk and others added 30 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
merge ResourceLevelLogfilteringCopy branch to get resource log levels
c2f7798
@chamikasudusinghe
removing regex usage for finding the resource path
5b3818c 
used the implementation of URITemplateBasedDispatcher for resource matching

issue: wso2/api-manager#1564
@chamikasudusinghe
fix for cors handler to process different method types
5fbe312
@chamikasudusinghe
including apilevel logging in the same logic
626eeb6
@chamikasudusinghe chamikasudusinghe changed the title API Log Filtering based on API Resource Feature with URITemplate based Resource Matching API Log Filtering based on API Resource Feature with Resource Matching Jun 20, 2023
@tharikaGitHub
Copy link
Member

Closing this PR as the changes are added from [1].

[1] #12288

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.
3 participants
@chamikasudusinghe @tharikaGitHub @shnrndk