fix: update URLs to use HTTPS in model migration prompts #7705
Conversation
|
All contributors have signed the CLA ✍️ ✅ |
|
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
Pull request overview
This PR updates model migration prompt URLs to use HTTPS protocol instead of plain domain names, improving security and following modern web standards. The change addresses issue #6685 by ensuring all OpenAI documentation links use secure connections.
- Updated URL format from
www.openai.comtohttps://openai.comwith trailing slashes - Applied changes consistently across source code and test snapshots
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| codex-rs/tui/src/model_migration.rs | Updated two URL strings to use HTTPS protocol and proper formatting |
| codex-rs/tui/src/snapshots/codex_tui__model_migration__tests__model_migration_prompt_gpt5_family.snap | Updated snapshot to reflect HTTPS URL for gpt-5.1 model family documentation |
| codex-rs/tui/src/snapshots/codex_tui__model_migration__tests__model_migration_prompt_gpt5_codex_mini.snap | Updated snapshot to reflect HTTPS URL for gpt-5.1 codex-mini documentation |
| codex-rs/tui/src/snapshots/codex_tui__model_migration__tests__model_migration_prompt_gpt5_codex.snap | Updated snapshot to reflect HTTPS URL for gpt-5.1 codex documentation |
| codex-rs/tui/src/snapshots/codex_tui__model_migration__tests__model_migration_prompt.snap | Updated snapshot to reflect HTTPS URL for gpt-5.1-codex-max documentation |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@etraut-openai I fixed formatting issue. Could you rerun CI checks? |
2 participants
Update URLs to use HTTPS in model migration prompts
Closes #6685