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

WIP: Remove the version restriction for http gem if its not needed #13

Merged
merged 1 commit into from
Aug 10, 2022

Conversation

arian-kh
Copy link
Contributor

@arian-kh arian-kh commented Jul 27, 2022

Remove the version restriction for http gem if it's not needed

@clio-overlord
Copy link

clio-overlord bot commented Jul 27, 2022

Overlord detected package changes in this pull request. As the PR author, you are responsible for completing the checklist to ensure these packages are safe to use at Clio.

😎 1 package approved and don't require manual review (click to expand)

🎉 13/19 checks passed on 4 packages

🕵️‍♀️ 4 packages require you to perform manual review: Playbook: How to Review a 3rd Party Package


ffi-compiler @ 1.0.1:

llhttp-ffi @ 0.4.0:

  • Package needs license review. License info could not be pulled automatically from repo.
  • Package has no recent commits, ensure it is actively maintained

http @ 5.1.0:

  • Verify the following keywords are safely used: `, strict_encode64, HTTP, Tempfile, JSON

http-form_data @ 2.3.0:

Disclaimer: Any package updates to this PR will reset this review checklist. To avoid losing your work, ensure you have completed all package updates/additions before beginning the review process. Questions, concerns, or errors? Check here or contact appsec in #product-overlord.

Generated by Overlord against bb53baf

@arian-kh arian-kh requested review from deanna-cooney and removed request for deanna-cooney July 27, 2022 21:08
@arian-kh arian-kh changed the title Remove the version restriction for http gem if its not needed WIP: Remove the version restriction for http gem if its not needed Jul 27, 2022
@arian-kh arian-kh merged commit bb53baf into main Aug 10, 2022
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.

None yet

1 participant