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

Refactor code to remove OODT dependencies #75

Closed
jordanpadams opened this issue Jun 13, 2024 · 2 comments · Fixed by #78 or #79
Closed

Refactor code to remove OODT dependencies #75

jordanpadams opened this issue Jun 13, 2024 · 2 comments · Fixed by #78 or #79
Assignees
@jordanpadams
Labels
B15.0 i&t.skip Skip I&T of this task/ticket task

Comments

@jordanpadams
Copy link
Member

jordanpadams commented Jun 13, 2024 •
edited
Loading

💡 Description

The code is very old and prone to vulnerabilities. Let's refactor it out.

It is primarily used for crawler, metadata extraction and in-memory storage, which I think we can just create objects to do and/or use existing libraries to do it (e.g. validate has it's own crawler code)

⚔️ Parent Epic / Related Tickets

No response
@jordanpadams jordanpadams added B15.0 i&t.skip Skip I&T of this task/ticket task labels Jun 13, 2024
@github-project-automation github-project-automation bot moved this to Release Backlog in B15.0 Jun 13, 2024
@jordanpadams jordanpadams self-assigned this Jun 13, 2024
@tloubrieu-jpl
Copy link
Member

dependencies have been removed but need to be tested...

@tloubrieu-jpl
Copy link
Member

@jordanpadams is fine tuning the dependencies to avoid the security issues.

jordanpadams added a commit that referenced this issue Jul 20, 2024
@jordanpadams
Remove legacy OODT dependencies
78adbdd 
* Copied numerous classes from OODT v1.9, which led to incompatibilities with some aspects of the code
* Additional refactoring to support upgraded OODT components
* New smoke tests for integration testing

Resolves #75
This was referenced Jul 21, 2024
Merged
Merged
@github-project-automation github-project-automation bot moved this from Release Backlog to 🏁 Done in B15.0 Jul 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jordanpadams jordanpadams

Labels
B15.0 i&t.skip Skip I&T of this task/ticket task
Projects
No open projects
B15.0
Status: 🏁 Done
Milestone
No milestone
2 participants
@jordanpadams @tloubrieu-jpl