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

Removing rule of finalizing path for API with .pinpoint #10256

Closed
smilu97 opened this issue Aug 28, 2023 · 0 comments · Fixed by #10257
Closed

Removing rule of finalizing path for API with .pinpoint #10256

smilu97 opened this issue Aug 28, 2023 · 0 comments · Fixed by #10257
Assignees
@smilu97
Labels
module:web

Comments

@smilu97
Copy link
Contributor

smilu97 commented Aug 28, 2023

Classiying request of which URI ends with .pinpoint as a request for back-end API is inappropriate because some path variables can abuse this, and make the re-write filter forward into wrong dispatcher
@smilu97 smilu97 self-assigned this Aug 28, 2023
smilu97 added a commit to smilu97/pinpoint that referenced this issue Aug 28, 2023
@smilu97
[pinpoint-apm#10256] Removed rule of finalizing path for API with .pi…
3383ac0 
…npoint
@smilu97 smilu97 mentioned this issue Aug 28, 2023
Merged
@smilu97 smilu97 changed the title Classifying requests for front-end application by request URI Removing rule of finalizing path for API with .pinpoint Aug 29, 2023
smilu97 added a commit that referenced this issue Aug 29, 2023
@smilu97
[#10256] Removed rule of finalizing path for API with .pinpoint
9b7c062
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@smilu97 smilu97

Labels
module:web
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#10256] Removed rule of finalizing path for API with .pinpoint smilu97/pinpoint
1 participant
@smilu97