Fix Rust 1.75 clippy finding, expired real world test certificate#46
Fix Rust 1.75 clippy finding, expired real world test certificate#46complexspaces merged 2 commits intorustls:mainfrom
Conversation
Running `cargo clippy-ci` using Rust 1.75 fails: ``` error: unused import: `tests::ffi::*` --> src/lib.rs:28:9 | 28 | pub use tests::ffi::*; | ^^^^^^^^^^^^^ | = note: `-D unused-imports` implied by `-D warnings` = help: to override `-D warnings` add `#[allow(unused_imports)]` ``` This commit adds a `allow(unused_imports)` gated on the same `ffi-testing` feature that the `pub use` statement is conditional upon.
Since this is going to keep happening every ~90d (who thought automated certificate issuance was a good idea? Sounds crazy to me! 🤪) we should probably fix this more robustly. Some ideas:
Edit: and of course, this also applies to the 1Password test certs - they just expire at a slower rate. |
|
Would running the tests under edit: and, by implication, only on linux. |
For another repo that would probably be an OK trade-off but I think for this one the platform coverage is important :-( |
|
I don't think that we need any platform-specific time-mocking. There's just a current defficency in both the mock and real world tests where we pass in If we update these to use a hardcoded date, such as today for example, the certificates should last much longer and only be prone to root expiry. I apologize for not calling that out earlier, I only remembered it when thinking about a better solution then something Linux-specific. |
Ah! That does sound best. Can we merge this to fix CI and I will follow up on that approach? |
|
That sounds good to me. I agree its better to get CI unbroken first. |
pub use tests::ffi::*;insrc/lib.rsthat's behind theffi-testingfeature flag as unused. This branch allows that unused import.