Skip to content

bug: handle invalid URLs in site CSV uploads#504

Open
yann120 wants to merge 4 commits intomainfrom
bug/csv-invalid-url
Open

bug: handle invalid URLs in site CSV uploads#504
yann120 wants to merge 4 commits intomainfrom
bug/csv-invalid-url

Conversation

@yann120
Copy link
Copy Markdown
Collaborator

@yann120 yann120 commented Mar 27, 2026

#505

Summary

Fixes a 500 error when importing a CSV containing invalid URLs.

Examples of invalid URLs handled by this change:

  • http://
  • /contact
  • example.com
  • https://exa mple.com/

Changes

  • detect invalid URL rows during site CSV import
  • log invalid rows with filename and line number
  • add a user-facing validation error instead of crashing
  • continue parsing the file to collect multiple invalid rows

Result

Site CSV imports no longer return a 500 for invalid URLs. Invalid rows are reported clearly, and valid rows are still parsed.

@yann120 yann120 self-assigned this Mar 27, 2026
@yann120 yann120 added the bug Something isn't working label Mar 27, 2026
@yann120 yann120 force-pushed the bug/csv-invalid-url branch from 99f24e9 to 33df965 Compare March 27, 2026 12:56
@yann120 yann120 marked this pull request as ready for review March 27, 2026 12:58
@yann120 yann120 requested a review from Josyann March 27, 2026 12:58
@yann120 yann120 linked an issue Mar 30, 2026 that may be closed by this pull request
@yann120 yann120 requested a review from Josyann April 1, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Ajouter un CSV avec une URL invalide retourne une erreur 500

2 participants