Skip to content

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

Merged
Zakaria-Kofiro merged 8 commits intomasterfrom
zkofiro/SRE-30568
May 17, 2024
Merged

[SRE-30568] Tank V2 API Module Refactor#324
Zakaria-Kofiro merged 8 commits intomasterfrom
zkofiro/SRE-30568

Conversation

@Zakaria-Kofiro
Copy link
Collaborator

@Zakaria-Kofiro Zakaria-Kofiro commented May 1, 2024

[SRE-30568] Tank V2 API Module Refactor

  1. Divide out the rest-mvc module into client and impl submodules
  2. Move the client module off of spring to httpclient
  3. Move Tank V2 API data model files to common module, the api module

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.

@Zakaria-Kofiro Zakaria-Kofiro marked this pull request as ready for review May 1, 2024 20:54
@Zakaria-Kofiro Zakaria-Kofiro marked this pull request as draft May 2, 2024 01:54
@Zakaria-Kofiro Zakaria-Kofiro marked this pull request as ready for review May 13, 2024 21:58
@Zakaria-Kofiro Zakaria-Kofiro merged commit 5a1498d into master May 17, 2024
@Zakaria-Kofiro Zakaria-Kofiro deleted the zkofiro/SRE-30568 branch May 17, 2024 01:11
Zakaria-Kofiro added a commit that referenced this pull request Jun 18, 2024
* 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 <zakaria_kofiro@intuit.com>
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 <zakaria_kofiro@intuit.com>

* 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 <zakaria_kofiro@intuit.com>

* [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 <zakaria_kofiro@intuit.com>

* 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 <zakaria_kofiro@intuit.com>

* null checks for input (#326)

Co-authored-by: zkofiro <zakaria_kofiro@intuit.com>

* 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 <zakaria_kofiro@intuit.com>
Co-authored-by: Kevin McGoldrick <kevin_mcgoldrick@intuit.com>
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