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.

Fixed

Assignee

Pothys - MitrahSoft

Reporter

Joe Wakefield

Labels

None

Fix versions

Affects versions

Priority

Major