Skip to content

fix(truapi-server): raise the recursion limit for the request_login future - #493

Closed
decrypto21 wants to merge 1 commit into
mainfrom
nidish/fix-nightly-recursion-limit
Closed

fix(truapi-server): raise the recursion limit for the request_login future#493
decrypto21 wants to merge 1 commit into
mainfrom
nidish/fix-nightly-recursion-limit

Conversation

@decrypto21

Copy link
Copy Markdown
Contributor

What

Adds #![recursion_limit = "256"] to truapi-server. Closes #476.

cargo +nightly clippy fails on all branches since 22 Aug. A recent nightly added recursion_depth_exceeding_limit; -D warnings promotes it to an error.

error: overflow evaluating the requirement ... pairing_host.rs:2423
  = help: consider increasing the recursion limit by adding a
          `#![recursion_limit = "256"]` attribute to your crate (`truapi_server`)
  = note: `-D recursion-depth-exceeding-limit` implied by `-D warnings`
  = warning: this was previously accepted by the compiler but is being phased out

Proving request_login's boxed trait future Send recurses through SsoPairingFlow::request_session, exceeding the default 128-step budget.

Verification

@decrypto21
decrypto21 requested a review from a team August 24, 2026 10:37
@decrypto21

Copy link
Copy Markdown
Contributor Author

Closed because of #475.

@decrypto21 decrypto21 closed this Aug 24, 2026
@decrypto21
decrypto21 deleted the nidish/fix-nightly-recursion-limit branch August 24, 2026 10:41
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

Successfully merging this pull request may close these issues.

ci.yml resolves an unpinned nightly with -D warnings, breaking cargo clippy on every branch since 22 Aug

1 participant