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

[5.4] Fix URL validation pattern #27594

Closed
wants to merge 1 commit into from
Closed

Conversation

rlanvin
Copy link

@rlanvin rlanvin commented Feb 20, 2019

Backport of #25194.

@ahinkle
Copy link
Contributor

ahinkle commented Feb 20, 2019

5.4 is no longer supported.

@rlanvin
Copy link
Author

rlanvin commented Feb 20, 2019

@ahinkle Yes I know, but I'm stuck with it for a project. Can you at least merge? You don't need to make a new release or anything.

@devcircus
Copy link
Contributor

The branch will never be overwritten so if it is an emergency just fix in vendor? Or pay JMAC a few dollars to Shift to 5.5 LTS.

@driesvints
Copy link
Member

Heya, unfortunately we don't support this version anymore. Please check out our support policy on which versions we are currently supporting. Can you please try to upgrade to the latest version and see if your problem persists? If so feel free to reply and we'll try to have a look.

@driesvints driesvints closed this Feb 20, 2019
@rlanvin
Copy link
Author

rlanvin commented Feb 20, 2019

@driesvints Of course the problem has been fixed in the latest version since this is a backport of said fix... If we could upgrade we would, but at this point we're stuck with 5.4, and it's not working on PHP 7.3 because of this bug (that is literally one character). There was another PHP 7.3 compatibility fix merged with 5.4 in December, so I'm a bit confused as to why this one is rejected.

@ahinkle
Copy link
Contributor

ahinkle commented Feb 21, 2019

@rlanvin - @devcircus Provided you a couple of resolutions. The Laravel core contributors can't track every single branch for potential breaking changes. The support for 5.4 has been dropped for well over a year now. There are significant security fixes along with bug fixes (like the one you had listed) with recent Laravel releases. If you are "stuck" take a look at Laravel Shift. It's painless and very affordable.

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.

4 participants