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

Bug: Url's not working #3245

Closed
InsiteFX opened this issue Jul 7, 2020 · 10 comments · Fixed by #3262
Closed

Bug: Url's not working #3245

InsiteFX opened this issue Jul 7, 2020 · 10 comments · Fixed by #3262
Labels
bug Verified issues on the current code behavior or pull requests that will fix them

Comments

@InsiteFX
Copy link
Contributor

InsiteFX commented Jul 7, 2020

Direction
We use github issues to track bugs, not for support.
If you have a support question, or a feature request, raise these as threads on our
forum.

Describe the bug
A clear and concise description of what the bug is.
Downloaded newest developer version install using my same vhost. setup all config
load index.php goes right to the home page. type in home on bar and get 404 page not found on all links
https://admin.local/ --. works
https://admin.local/home --> 404 error page not found
Does the same thing with all the Myth/Auth links 404 page not found.

So I downloaded the production 4,0,3 version and guess what everything works right out of the box.
with the same setup parameters.

CodeIgniter 4 version
Which version (and branch, if applicable) the bug is in.
This developer version this branch

Affected module(s)
Which package or class is the bug in, if known.
Seems to be something with the routes or router.

Expected behavior, and steps to reproduce if appropriate
A clear and concise description of what you expected to happen,
and how you got there.
Feel free to include a text/log extract, but use a pastebin facility for any
screenshots you deem necessary.
Expected all the url links to work.
Context

  • OS: [e.g. Windows 99]
  • Web server [e.g. Apache 1.2.3]
  • PHP version [e.g. 6.5.4]
@InsiteFX InsiteFX added the bug Verified issues on the current code behavior or pull requests that will fix them label Jul 7, 2020
@MGatner
Copy link
Member

MGatner commented Jul 7, 2020

Could you share at least the relevant parts of app/Config/Routes.php and any changes you have made to public/.htaccess?

@InsiteFX
Copy link
Contributor Author

InsiteFX commented Jul 7, 2020

Nothing has been changed in the Routes.php or the ,htaccess file only change is to the app.php config
#BaseUrl = 'https://admin.local/ and removed index.php from the config.

The released ver 4.0.3 worked right off the developer version will not work at all. Same changes made to the production framework.

@michalsn
Copy link
Member

michalsn commented Jul 7, 2020

I think it's because of the recent changes in .htaccess.

I'm not sure if we should use all these flags by default. Personally I would stay with the initial rule:

RewriteRule ^(.*)$ index.php/$1 [L]

@MGatner
Copy link
Member

MGatner commented Jul 7, 2020

@InsiteFX could you try reverting that change (f0e6f60) and see if it works? If that fails revert this one and try again: 7c8d3bf

@InsiteFX
Copy link
Contributor Author

InsiteFX commented Jul 8, 2020

This one is the problem!

I reverted this one and everything started working again --> (f0e6f60)

@MGatner
Copy link
Member

MGatner commented Jul 8, 2020

@lonnieezell ’s in trouble 😂

@paulbalandan
Copy link
Member

Adding square brackets on \s\S works for me.

RewriteRule ^([\s\S]*)$ index.php/$1 [L,NC,QSA]

@MGatner
Copy link
Member

MGatner commented Jul 9, 2020

@InsiteFX can you try that with square brackets?

@lonnieezell
Copy link
Member

Square brackets makes sense there.

@InsiteFX
Copy link
Contributor Author

InsiteFX commented Jul 9, 2020

Yes, this did work I try several url's like to home and Myth/Auth login and register all work with these square brackets.

RewriteRule ^([\s\S]*)$ index.php/$1 [L,NC,QSA]

@paulbalandan paulbalandan mentioned this issue Jul 10, 2020
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Verified issues on the current code behavior or pull requests that will fix them
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants