developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

#504 closed enhancement (fixed)

jSession : configuration about (un)shared sessions should be done in jSession and not in jCoordinator

Reported by: Julien Owned by: Julien
Priority: normal Milestone: Jelix 1.0.3
Component: jelix:core:jSession Version: 1.0.2
Severity: trivial Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

If sessions are not shared, then in jCoordinator we have : 

//make sure that the session cookie is only for the current application
if(!$gJConfig->sessions['shared_session'])
   session_set_cookie_params ( 0 , $gJConfig->urlengine['basePath']);

Theses lines should better be in jSession::start()

Attachments (1)

504-jSession-unshared-sessions.diff​ (1.9 KB) - added by Julien 12 years ago.

Download all attachments as: .zip

Change History (3)

Changed 12 years ago by Julien

comment:1 Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.0.3
  • review set to review+
  • Severity changed from normal to trivial

ok. You can land it into branch 1.0.x and trunk.

comment:2 Changed 12 years ago by Julien

  • Resolution set to fixed
  • Status changed from new to closed

Ok, the patch is landed in the trunk and the 1.0.x branch

Note: See TracTickets for help on using tickets.