Skip to content

(bug) fix registrar seed TLS verification#530

Merged
angelina-ji merged 1 commit into
masterfrom
bug/local-course-seed-tls
May 3, 2026
Merged

(bug) fix registrar seed TLS verification#530
angelina-ji merged 1 commit into
masterfrom
bug/local-course-seed-tls

Conversation

@Ammaar-Alam

@Ammaar-Alam Ammaar-Alam commented May 1, 2026

Copy link
Copy Markdown
Member

What does this PR do?

  • Uses Certifi's CA bundle for registrar seeding requests.
  • Fixes local course seed/import TLS failures without changing production data.
  • Keeps the scraper change isolated to request verification setup.

Why?

Local course seeding can fail when the Python/OpenSSL trust store cannot validate the registrar endpoint. Using Certifi gives the scraper a stable CA bundle without weakening TLS verification.

How to test

  • Run the registrar course seed/import path locally.
  • Confirm requests verify successfully without disabling TLS verification.

Screenshots

No UI surface changes in this PR.

Verification

  • Django tests passed locally as part of dashboard branch verification: 19 passed.
  • Local database only: DATABASE_URL=postgres:///tigerpath.

Note

This entire PR was automatically implemented, tested, and written by Codex.

@Ammaar-Alam Ammaar-Alam changed the title Fix registrar seed TLS verification (bug) fix registrar seed TLS verification May 1, 2026
@Ammaar-Alam

Copy link
Copy Markdown
Member Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. 👍

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@angelina-ji angelina-ji self-requested a review May 3, 2026 18:27

@angelina-ji angelina-ji left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@angelina-ji angelina-ji merged commit ceb292c into master May 3, 2026
1 check passed
@angelina-ji angelina-ji deleted the bug/local-course-seed-tls branch May 3, 2026 18:28
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