Skip to content

feat: support multi-challenge MPP headers and charge intent#373

Merged
grandizzy merged 4 commits intotempofrom
fix/mpp-multi-challenge
Apr 1, 2026
Merged

feat: support multi-challenge MPP headers and charge intent#373
grandizzy merged 4 commits intotempofrom
fix/mpp-multi-challenge

Conversation

@grandizzy
Copy link
Copy Markdown
Contributor

Depends on tempoxyz/mpp-rs#185

Changes:

  • Transport uses parse_www_authenticate_all + provider.supports() to select challenges from multi-challenge WWW-Authenticate headers (matching mpp-rs's new approach)
  • SessionProvider now supports intent="charge" via TempoCharge, enabling QuickNode MPP endpoints
  • Points to mpp-rs fix/multi-challenge-www-authenticate branch (update to release after mpp-rs PR merges)

@grandizzy grandizzy self-assigned this Mar 31, 2026
@grandizzy grandizzy force-pushed the fix/mpp-multi-challenge branch 2 times, most recently from 4c2aed3 to e10c403 Compare March 31, 2026 18:17
@grandizzy grandizzy force-pushed the fix/mpp-multi-challenge branch from e10c403 to 981695e Compare March 31, 2026 18:28
Co-authored-by: grandizzy <38490174+grandizzy@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d4566-7c96-7239-83b2-917a9f34909a
@grandizzy grandizzy marked this pull request as ready for review March 31, 2026 19:59
zerosnacks
zerosnacks previously approved these changes Apr 1, 2026
Copy link
Copy Markdown
Contributor

@zerosnacks zerosnacks left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense 👍

Co-Authored-By: grandizzy <38490174+grandizzy@users.noreply.github.com>
Amp-Thread-ID: https://ampcode.com/threads/T-019d483f-94f7-71f7-bbb0-abbb1d9a69b0
@grandizzy grandizzy requested a review from zerosnacks April 1, 2026 09:46
@grandizzy grandizzy merged commit 57bd2ce into tempo Apr 1, 2026
15 checks passed
@grandizzy grandizzy deleted the fix/mpp-multi-challenge branch April 1, 2026 09:49
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.

3 participants