Skip to content

Remove remaining usages of commons HttpClient 3.1 - #159

Merged
labkey-adam merged 5 commits into
developfrom
fb_mascot
Apr 17, 2020
Merged

Remove remaining usages of commons HttpClient 3.1#159
labkey-adam merged 5 commits into
developfrom
fb_mascot

Conversation

@labkey-adam

@labkey-adam labkey-adam commented Apr 17, 2020

Copy link
Copy Markdown
Contributor

Rationale

Commons HttpClient 3.1 was released and end-of-lifed 13 years ago. The last remaining LabKey code that uses this library is MascotClientImpl. The goal here is to remove these usages and eliminate this old, unsupported library from the product.

Related Pull Requests

Changes

  • Add a test case that invokes the mock Mascot servlet
  • Convert two usages of commons HttpClient 3.1 to use modern HTTP libraries and ensure the test continues to pass
  • Remove commons HttpClient 3.1 from the product!

… (one to the Java HttpClient and the other to Apache HttpComponents). Remove commons HttpClient 3.1 from the product!
Make Mascot TestCase work on TeamCity
@labkey-adam
labkey-adam requested a review from a team as a code owner April 17, 2020 02:49
@labkey-adam labkey-adam changed the title Remove remaining usage of commons HttpClient 3.1 Remove remaining usages of commons HttpClient 3.1 Apr 17, 2020
@labkey-adam
labkey-adam merged commit 2ddeea7 into develop Apr 17, 2020
@labkey-adam
labkey-adam deleted the fb_mascot branch April 17, 2020 20:13
cnathe added a commit that referenced this pull request Aug 3, 2026
…ontainer's @scripts dir (#1048)

## Rationale
LabKey/internal-issues#159
See related PR for rationale. This PR fixes the Luminex tests so that
they include their utility file in the `@scripts` dir when using the
assay transform scripts.

## Related Pull Requests
- LabKey/platform#7895

## Changes
- Luminex test fixes to add youtil.R file to `@scripts `dir
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.

2 participants