PUBLIC - Liferay Portal Community Edition

CKEditor generates invalid links to documents when portal.proxy.path is set

Details

  • Branch Version/s:
    6.1.x, 6.0.x
  • Backported to Branch:
    Committed
  • Similar Issues:
    Show 3 results 

Description

When using a reverse proxy and the portal is configured to use a path component in the URLs specified by portal.proxy.path property, the CKEditor's FileManager will select files without this path prefix which causes the portal to not find the related file when user clicks on the link.

Steps to reproduce:

1.) Setup Apache to connect Tomcat through AJP:
LoadModule proxy_ajp_module libexec/apache2/mod_proxy_ajp.so
...
ProxyPass /lfr ajp://localhost:8009/
ProxyPassReverse /lfr ajp://localhost:8009/
2.) Set portal to use proxy path in portal-ext.properties:
portal.proxy.path=/lfr
3.) Start Tomcat
4.) Start Apache
5.) Navigate to http://localhost/lfr and login
6.) Navigate to Control Panel > Document And Media
7.) Upload a file like test.doc
8.) Add a Web Content Display to the opening page
9.) Create new Web Content
10.) Insert link into Web Content and select 'Browse Server'
11.) Navigate to the folder where you uploaded test.doc to and select it

Expected behavior:
The selected URL should start as /lfr/documents/

Current behavior:
The selected URL starts with /documents/

Issue Links

is related to

Bug - A problem which impairs or prevents the functions of the product. LPS-27519 Resource importer throws a NullPointerException when matching DL Files in Web Content

  • Minor - Minor loss of function, or other problem where easy workaround is present.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Michael Saechang added a comment -

Committed on:
6.1.x GIT ID: 23803efb127684a57239632eb082a9d837175ff2.
6.2.x GIT ID: ada97de9b186311a38f58126375596ed245b2f85.

Show
Michael Saechang added a comment - Committed on: 6.1.x GIT ID: 23803efb127684a57239632eb082a9d837175ff2. 6.2.x GIT ID: ada97de9b186311a38f58126375596ed245b2f85.
Hide
Permalink
Pani Gui added a comment -

PASSED Manual Testing following the steps in the description.

Reproduced on:
Tomcat 6.0 + MySQL 5. 6.0.12 EE.
Tomcat 7.0 + MySQL 5. 6.1.10 CE.

The link for document URL starts with /documents/.

Fixed on:
Tomcat 6.0 + MySQL 5. 6.0.x GIT ID: 63d6e7b854e8a2ccb994adfcae22ae751c0e2517.
Tomcat 7.0 + MySQL 5. 6.1.x EE GIT ID: 695b2376cd8db5015cbbb9036bd4ed331736391b.
Tomcat 7.0 + MySQL 5. 6.2.x GIT ID: c4c1edc8f3f2c5d652ea866787b5a8e0294b4d6a.

The link for document URL starts as /lfr/documents/.

Show
Pani Gui added a comment - PASSED Manual Testing following the steps in the description. Reproduced on: Tomcat 6.0 + MySQL 5. 6.0.12 EE. Tomcat 7.0 + MySQL 5. 6.1.10 CE. The link for document URL starts with /documents/. Fixed on: Tomcat 6.0 + MySQL 5. 6.0.x GIT ID: 63d6e7b854e8a2ccb994adfcae22ae751c0e2517. Tomcat 7.0 + MySQL 5. 6.1.x EE GIT ID: 695b2376cd8db5015cbbb9036bd4ed331736391b. Tomcat 7.0 + MySQL 5. 6.2.x GIT ID: c4c1edc8f3f2c5d652ea866787b5a8e0294b4d6a. The link for document URL starts as /lfr/documents/.
Hide
Permalink
Pani Gui added a comment -

PASSED Manual Testing following the steps in the description.

Fixed on:
Tomcat 7.0 + MySQL 5. Portal 6.1.x CE GIT ID: 974e58c9630c9471da61a56dc052083a17c05b72.

Show
Pani Gui added a comment - PASSED Manual Testing following the steps in the description. Fixed on: Tomcat 7.0 + MySQL 5. Portal 6.1.x CE GIT ID: 974e58c9630c9471da61a56dc052083a17c05b72.
Hide
Permalink
Michael Saechang added a comment -

Thank you Pani for testing. Closing as 'Fixed'.

Show
Michael Saechang added a comment - Thank you Pani for testing. Closing as 'Fixed'.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
    Days since last comment:
    1 year, 3 weeks ago