Thanks Vladimir! Could have gone either way with this. The other option would have been to not store the context at all with the saved request to save some bytes, but the benefit of doing it this way is that the fallback url could potentially be in a different context (saved request obviously is always in this context)

Kalle Korhonen
added a comment - 08/Mar/12 3:38 PM Thanks Vladimir! Could have gone either way with this. The other option would have been to not store the context at all with the saved request to save some bytes, but the benefit of doing it this way is that the fallback url could potentially be in a different context (saved request obviously is always in this context)

Reverted the earlier patch. The root cause of the issue is the context path shouldn't be stored at all since the
default fallback urls cannot specify context. There are several places where we need to keep this in mind, TYNAMO-144 and TYNAMO-145 are related, should be all fixed and wrote a few more functional tests for verification.

Kalle Korhonen
added a comment - 29/Mar/12 1:44 PM Reverted the earlier patch. The root cause of the issue is the context path shouldn't be stored at all since the
default fallback urls cannot specify context. There are several places where we need to keep this in mind, TYNAMO-144 and TYNAMO-145 are related, should be all fixed and wrote a few more functional tests for verification.