fix(security): remove insecure SSL verification bypass in dataset downloaders

[rgsl888prabhu]

Summary

• Remove ssl.CERT_NONE and check_hostname = False bypass for plato.asu.edu URLs in dataset download scripts
• The site has a valid SSL certificate — the bypass was unnecessary and flagged by SonarQube as 2 Medium-severity vulnerabilities
• Clean up unused import ssl

Fixes SonarQube rules python:S4830 and python:S5527 (the only 2 open security vulnerabilities in the report).

Test plan

• Run regression/get_datasets.py and verify plato.asu.edu downloads succeed without SSL bypass
• Run benchmarks/linear_programming/utils/get_datasets.py and verify same

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 4ac0d007-778f-4104-9f34-6eede1ba9f31

📥 Commits

Reviewing files that changed from the base of the PR and between 1f2fb22 and b08ae44.

📒 Files selected for processing (2)

• benchmarks/linear_programming/utils/get_datasets.py
• regression/get_datasets.py

---

📝 Walkthrough

Walkthrough

Removes SSL certificate verification bypass logic that was conditionally applied to plato.asu.edu URLs in both files. Downloads now uniformly use default urllib.request.urlopen() behavior for all domains, eliminating custom SSL context handling and associated conditional branching.

Changes

Cohort / File(s)	Summary
SSL Bypass Removal benchmarks/linear_programming/utils/get_datasets.py, regression/get_datasets.py	Removed conditional SSL context that disabled hostname checking and certificate verification for plato.asu.edu domains. Eliminated ssl module import and replaced context manager with direct urllib.request.urlopen() call, unifying download behavior across all URLs.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: removing insecure SSL verification bypass logic from dataset downloaders, which is the core change in both modified files.
Description check	✅ Passed	The description is directly related to the changeset, explaining the removal of SSL bypass logic, the rationale (plato.asu.edu has valid certificate), SonarQube violations addressed, and a test plan to verify the changes.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[rgsl888prabhu]

/merge