[#UP-863] Portlets shown in place of other portlets when not logged in to the portal

uPortal

Portlets shown in place of other portlets when not logged in to the portal

Created: 11/Mar/05 10:30 AM Updated: 07/Feb/08 05:58 PM

Component/s:

Portlet Container

Affects Version/s:

2.4.1, 2.4.2, 2.5.0 M1, 2.6.0 M1, 2.5.1 RC1, 2.5.0 RC1, 2.5.0 RC3, 2.5.0 RC2, 2.5.0 GA, 2.5.1 RC2, 2.5.2 RC1, 2.4.4, 2.4.3, 2.5.1 RC3, 2.5.1 GA, 2.4.3.1, 2.5.3 RC1, 2.5.2 GA, 2.5.3 RC2, 2.5.3 RC3, 2.5.3 GA, 2.6.0 RC1, 2.6.0 RC2, 2.5.3.1, 2.6.0 GA, 2.6.1 RC1, 2.6.1 RC2

Fix Version/s:

2.6.1

Original Estimate:

3 days

Remaining Estimate:

3 days

Time Spent:

Unknown

File Attachments:

None

Image Attachments:

1. screenshot-1.jpg
(256 kb)

Environment:

Linux, java 1.4.2, HSQLDB 1.7

Issue Links:

Duplicate

This issue is duplicated by:
	~~UP-1869~~ DLM Guest Layout unable to properly h...

Description

« Hide

I labeled this as "Blocker" because we are trying to go live with a uPortal-based bioinformatics portal at Argonne National Labs on March 15. We really need this to work, so suggestions for workarounds will also be appreciated.

We are trying to add tabs with portlets to the guest/anonymous user view, but this only works correctly for the first tab/portlet that you add. For an example of the problem, see http://gf2.ucs.indiana.edu:8080/uPortal/. Click the tab "Portlet 1" and you will see the Google portlet correctly loaded. Click the "Portlet 2" tab and you will also see (incorrectly) the Google portlet. Tab 2 should display the Pluto Test Suite portlet. If you log into the guest account (guest/guest), you will see that these portlets now behave correctly. We have reproduced this problem with several other portlets, so the Google and Test Suite portlets are not special cases.

Below is a basic summary:
* Start clean with a new installation of uPortal 2.4.2 or 2.4.1 into Tomcat 5.5.4
* Enable the guest account using "ant md5password -Dusername=guest".
* Log in as guest, add 2 or more tabs, and place one CHANNEL (number guesser, cartoon, word of the day, etc) into each tab for content.
* Log out, restart tomcat, and see the tabs and channels work correctly for the anonymous view. CHANNELS WORK CORRECTLY.
* Now log in as guest and add 2 or more tabs.
* Add one portlet (google, testsuite) to each of the new tabs.
* Logout, restart tomcat, go to uportal start page.
* Second portlet tab on the guest view now works INCORRECTLY, as described above.
* Log in as guest. The portlet in the second tab now works CORRECTLY.

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

ho ngoc huy [01/Jun/05 10:18 PM]

I have same error ~~UP-863~~.

And why http://jasigch.princeton.edu don't have error ~~UP-863~~?

How do I can't fixed error ~~UP-863~~?

[ Permlink | « Hide ]

ho ngoc huy [01/Jun/05 10:20 PM]

Sorry:

I have same error ~~UP-863~~.

And why http://jasigch.princeton.edu don't have error ~~UP-863~~?

How do I can fixed error ~~UP-863~~?

[ Permlink | « Hide ]

Andrew Petro [01/Jun/05 10:59 PM]

I assume this issue also affects 2.5.0?

[ Permlink | « Hide ]

ly van thanh [02/Jun/05 05:54 AM]

Example: "Huong Dan Dang Ky Kinh Doanh" Portlet and "Dang Ky Kinh Doanh" Portlet don't work correctly.

[ Permlink | « Hide ]

Matt Young [08/Jun/05 10:26 AM]

We had (and still have this problem) on 2.3.x and 2.4.x... however, I was unable to help the uP people re-produce the problem. To correct it, I made a change to CPortletAdapter.java

public ChannelCacheKey generateKey(String uid) {
ChannelState channelState = (ChannelState)channelStateMap.get(uid);
ChannelStaticData staticData = channelState.getStaticData();

// DUKE MCY Commented out the isGuest part of things.

ChannelCacheKey cck = null;
// Anonymously accessed pages can be cached system-wide
// if(staticData.getPerson().isGuest()) { // cck = systemCacheKey; // } else { cck = instanceCacheKey; // }
return cck;
}

I don't know if this is the right solution, but it has been working in production for us for 1+ years.

[ Permlink | « Hide ]

Matt Young [08/Jun/05 10:32 AM]

The above solution does give us at least one problem though – portlet settings for 'guest' end up shared across anyone who is using the guest account - i.e. our event calendar portlet will show events for whatever date a guest user last selected, even if it isn't the same session. Ideally, guests would be first presented with that day's events and then see whatever day's events they chose, but keep that selection limited to their session.

[ Permlink | « Hide ]

ly van thanh [10/Jun/05 05:07 AM]

Thank Matt Young very much. Your comment helps me fix this error.

[ Permlink | « Hide ]

Andrew Petro [14/Jul/05 12:05 PM]

Is there a way to solve this problem while supporting different state for different users of the portlet? How can we progress towards a concensus patch to fix this issue?

[ Permlink | « Hide ]

Andreas Christoforides [28/Jul/05 07:59 PM]

I was having the same issue with a stateful portlet in the guest layout. Changing the caching logic in the CPortletAdapter as shown above resolved the issue. The original implementation assumed that all portlets appearing in a guest layout will be have system-wide content.

[ Permlink | « Hide ]

Keith Zantow [06/Dec/05 01:14 PM]

I believe that the same issue is discussed here: ~~UP-1040~~ including a workaround. The workaround may be viable if portlet caching can be disabled per-portlet, otherwise it introduces a large amount of overhead.

[ Permlink | « Hide ]

Keith Zantow [06/Dec/05 01:16 PM]

I should mention, that this is a workaround more for interactive portlets, including guest portlets, I believe. it may require the aforementioned fix also.

[ Permlink | « Hide ]

Michał Politowski [25/Apr/06 05:48 AM]

~~UP-1436~~ is a duplicate of this issue.

[ Permlink | « Hide ]

Gary Roybal [07/Feb/08 05:48 PM]

~~UP-1869~~ is a duplicate of this issue. The fix for ~~UP-1869~~ has been applied to 2.6.1, so this issue can probably be closed.

[ Permlink | « Hide ]

Andrew Petro [07/Feb/08 05:55 PM]

Closed as a duplicate of ~~UP-1869~~.

[ Permlink | « Hide ]

Andrew Petro [07/Feb/08 05:56 PM]

Re-opening to fix fix version.

[ Permlink | « Hide ]

Andrew Petro [07/Feb/08 05:57 PM]

Fixed affects and fix versions.

[ Permlink | « Hide ]

Andrew Petro [07/Feb/08 05:58 PM]

Closed as duplicate of ~~UP-1869~~.