getPageContext().getRequest().getRequestUrl() different
Description
Lucee returns the port, ACF doesn't.
I actually couldn't find any combination of variables that work the same on Lucee, ACF10, and ACF2018, so the best workaround I came up with is this:
Activity
Show:
Joe Wakefield
September 18, 2020, 11:27 PM
there’s an issue with your pull request. From the comment on github:
The port should only be hidden if 80 or 443. The getRequestUrl() function still needs to append the port in other circumstances (e.g. 8080, 8443, etc.)
Pothys - MitrahSoft
September 24, 2020, 1:15 AM
I've added a fix for this issue as you said.
Pull request: https://github.com/lucee/Lucee/pull/1046
Michael Offner
December 19, 2020, 3:45 AM
Michael Offner
December 19, 2020, 3:45 AM
please give the build a try
Pothys - MitrahSoft
December 23, 2020, 5:37 PM
, I've checked this ticket with lucee version 5.3.8.129-SNAPSHOT. This issue was fixed. So I deployed the ticket with the fixed version.
