-
Notifications
You must be signed in to change notification settings - Fork 71
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
add mime-type checking; better panic handling #2304
Merged
jqnatividad
merged 8 commits into
master
from
2301-check-mime-type_better-panic-handling
Nov 21, 2024
Merged
add mime-type checking; better panic handling #2304
jqnatividad
merged 8 commits into
master
from
2301-check-mime-type_better-panic-handling
Nov 21, 2024
Commits on Nov 21, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 27e527a - Browse repository at this point
Copy the full SHA 27e527aView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8ad594b - Browse repository at this point
Copy the full SHA 8ad594bView commit details -
that can be suppressed with QSV_SKIP_FORMAT_CHECK env var
Configuration menu - View commit details
-
Copy full SHA for 4121f07 - Browse repository at this point
Copy the full SHA 4121f07View commit details -
Configuration menu - View commit details
-
Copy full SHA for 500c13f - Browse repository at this point
Copy the full SHA 500c13fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 94bda2f - Browse repository at this point
Copy the full SHA 94bda2fView commit details -
Configuration menu - View commit details
-
Copy full SHA for 557aba4 - Browse repository at this point
Copy the full SHA 557aba4View commit details -
docs: added QSV_SKIP_FORMAT_CHECK; corrected/updated some format erro…
…rs and old default buffer values
Configuration menu - View commit details
-
Copy full SHA for 0132fcf - Browse repository at this point
Copy the full SHA 0132fcfView commit details -
chore: apply clippy suggestions
warning: immediately dereferencing a reference --> src/util.rs:1995:54 | 1995 | match simd_json::serde::from_slice(&mut **&mut s_slice) { | ^^^^^^^^^^^^^ help: try: `s_slice` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#deref_addrof = note: `#[warn(clippy::deref_addrof)]` on by default warning: immediately dereferencing a reference --> src/util.rs:2139:54 | 2139 | match simd_json::serde::from_slice(&mut **&mut s_slice) { | ^^^^^^^^^^^^^ help: try: `s_slice` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#deref_addrof warning: deref which would be done by auto-deref --> src/util.rs:1995:48 | 1995 | match simd_json::serde::from_slice(&mut **&mut s_slice) { | ^^^^^^^^^^^^^^^^^^^ help: try: `&mut s_slice` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref = note: `#[warn(clippy::explicit_auto_deref)]` on by default warning: this expression creates a reference which is immediately dereferenced by the compiler --> src/util.rs:2126:63 | 2126 | csv_to_jsonl(&tempfile_path, &get_stats_data_types(), &statsdatajson_path)?; | ^^^^^^^^^^^^^^^^^^^ help: change this to: `statsdatajson_path` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrow = note: `#[warn(clippy::needless_borrow)]` on by default warning: deref which would be done by auto-deref --> src/util.rs:2139:48 | 2139 | match simd_json::serde::from_slice(&mut **&mut s_slice) { | ^^^^^^^^^^^^^^^^^^^ help: try: `&mut s_slice` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#explicit_auto_deref
Configuration menu - View commit details
-
Copy full SHA for 219eb84 - Browse repository at this point
Copy the full SHA 219eb84View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.