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

[Fix] request body logged content #1316

Merged
merged 1 commit into from
Mar 25, 2024

Conversation

jstourac
Copy link
Member

@jstourac jstourac commented Mar 24, 2024

There were two headers section logged instead of one request body section logged in the report.


After merge, I shall backport to releases/2.8.0 branch too.

@jstourac jstourac requested review from bdattoma and jiridanek March 24, 2024 19:03
@jstourac jstourac self-assigned this Mar 24, 2024
@jstourac jstourac added verified This PR has been tested with Jenkins enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) labels Mar 24, 2024
Copy link
Contributor

Robot Results

✅ Passed ❌ Failed ⏭️ Skipped Total Pass %
448 0 0 448 100

@jstourac jstourac marked this pull request as draft March 24, 2024 19:14
There were two headers section logged instead of one request body
section logged in the report.
@jstourac jstourac force-pushed the fixRequestBodyLog branch from 1059165 to 101cf73 Compare March 24, 2024 20:32
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@jstourac jstourac marked this pull request as ready for review March 24, 2024 20:33
@jstourac jstourac merged commit abfb8a5 into red-hat-data-services:master Mar 25, 2024
12 checks passed
@jstourac jstourac deleted the fixRequestBodyLog branch March 25, 2024 11:50
jstourac added a commit that referenced this pull request Mar 28, 2024
This PR contains following fixes from the `master` branch:

* edafbcc (#1296)
* 5e7f4b1 (#1302)
* b6fb7f7 (#1307)
* 9485246 (#1314)
* abfb8a5 (#1316)
* fc76761 (#1317)
* 439e2b3 (#1306)
* 5db9e26 (#1324)

---

I kept the order in which these commits were merged in the `master`
branch and provided both link to the particular commit and also link to
the particular PR so it's clearer to understand later on.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancements Bugfixes, enhancements, refactoring, ... in tests or libraries (PR will be listed in release-notes) verified This PR has been tested with Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants