Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pingora-http test fails on patched_http1 feature #481

Open
vivekkj123 opened this issue Nov 23, 2024 · 0 comments
Open

pingora-http test fails on patched_http1 feature #481

vivekkj123 opened this issue Nov 23, 2024 · 0 comments
Assignees

Comments

@vivekkj123
Copy link

Describe the bug

On running cargo test --all --features=patched_http1 on pingora-http crate, the following error is logged

Compiling pingora-http v0.4.0 (/home/vivekdeb/Debian_Packaging/debcargo-conf/build/pingora-http)
    Finished `test` profile [unoptimized + debuginfo] target(s) in 0.58s
     Running unittests src/lib.rs (target/debug/deps/pingora_http-6e4dc386229d77ba)

running 8 tests
test tests::header_map_upper_bound ... ok
test case_header_name::tests::test_case_header_name ... ok
test tests::set_test_send_end_stream ... ok
test tests::test_reason_phrase ... ok
test tests::test_multiple_header ... ok
test tests::test_invalid_path ... FAILED
test tests::test_single_header ... ok
test tests::test_single_header_no_case ... ok

failures:

---- tests::test_invalid_path stdout ----
thread 'tests::test_invalid_path' panicked at src/lib.rs:715:68:
called `Result::unwrap()` on an `Err` value: Error { etype: InvalidHTTPHeader, esource: Unset, retry: Decided(false), cause: None, context: Some(Owned("invalid uri Hello�World")) }
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace


failures:
    tests::test_invalid_path

test result: FAILED. 7 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s

error: test failed, to rerun pass `--lib`

Pingora version: 0.4.0
Rust version: i.e. cargo --version 1.82.0
Operating system version: Debian Sid

Expected results

All tests pass successfully

Observed results

one of the feature test fails

@johnhurt johnhurt self-assigned this Dec 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants