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

[NR-265913] API Endpoint Phase 2 : Detect route of an endpoint for Jax-RS Framework #256

Merged
merged 36 commits into from
Jun 22, 2024
Merged

[NR-265913] API Endpoint Phase 2 : Detect route of an endpoint for Jax-RS Framework #256

merged 36 commits into from
Jun 22, 2024

Conversation

IshikaDawda
Copy link
Contributor

@IshikaDawda IshikaDawda commented May 24, 2024 •
edited by jira bot
Loading

  • Add endpoint route to the generated event bean
  • Detect route of an endpoint for following Jax-RS based frameworks:
  1. CXF-JaxRS Framework
  2. Jersey Framework
  3. RestEasy Framework

Changelogs:

  • PR-256 Route detection of an incoming request for Jax-RS Framework. NR-265913
  • PR-259 Route detection of an incoming request for Spring-WebMVC Framework. NR-261653
  • PR-258 Route detection of an incoming request for Spring-WebFlux Framework. NR-273605

@IshikaDawda IshikaDawda added the endpoint-detect API endpoint detection feature label May 24, 2024
@IshikaDawda IshikaDawda requested a review from lovesh-ap May 24, 2024 11:17
@IshikaDawda IshikaDawda self-assigned this May 24, 2024
IshikaDawda and others added 21 commits May 26, 2024 10:57
@IshikaDawda
API Endpoint Phase 2: Add Endpoint route to httpRequestBean
c30ebc6
@IshikaDawda
Add framework to AgentMetaData & reset route if previous framework is…
48ceec6 
… servlet
@IshikaDawda
Add critical message to cxf-jaxrs
b37f0dd
@IshikaDawda
Add critical message to jersey
6b04bed
@IshikaDawda
Add endsWith functions to StringUtils
1dbe4e2
@IshikaDawda @lovesh-ap
NR-273607 : Fallback mechanism for route calculation
80d16c0 
Co-authored-by: Lovesh Baya <lbaya@newrelic.com>
@lovesh-ap
NPE fix for FRAMEWORK
8e3dd1d 
Redefine Route Comparator
@IshikaDawda
Add setter for detected route
d0952fa
@IshikaDawda
Bypass Route Calculation if the incoming request is an IAST Request
63fe995
@IshikaDawda
Merge branch 'refs/heads/feature/api-endpoint/JaxRS-NR-265913' into f…
d0b95df 
…eature/api-endpoint/fallback-mech-NR-273607
@IshikaDawda
Route Calculation if the route is blank for a request
c754664
@IshikaDawda
Optimise the normalization of URI Path in API Endpoint detection
7b12f8e
@IshikaDawda
Merge branch 'refs/heads/feature/api-endpoint/JaxRS-NR-265913' into f…
e38199b 
…eature/api-endpoint/fallback-mech-NR-273607
@IshikaDawda
update set route remove checks
53847b0
@IshikaDawda
Merge branch 'refs/heads/feature/api-endpoint/JaxRS-NR-265913' into f…
2df904f 
…eature/api-endpoint/fallback-mech-NR-273607
@IshikaDawda
ApplicationURLMapping path normalization
b2d2724
@IshikaDawda
Merge branch 'refs/heads/feature/api-endpoint/JaxRS-NR-265913' into f…
dccad86 
…eature/api-endpoint/fallback-mech-NR-273607
@IshikaDawda
[NR-261653] API Endpoint Phase 2 : Detect route of an endpoint for Sp…
7e3ed90 
…ring-WebMVC Framework (#259)

API Endpoint Phase 2: Add Endpoint route in Spring-WebMVC Framework
@IshikaDawda
Bump json version to 1.2.2 due to route reporting in http request bean
05e6f34
@IshikaDawda
[NR-273605] API Endpoint Phase 2 : Detect route of an endpoint for Sp…
321b03e 
…ring-WebFlux Framework (#258)

* API Endpoint Phase 2: Add Endpoint route in Spring-WebFlux Framework
@IshikaDawda
Updated check for framework in route detection
708e7af
IshikaDawda and others added 4 commits June 14, 2024 13:46
@IshikaDawda
Updated check for framework in route detection
0e20f94
@IshikaDawda
Add logging for route detection using Apps Endpoint
5298fd2
@IshikaDawda @lovesh-ap
[NR-273281] Sub-resource case handling in JAX-RS Framework for Route …
0bd5233 
…Detection (#263)

* Normalize the endpoint path in API Endpoint detection
* API Endpoint : Support for sub resource case in CXF-JaxRS
* API Endpoint : Support for sub resource case in Jersey
* Add method to count segments in uri path
* API Endpoint 2: Support for sub resource case in RestEasy

Co-authored-by: Lovesh Baya <lbaya@newrelic.com>
@IshikaDawda
Merge branch 'refs/heads/main' into feature/api-endpoint/JaxRS-NR-265913
355db2e 
# Conflicts:
#	newrelic-security-api/src/main/java/com/newrelic/api/agent/security/instrumentation/helpers/GenericHelper.java
@IshikaDawda IshikaDawda requested a review from harshit-ap June 22, 2024 04:15
harshit-ap
harshit-ap previously approved these changes Jun 22, 2024
View reviewed changes
@IshikaDawda
Merge branch 'feature/api-endpoint/JaxRS-NR-265913' into feature/api-…
66463f5 
…endpoint/fallback-mech-NR-273607
@IshikaDawda
Merge pull request #267 from newrelic/feature/api-endpoint/fallback-m…
ecf40a2 
…ech-NR-273607

[NR-273607] Fallback mechanism for route calculation
@harshit-ap harshit-ap requested review from harshit-ap June 22, 2024 05:11
harshit-ap
harshit-ap previously approved these changes Jun 22, 2024
View reviewed changes
@IshikaDawda
Merge branch 'refs/heads/main' into feature/api-endpoint/JaxRS-NR-265913
a0196bc
@IshikaDawda
Merge branch 'refs/heads/main' into feature/api-endpoint/JaxRS-NR-265913
a74b739 
# Conflicts:
#	gradle.properties
#	newrelic-security-agent/src/main/java/com/newrelic/api/agent/security/Agent.java
@IshikaDawda IshikaDawda merged commit 5d26d5a into main Jun 22, 2024
9 checks passed
@IshikaDawda IshikaDawda deleted the feature/api-endpoint/JaxRS-NR-265913 branch June 22, 2024 05:57
@lovesh-ap lovesh-ap mentioned this pull request Jun 23, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@harshit-ap harshit-ap harshit-ap approved these changes

@lovesh-ap lovesh-ap Awaiting requested review from lovesh-ap

Assignees

@IshikaDawda IshikaDawda

Labels
endpoint-detect API endpoint detection feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@IshikaDawda @harshit-ap @lovesh-ap