Similar to #1635446: problem when path alias contains utf8 characters for the 7.x version, the following valid URLs all responded with 404 when the fast_404 module was enabled:
‘outstanding-efforts’
‘class-act’
social-media-news-–-did-you-know
Steps to replicate
- Create a new content page with the urls alias "outstanding-efforts"
- Visited the page - and verify if it can be accessed
- Enable the module
- Visited the newly created page again should results in 404
- Apply the patch
- Clear cache
- Verify if the page can be accessed now
| Comment | File | Size | Author |
|---|---|---|---|
| #4 | fast_404-encoding-3144403-4.patch | 571 bytes | malcomio |
Comments
Comment #2
malcomio commentedComment #3
malcomio commentedThe previous patch included some unnecessary changes, taken from the patch on #1635446: problem when path alias contains utf8 characters
Comment #4
malcomio commentedSorry - the previous patch had the wrong git root.
Comment #5
joseph.olstadwe are using this patch, thanks it seems to work properly..
here is our working configuration details:
see our configuration details here:
https://www.drupal.org/project/fast_404/issues/2961512#comment-14040732
Comment #6
kristen pol@joseph.olstad Do you feel comfortable RTBCing this since you've tested it?
Comment #7
joseph.olstad@Kristen Pol
The patch appeared to work for us but I didn't spent enough time with the client and reviewing this to be 100% confident. I am no longer doing work for the client that asked me to implement this so I am not sure how well this worked out in the long term.
Comment #8
kristen polThanks for the feedback. Marking for manual testing.
Comment #9
joshua1234511Tested the patch from #4
Steps followed
- Created a new content page with the urls alias "outstanding-efforts"
- Visited the page - can be accessed
- Enabled the module
- Visited the url results in 404
- Applied the patch
- Cleared cache
- The page can be accessed now
Based on the test results marking the issue as RTBC
Comment #10
kristen polThanks @joshua1234511! This is ready to be merged by a maintainer :)
Comment #11
joseph.olstadComment #12
joseph.olstadtriggered tests for 8.x-3.x branch
Comment #13
kristen polMoving this to major as it is a big deal and pretty much equivalent to a "fatal error" without the WSOD. Assigning to me to see if I can get this in.
Comment #16
kristen polThanks, everyone! This has been merged for both 2.x and 3.x and will be in the next 3.x release though I don't think there will be any more 2.x releases.
Comment #17
joseph.olstadhttps://www.drupal.org/project/fast_404/releases/8.x-3.2