Skip to content

Commit 1f8c2ef

Browse files
909: Update to latest conduit-cookie r=carols10cents Will invalidate everyone's cookies, but we're only setting session cookies right now anyway Want to get this change: conduit-rust/conduit-cookie#4
2 parents 946502d + a640d62 commit 1f8c2ef

File tree

4 files changed

+88
-12
lines changed

4 files changed

+88
-12
lines changed

Cargo.lock

Lines changed: 83 additions & 10 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,6 +53,7 @@ chrono = "0.4.0"
5353
conduit = "0.8"
5454
conduit-conditional-get = "0.8"
5555
conduit-cookie = "0.8"
56+
cookie = "0.9"
5657
conduit-json-parser = "0.8"
5758
conduit-log-requests = "0.8"
5859
conduit-middleware = "0.8"

src/lib.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,7 @@ extern crate url;
4343
extern crate conduit;
4444
extern crate conduit_conditional_get;
4545
extern crate conduit_cookie;
46+
extern crate cookie;
4647
extern crate conduit_git_http_backend;
4748
extern crate conduit_json_parser;
4849
extern crate conduit_log_requests;
@@ -219,9 +220,10 @@ pub fn middleware(app: Arc<App>) -> MiddlewareBuilder {
219220

220221
m.around(util::Head::default());
221222
m.add(conduit_conditional_get::ConditionalGet);
222-
m.add(conduit_cookie::Middleware::new(app.session_key.as_bytes()));
223+
m.add(conduit_cookie::Middleware::new());
223224
m.add(conduit_cookie::SessionMiddleware::new(
224225
"cargo_session",
226+
cookie::Key::from_master(app.session_key.as_bytes()),
225227
env == Env::Production,
226228
));
227229
m.add(app::AppMiddleware::new(app));

src/tests/all.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -121,7 +121,7 @@ fn app() -> (record::Bomb, Arc<App>, conduit_middleware::MiddlewareBuilder) {
121121

122122
let config = cargo_registry::Config {
123123
uploader: uploader,
124-
session_key: "test".to_string(),
124+
session_key: "test this has to be over 32 bytes long".to_string(),
125125
git_repo_checkout: git::checkout(),
126126
gh_client_id: env::var("GH_CLIENT_ID").unwrap_or(String::new()),
127127
gh_client_secret: env::var("GH_CLIENT_SECRET").unwrap_or(String::new()),

0 commit comments

Comments
 (0)