Skip to content

Commit 71183ae

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

File tree

1 file changed

+6
-5
lines changed

1 file changed

+6
-5
lines changed

Diff for: src/lib.rs

+6-5
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
use async_session::{Result, Session, SessionStore};
1414
use async_trait::async_trait;
1515
use mongodb::bson::doc;
16+
use mongodb::options::ReplaceOptions;
1617
use mongodb::Client;
1718

1819
/// A MongoDB session store.
@@ -48,19 +49,19 @@ impl SessionStore for MongodbSessionStore {
4849
// TODO: mongodb supports TTL for auto-expiry somehow, need to figure out how!
4950
let value = serde_json::to_string(&session)?;
5051
let id = session.id();
51-
let doc = doc! { "session_id": id, "session": value };
52-
coll.insert_one(doc, None).await?;
52+
let query = doc! { "session_id": id };
53+
let replacement = doc! { "session_id": id, "session": value };
54+
let opts = ReplaceOptions::builder().upsert(true).build();
55+
coll.replace_one(query, replacement, Some(opts)).await?;
5356

5457
Ok(session.into_cookie_value())
5558
}
5659

5760
async fn load_session(&self, cookie_value: String) -> Result<Option<Session>> {
5861
let id = Session::id_from_cookie_value(&cookie_value)?;
5962
let coll = self.client.database(&self.db).collection(&self.coll_name);
60-
6163
let filter = doc! { "session_id": id };
62-
let result = coll.find_one(filter, None).await?;
63-
match result {
64+
match coll.find_one(filter, None).await? {
6465
None => Ok(None),
6566
Some(doc) => Ok(Some(serde_json::from_str(doc.get_str("session")?)?)),
6667
}

0 commit comments

Comments
 (0)