Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Underscores from project name no longer used to create URLs #119

Open
mpbixal opened this issue Feb 26, 2024 · 2 comments
Open

Underscores from project name no longer used to create URLs #119

mpbixal opened this issue Feb 26, 2024 · 2 comments

Comments

@mpbixal
Copy link

mpbixal commented Feb 26, 2024

Using the following .lando.yml:

name: just_a_test
recipe: drupal10

Lando v3.20.8:

APPSERVER URLS
https://localhost:32884 [200]
http://localhost:32885 [200]
http://just_a_test.lndo.site/ [200]
https://just_a_test.lndo.site/ [200]

Lando v3.21.0-beta.1:

APPSERVER URLS
https://localhost:32869 [200]
http://localhost:32870 [200]
http://justatest.lndo.site/ [200]
https://justatest.lndo.site/ [200]

With the new beta, the _ from the project name are no longer used to create the URLs.

Is this intended functionality? I did not see it in the extended release notes

@pirog pirog transferred this issue from lando/lando Feb 26, 2024
@reynoldsalec reynoldsalec transferred this issue from lando/drupal Feb 26, 2024
@reynoldsalec
Copy link
Member

I was able to replicate this issue in Laravel, so I suspect it's a @lando/core issue, transferred here.

@reynoldsalec
Copy link
Member

reynoldsalec commented Mar 14, 2024

@pirog filled me in on the details here; this was a change introduced by lando/core when we traded the transliteration package for slugify as the tool the project names into URL-friendly values. slugify is more strict about URL best practices; underscores are technically acceptable for URLs, but not recommended, hence why slugify strips them.

At this point we think the breaking change is probably acceptable, but it certainly needs to be documented and prominently announced in the blog post for v3.21, since it'll be a breaking change for anyone relying on an underscore URL in a settings.php-style file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants