[PLA-2023] Remove without verifying

[enjinabner]

PR Type

enhancement

---

Description

• Removed the withoutVerifying() method call from the HTTP client configuration in the getClient method of the HttpAbstract class.
• This change enhances the security by ensuring SSL certificates are verified during HTTP requests.

---

Changes walkthrough 📝

	Relevant files
Enhancement	HttpAbstract.php Remove `withoutVerifying()` from HTTP client configuration src/Clients/Abstracts/HttpAbstract.php Removed withoutVerifying() method call from the HTTP client configuration. +1/-1

---

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

[github-actions]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ No major issues detected

[github-actions]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
Security	Enhance security by enabling SSL certificate verification Consider implementing SSL certificate verification for the HTTP client to ensure secure data transmission and prevent potential man-in-the-middle attacks. src/Clients/Abstracts/HttpAbstract.php [18] -return Http::retry(3, 500)->timeout(60)->asJson()->acceptJson(); +return Http::retry(3, 500)->timeout(60)->withVerifying()->asJson()->acceptJson(); Suggestion importance[1-10]: 9 Why: The suggestion to enable SSL certificate verification addresses a critical security concern by preventing potential man-in-the-middle attacks. This change enhances the security of HTTP requests, making it highly relevant and impactful.	9