If you can avoid creating the session in the first place I think that
would be the best thing to do. The google-able portion of my site
<plug>www.thedailytube.com</plug> only uses the session for the search
form. To avoid creating the session for the form I created a new
persistent field strategy that only creates a session if the value
being stored is non-null. Then, I extended the Tapestry Form component
to hold onto the validator in a non-persisted field unless there are
errors to maintain...
Here is some code that you can use if you like:
****** NonNullSessionPersistentFieldStrategy.java, essentially a dupe
of the SessionPersistentFieldStrategy with null check logic ****
package com.thedailytube.ui.tapestry.services;
import org.apache.tapestry.internal.services.PersistentFieldChangeImpl;
import static org.apache.tapestry.ioc.internal.util.CollectionFactory.newList;
import static org.apache.tapestry.ioc.internal.util.Defense.notBlank;
import org.apache.tapestry.services.PersistentFieldChange;
import org.apache.tapestry.services.PersistentFieldStrategy;
import org.apache.tapestry.services.Request;
import org.apache.tapestry.services.Session;
import java.util.Collection;
import java.util.Collections;
import java.util.List;
/**
* Created by IntelliJ IDEA.
* User: joshcanfield
* Date: Oct 26, 2007
*/
public class NonNullSessionPersistentFieldStrategy implements
PersistentFieldStrategy {
/**
* Prefix used to identify keys stored in the session.
*/
static final String PREFIX = "nonnull:";
private final Request _request;
protected NonNullSessionPersistentFieldStrategy(Request request) {
_request = request;
}
public final Collection<PersistentFieldChange>
gatherFieldChanges(String pageName) {
Session session = _request.getSession(false);
if (session == null) return Collections.emptyList();
List<PersistentFieldChange> result = newList();
String fullPrefix = PREFIX + pageName + ":";
for (String name : session.getAttributeNames(fullPrefix)) {
PersistentFieldChange change = buildChange(name, session.getAttribute(name));
result.add(change);
}
return result;
}
private PersistentFieldChange buildChange(String name, Object attribute) {
// TODO: Regexp is probably too expensive for what we need here.
Maybe an IOC InternalUtils
// method for this purpose?
String[] chunks = name.split(":");
// Will be empty string for the root component
String componentId = chunks[2];
String fieldName = chunks[3];
return new PersistentFieldChangeImpl(componentId, fieldName, attribute);
}
public final void postChange(String pageName, String componentId,
String fieldName, Object newValue) {
notBlank(pageName, "pageName");
notBlank(fieldName, "fieldName");
StringBuilder builder = new StringBuilder(PREFIX);
builder.append(pageName);
builder.append(':');
if (componentId != null) builder.append(componentId);
builder.append(':');
builder.append(fieldName);
// because we don't want to create a session when the object is null
Session session = _request.getSession(newValue != null);
if (session != null) {
session.setAttribute(builder.toString(), newValue);
}
}
}
********************
*** Form.java ****
********************
package com.thedailytube.ui.tapestry.components;
import org.apache.tapestry.ValidationTracker;
import org.apache.tapestry.ValidationTrackerImpl;
import org.apache.tapestry.annotations.Persist;
/**
* Overrides the core [EMAIL PROTECTED]
org.apache.tapestry.corelib.components.Form} in order to store the
validation tracker only
* when there is something to track.
* <p/>
* Created by IntelliJ IDEA.
* User: joshcanfield
* Date: Oct 26, 2007
*/
public class Form extends org.apache.tapestry.corelib.components.Form {
@Persist("nonnull")
private ValidationTracker _tracker;
private ValidationTracker _nonPersistedTracker;
public ValidationTracker getDefaultTracker() {
if (_nonPersistedTracker == null) {
if (_tracker != null) {
// _tracker is loaded via injection magic when it's in the session
_nonPersistedTracker = _tracker;
} else {
_nonPersistedTracker = new ValidationTrackerImpl();
}
}
return _nonPersistedTracker;
}
public void setDefaultTracker(ValidationTracker defaultTracker) {
_nonPersistedTracker = defaultTracker;
}
protected void onAction() {
if (_nonPersistedTracker.getHasErrors()) {
_tracker = _nonPersistedTracker;
} else {
_tracker = null;
}
}
}
Josh
On Tue, Mar 4, 2008 at 10:05 PM, Daniel Leffel <[EMAIL PROTECTED]> wrote:
> Hi,
> We're in the process of developing a webapp using T5. The site leverages
> session persistence on many pages, including a number of pages important for
> the natural search content. It seems to us that levering a filter to strip
> the jessionid from non-cookie enabled user agents would be an effective way
> to keep jsessionids from popping up in the url. Other than insuring all
> pages gracefully handle sessions which are effectively stateless (from
> non-cookied enabled user agents), are there any other concerns we should be
> thinking about?
>
> Danny
>
--
--
TheDailyTube.com. Sign up and get the best new videos on the internet
delivered fresh to your inbox.
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
