Skip to content

Commit

Permalink
Merge pull request #2464 from zspitzer/LDEV-5230-internalRequest-alwa…
Browse files Browse the repository at this point in the history
…ys-creates-a-session

LDEV-5230 internalRequest should not create sessions
  • Loading branch information
zspitzer authored Jan 1, 2025
2 parents 4eef837 + ab6d39c commit 55fd4db
Showing 1 changed file with 1 addition and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -147,7 +147,7 @@ else if (body != null) {
_pc.flush();
// cookie = _pc.cookieScope().duplicate(false);
request = _pc.requestScope().duplicate(false);
session = sessionEnabled(_pc) ? _pc.sessionScope().duplicate(false) : null;
session = _pc.hasCFSession() ? _pc.sessionScope().duplicate(false) : null;
exeTime = System.currentTimeMillis() - pc.getStartTime();
// debugging=_pc.getDebugger().getDebuggingData(_pc).duplicate(false);

Expand Down Expand Up @@ -244,12 +244,6 @@ private static Struct toStruct(Object obj) throws PageException {
return data;
}

private static boolean sessionEnabled(PageContextImpl pc) {
ApplicationContext ac = pc.getApplicationContext();
if (ac == null) return false;// this test properly is not necessary
return ac.hasName() && ac.isSetSessionManagement();
}

private static void fillForm(PageContextImpl _pc, Struct src, Charset charset) throws PageException {
if (src == null) return;

Expand Down

0 comments on commit 55fd4db

Please sign in to comment.