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.

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

Prevent http route resolution for requests made directly to server with 4xx response codes. #30568

Open
wants to merge 1 commit into
base: integration
Choose a base branch
from

Conversation

Channyboy
Copy link
Contributor

@Channyboy Channyboy commented Jan 15, 2025

When the server is handling requests made to the server itself (i.e., not through a deployed application) and a 4xx response code is encountered, there should not be an HTTP route resolved for the HTTP stat

################################################################################################
fixes #30567

… not through a deployed application) and a 4xx response code is encountered, there should not be an HTTP route resolved for the HTTP stat
@Channyboy
Copy link
Contributor Author

Channyboy commented Jan 15, 2025

#build
(view Open Liberty Personal Build - ❌ completed with errors/failures.)
#spawn.fullfat.buckets=io.openliberty.http.monitor_fat

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 3 FAT files were changed, added, or removed.

  • Check that the build did not break the affected FAT suite(s).

  • 1 product code files were changed.

  • Please describe in a separate comment how you tested your changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

HTTP Stat should not resolve HTTP route for requests that end with 4xx respone code
2 participants