getPageContext().getRequest().getRequestUrl() different

Description

Lucee returns the port, ACF doesn't.

// Lucee = http://127.0.0.1:80/test.cfm // ACF10 = http://127.0.0.1/test.cfm // ACF2018 = http://127.0.0.1/test.cfm WriteDump(getPageContext().getRequest().getRequestUrl().toString());

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:

<cfif server.coldfusion.productname EQ "Lucee"> <cfset local.request_url = cgi.request_url> <cfelse> <cfset local.request_url = variables.request.GetRequestUrl()> </cfif>

Activity

Pothys - MitrahSoft 
23 December 2020 at 06:37

, 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.

Michael Offner 
18 December 2020 at 16:45

please give the build a try

Michael Offner 
18 December 2020 at 16:45

Pothys - MitrahSoft 
23 September 2020 at 15:15

I've added a fix for this issue as you said.

Pull request: https://github.com/lucee/Lucee/pull/1046

Joe Wakefield 
18 September 2020 at 13:27

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.)

Fixed

Details

Assignee

Reporter

Labels

New Issue warning screen

Before you create a new Issue, please post to the mailing list first https://dev.lucee.org

Once the issue has been verified, one of the Lucee team will ask you to file an issue

Fix versions

Affects versions

Priority

Created 13 September 2018 at 16:48
Updated 18 September 2024 at 10:19
Resolved 23 December 2020 at 06:38