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 13 years ago

Closed 13 years ago

#445 closed enhancement (fixed)

Désactiver les sessions pour cmdline.php

Reported by: hugues Owned by: Julien
Priority: low Milestone: Jelix 1.0.3
Component: jelix-scripts Version: 1.0.1
Severity: trivial Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

Ça serait top de désactiver la session pour les scripts qui tournent via cmdline.php

Attachments (1)

445-jSession-no-session-for-cmdline-requests.diff (751 bytes) - added by Julien 13 years ago.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 13 years ago by laurentj

  • Summary changed from désactivé session pour cmdline.php to Désactiver lesøsessions pour cmdline.php

comment:2 Changed 13 years ago by laurentj

  • Summary changed from Désactiver lesøsessions pour cmdline.php to Désactiver les sessions pour cmdline.php

comment:3 Changed 13 years ago by laurentj

  • Priority changed from normal to low
  • Severity changed from normal to trivial

comment:4 Changed 13 years ago by Julien

  • Documentation needed unset
  • Owner set to Julien

comment:5 Changed 13 years ago by Julien

  • Status changed from new to assigned

comment:6 Changed 13 years ago by Julien

  • review set to review?

Voici le patch.

J'ai fait au plus simple, à savoir tester le type de requête auquel on répond.

comment:7 Changed 13 years ago by laurentj

Ça ne pose pas de problème le session_write_close(); si il n'y a pas eu le session_start() ?

comment:8 Changed 13 years ago by Julien

D'après les tests pratiques non, pas de problème.

L'idée pourrait être de rajouter une propriété :

 protected $_started = false;

que l'on passerait à true lors du session_start effectif.

Du coup, jSession::end() vérifierai cet état pour savoir s'il faut faire le session_write_close().

Re-patch dans ce sens ?

comment:9 Changed 13 years ago by laurentj

  • Milestone set to Jelix 1.0.3
  • review changed from review? to review+

laissons comme ça, on verra si on a des remontées de bug.

Patch à appliquer dans la branche et le trunk.

comment:10 Changed 13 years ago by Julien

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

Ok, patch intégré.

Note: See TracTickets for help on using tickets.