Switching to a different session store leaves memstore goroutine unreleased

[chucamphong]

scs.New() automatically invokes memstore.New(), which creates a goroutine to handle expired session cleanup. Because memstore is the default store, switching to a different store (like gormstore) leaves the memstore goroutine active and unreleased

scs/session.go

Line 106 in 7e11d57

Store: memstore.New(),

// Initialize a new session manager and configure it to use gormstore as the session store.
sessionManager = scs.New()
if sessionManager.Store, err = gormstore.New(db); err != nil {
    log.Fatal(err)
}

Workaround solution:

if memStore, ok := sessionManager.Store.(*memstore.MemStore); ok {
    // HACK: Not sure why, but it seems a delay is needed before stopping the cleanup goroutine.
    time.Sleep(time.Second)
    memStore.StopCleanup()
}

if sessionManager.Store, err = gormstore.New(db); err != nil {
    log.Fatal(err)
}