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

Simplify slow call logging events #329

Merged
merged 1 commit into from
Jun 5, 2024
Merged

Conversation

kevin-mcgoldrick
Copy link
Collaborator

Simplify slow call logging events

Please make sure these check boxes are checked before submitting

  • ** Squashed Commits **
  • ** All Tests Passed ** - mvn clean test -P default

** PR review process **

  • Requires one +1 from a reviewer
  • Repository owners will merge your PR once it is approved.

Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kevin-mcgoldrick kevin-mcgoldrick merged commit bb56666 into master Jun 5, 2024
@kevin-mcgoldrick kevin-mcgoldrick deleted the slow_logging_events branch June 5, 2024 21:14
Zakaria-Kofiro pushed a commit that referenced this pull request Jun 18, 2024
Zakaria-Kofiro added a commit that referenced this pull request Sep 20, 2024
* filter by content-type to save responseData

* Add client5

* Refactor checkContentType to use AgentConfig for MIME types

* IMDSv2 (#322)

* [SRE-30912] Disallow Doctype Declaration (#323)

* Disallow Doctype Decleration

* disable DTD in XMLInputFactory of the ScriptProcessor
---------

Co-authored-by: zkofiro <[email protected]>

* Nonlinear Workload - Calculate Number of Agents based on Total Ramp Rate and Max Users/Sec per Agent (#320)

* initial implementation

* unneeded

---------

Co-authored-by: zkofiro <[email protected]>

* [SRE-30568] Tank V2 API Module Refactor (#324)

* Tank V2 API Module Refactor

* unneeded test file

* updating pom

* missing resource file for tests

* update all requests w/ auth header

* small updates, unneeded files

---------

Co-authored-by: zkofiro <[email protected]>

* Update POST /scripts endpoint and add "Copy From Script" functionality (#325)

* Tank V2 API Module Refactor

* unneeded test file

* updating pom

* missing resource file for tests

* update all requests w/ auth header

* moving to new branch

* small updates, unneeded files

* small fixes, unit tests, and fix for script upload error

* match convention

---------

Co-authored-by: zkofiro <[email protected]>

* null checks for input (#326)

Co-authored-by: zkofiro <[email protected]>

* JDK httpclient (HTTP/2.0) (#327)

* Add JDK Http Client
* Remove: apache httpclient 5

* library updates (#328)

* Simplify slow call logging events (#329)

* Refactor checkContentType to use AgentConfig for MIME types

* removing http client 5

* updated for JDK client

* create TankConfig once

* pull mimeTypes once + update checkContentType

---------

Co-authored-by: zkofiro <[email protected]>
Co-authored-by: Kevin McGoldrick <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants