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

Fixes problem with detecting HTTP dump request via HTTP user #112

Merged
merged 1 commit into from
Dec 5, 2023
Merged

Fixes problem with detecting HTTP dump request via HTTP user #112

merged 1 commit into from
Dec 5, 2023

Commits on Dec 4, 2023

  1. Fixes problem with detecting HTTP dump request via HTTP user 

    ```
http:https://http-dump@localhost:8000/user/3/update
         ^^^
```

fixes #108

---

AI commit text

Updated also the test files and methods dealing with the handling of HTTP dumps, Inspector and Sentry events to include test cases that cover the scenario when an HTTP user is present. The goal of this change is to ensure handling even when an HTTP user is specified. In `AnyHttpRequestDump.php`, a condition was added to check if the HTTP user equals "http-dump", which is anticipated in the newly added scenarios. In the test files, new test methods were written to simulate requests with an HTTP user and additional assertions were made to verify if the events were correctly processed.
    @butschster
    butschster committed Dec 4, 2023
    Configuration menu
    Copy the full SHA
    d21e102 View commit details
    Browse the repository at this point in the history