Bump time from 0.3.44 to 0.3.47 - #103
Conversation
Bumps [time](https://github.com/time-rs/time) from 0.3.44 to 0.3.47. - [Release notes](https://github.com/time-rs/time/releases) - [Changelog](https://github.com/time-rs/time/blob/main/CHANGELOG.md) - [Commits](time-rs/time@v0.3.44...v0.3.47) --- updated-dependencies: - dependency-name: time dependency-version: 0.3.47 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
c2e5551 to
b5d9812
Compare
|
Claude finished @leighmcculloch's task in 1m 38s —— View job PR Review: Bump
|
| Crate | Old | New |
|---|---|---|
time |
0.3.44 | 0.3.47 |
time-core |
0.1.6 | 0.1.8 |
time-macros |
0.2.24 | 0.2.27 |
num-conv |
0.1.0 | 0.2.0 |
time now depends on serde_core instead of the full serde crate — a small dependency-tree improvement, no API impact for this project.
Security & correctness assessment
- RUSTSEC-style fix in 0.3.47: stack-exhaustion DoS when parsing RFC 2822 input. Not exploitable here — this crate does no datetime parsing of any kind;
serde_withusestimefor serialization formats, andstellar-strkeydoes not use anytime-based serializer. The fix is a freebie via the transitive bump. - 0.3.47 integer-overflow fix (depth tracking switched
u8→u32): also not reachable from this codebase. - Compile-time error for ill-formed well-known formats: a behavioral change in
timeitself, but again no code here uses those APIs.
Compatibility
- MSRV stays at 1.81.0 (
Cargo.toml:11); thetime0.3.x line has historically held a much lower MSRV, so this should not regress the matrix. - CI runs both
msrvandlatest(.github/workflows/rust.yml:23-26), so any MSRV regression would be caught automatically — no need to verify locally. - The
num-convmajor bump (0.1 → 0.2) is internal totime-macros; nothing in this repo importsnum-conv.
Build verification
I did not run cargo build / cargo test locally — the toolchain is not in the sandbox PATH for this review job. The repo's CI matrix in .github/workflows/rust.yml covers msrv and latest builds plus tests, so the green CI signal on this PR is the authoritative check. If CI is green, this is safe to merge.
Recommendation
LGTM. Low-risk lockfile-only update; transitive only; the headline security fix is a defense-in-depth bonus rather than a load-bearing one for this crate. No inline comments — the diff is purely checksum/version bumps with nothing to suggest changing.
• Branch
|
@dependabot rebase |
|
Looks like this PR has been edited by someone other than Dependabot. That means Dependabot can't rebase it - sorry! If you're happy for Dependabot to recreate it from scratch, overwriting any edits, you can request |
|
A newer version of time exists, but since this PR has been edited by someone other than Dependabot I haven't updated it. You'll get a PR for the updated version as normal once this PR is merged. |
Bumps time from 0.3.44 to 0.3.47.
Release notes
Sourced from time's releases.
Changelog
Sourced from time's changelog.
... (truncated)
Commits
d5144cdv0.3.47 releasef6206b0Guard against integer overflow in release mode1c63dc7Avoid denial of service when parsing Rfc28225940df6Add builder methods to avoid verbose construction00881a4Manually format macros everywherebb723b6Addtrailing_inputmodifier toend31c4f8ePermitW12indate!macro490a17bMark error paths in well-known formats as cold6cb1896OptimizeRfc2822parsing6d264d5Remove erroneous#[inline(never)]attributesYou can trigger a rebase of this PR by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)You can disable automated security fix PRs for this repo from the Security Alerts page.