Skip to content

Commit 5fed5a2

Browse files
committed
Rust: It turns out Poem defaults 'secure' to true.
1 parent 7383e4f commit 5fed5a2

File tree

1 file changed

+4
-4
lines changed
  • rust/ql/test/query-tests/security/CWE-614

1 file changed

+4
-4
lines changed

rust/ql/test/query-tests/security/CWE-614/main.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -233,9 +233,9 @@ fn test_poem() {
233233
cookie2.set_secure(true); // good
234234
jar.add(cookie2.clone());
235235

236-
// secure left as default
237-
let cookie3 = PoemCookie::new_with_str("name", "value"); // $ MISSING: Source
238-
jar.add(cookie3.clone()); // $ MISSING: Alert[rust/insecure-cookie]
236+
// secure left as default (which is `true` for Poem)
237+
let cookie3 = PoemCookie::new_with_str("name", "value");
238+
jar.add(cookie3.clone()); // good
239239

240240
// set secure via CookieConfig
241241
let cookie_config_bad = poem::session::CookieConfig::new().secure(false);
@@ -248,7 +248,7 @@ fn test_poem() {
248248
_ = poem::session::ServerSession::new(cookie_config_good, ()); // good
249249

250250
let cookie_config_default = poem::session::CookieConfig::new();
251-
_ = poem::session::ServerSession::new(cookie_config_default, ()); // $ MISSING: Alert[rust/insecure-cookie]
251+
_ = poem::session::ServerSession::new(cookie_config_default, ()); // good
252252
}
253253

254254
fn test_http_types() {

0 commit comments

Comments
 (0)