Skip to content

πŸ”™ get the url validator back#583

Draft
rdubigny wants to merge 1 commit into
mainfrom
the-return-of-the-secure-url
Draft

πŸ”™ get the url validator back#583
rdubigny wants to merge 1 commit into
mainfrom
the-return-of-the-secure-url

Conversation

@rdubigny

@rdubigny rdubigny commented Oct 13, 2025

Copy link
Copy Markdown
Contributor

Problem

We previously had stricter URL validation in production.
The new implementation introduced looser control.
For example, http://localhost should not be allowed in production.

Proposal

Enforce strict URL validation only in production environments (where espace-partenaires is not used).
This allows integration environments to share a simpler, consistent configuration with espace-partenaires.

@rdubigny rdubigny changed the title 🚸 Homogenize URL validation πŸ”™ get the url validator back Oct 13, 2025
@rdubigny

Copy link
Copy Markdown
Contributor Author

@rdubigny
rdubigny force-pushed the the-return-of-the-secure-url branch from 0b0d154 to 11d0acb Compare October 13, 2025 12:44
@rdubigny
rdubigny force-pushed the the-return-of-the-secure-url branch from 11d0acb to 75183c9 Compare October 24, 2025 15:15
@rdubigny
rdubigny marked this pull request as draft October 24, 2025 15:15
@rdubigny
rdubigny force-pushed the the-return-of-the-secure-url branch from 75183c9 to 8d63537 Compare October 24, 2025 15:38
@rdubigny
rdubigny requested a review from gdarquie October 27, 2025 15:41
@rdubigny
rdubigny force-pushed the the-return-of-the-secure-url branch from 8d63537 to 2d77ba7 Compare October 27, 2025 15:43
**Problem**

We previously had stricter URL validation in production.
The new implementation introduced looser control.
For example, http://localhost should not be allowed in production.

**Proposal**

Enforce strict URL validation only in production environments (where espace-partenaires is not used).
This allows integration environments to share a simpler, consistent configuration with espace-partenaires.
@rdubigny
rdubigny force-pushed the the-return-of-the-secure-url branch from 2d77ba7 to 1f661dc Compare October 27, 2025 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant