Skip to content

Commit bd2f4ad

Browse files
committed
Fix store_session impl
Fixes #3
1 parent 9a62fc1 commit bd2f4ad

File tree

2 files changed

+9
-5
lines changed

2 files changed

+9
-5
lines changed

Diff for: Cargo.toml

+1
Original file line numberDiff line numberDiff line change
@@ -22,6 +22,7 @@ mongodb = { git = "https://github.com/saghm/mongo-rust-driver", default-features
2222
async-session = "2.0.0"
2323
async-trait = "0.1.36"
2424
serde_json = "1.0.56"
25+
# chrono = "0.4.13"
2526

2627
[dev-dependencies]
2728
async-std = { version = "1.6.2", features = ["attributes"] }

Diff for: src/lib.rs

+8-5
Original file line numberDiff line numberDiff line change
@@ -12,8 +12,11 @@
1212

1313
use async_session::{Result, Session, SessionStore};
1414
use async_trait::async_trait;
15+
// use mongodb::bson::{doc, DateTime};
1516
use mongodb::bson::doc;
17+
use mongodb::options::ReplaceOptions;
1618
use mongodb::Client;
19+
// use chrono::offset::Utc;
1720

1821
/// A MongoDB session store.
1922
#[derive(Debug, Clone)]
@@ -48,19 +51,19 @@ impl SessionStore for MongodbSessionStore {
4851
// TODO: mongodb supports TTL for auto-expiry somehow, need to figure out how!
4952
let value = serde_json::to_string(&session)?;
5053
let id = session.id();
51-
let doc = doc! { "session_id": id, "session": value };
52-
coll.insert_one(doc, None).await?;
54+
let query = doc! { "session_id": id };
55+
let replacement = doc! { "session_id": id, "session": value };
56+
let opts = ReplaceOptions::builder().upsert(true).build();
57+
coll.replace_one(query, replacement, Some(opts)).await?;
5358

5459
Ok(session.into_cookie_value())
5560
}
5661

5762
async fn load_session(&self, cookie_value: String) -> Result<Option<Session>> {
5863
let id = Session::id_from_cookie_value(&cookie_value)?;
5964
let coll = self.client.database(&self.db).collection(&self.coll_name);
60-
6165
let filter = doc! { "session_id": id };
62-
let result = coll.find_one(filter, None).await?;
63-
match result {
66+
match coll.find_one(filter, None).await? {
6467
None => Ok(None),
6568
Some(doc) => Ok(Some(serde_json::from_str(doc.get_str("session")?)?)),
6669
}

0 commit comments

Comments
 (0)