expressjs/session

update Readme to include details on extending Store implementation from store.js

Open

#349 aperta il 30 ago 2016

Vedi su GitHub
 (4 commenti) (2 reazioni) (0 assegnatari)JavaScript (977 fork)batch import
docshelp wanted

Metriche repository

Star
 (6073 star)
Metriche merge PR
 (Merge medio 8g) (2 PR mergiate in 30 g)

Descrizione

As noted in a comment on #97, developers who want to write a Store implementation either need to:

  1. extend their Store from express-session's store.js, which typically means requiring express-session as a dependency
  2. implement their own version of the functionality in store.js, session.js, and cookie.js (or copy the files from express-session)

This is because express-session requires the following methods on the Store in order to work:

  • regenerate()
  • load()
  • createSession()

These are available in store.js, but createSession itself requires both session.js and cookie.js.

The documentation should be updated to provide guidance on these facts.

Guida contributor

Direzione di ricerca
Esamina il commento riferito nell'issue #97 e analizza i file store.js, session.js e cookie.js nel repository per comprendere i metodi necessari per estendere Store. Aggiorna il README per spiegare la dipendenza da express session e la necessità dei metodi regenerate(), load() e createSession().
Tech stack
javascriptnodejs
Dominio
backend
Tipo issue
Documentazione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitNode.jsExpress
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75