Skip to content

Add support for reqwest-middleware#41

Merged
C0D3-M4513R merged 1 commit intovrchatapi:mainfrom
C0D3-M4513R:feature/request-middleware
Feb 23, 2026
Merged

Add support for reqwest-middleware#41
C0D3-M4513R merged 1 commit intovrchatapi:mainfrom
C0D3-M4513R:feature/request-middleware

Conversation

@C0D3-M4513R
Copy link
Collaborator

This pr adds support for using reqwest-middleware in the api

@haileyscommit
Copy link

As soon as I switched, the new client now requires reqwest_middleware's client. That being said, adding .into() seems to fix it (if someone doesn't want reqwest_middleware for some reason). I'm not sure if this still works even if reqwest_middleware isn't in the project.

Other than that, this worked out of the box, and reqwest_drive appears to be set up now. Thanks!

@C0D3-M4513R
Copy link
Collaborator Author

As soon as I switched, the new client now requires reqwest_middleware's client

Yeah, that's one thing I can do little about.

That being said, adding .into() seems to fix it (if someone doesn't want reqwest_middleware for some reason).

Because of this I am inclined to just merge it.
I just haven't done so, in case anyone wants to add some other remarks.

And not wanting reqwest_middleware is for most dev's probably secondary to reqwest_middleware itself btw.
It's mostly about having to deal with additional complexity.

I'm not sure if this still works even if reqwest_middleware isn't in the project.

It should.

@C0D3-M4513R C0D3-M4513R merged commit 64acffc into vrchatapi:main Feb 23, 2026
2 checks passed
@C0D3-M4513R C0D3-M4513R deleted the feature/request-middleware branch February 23, 2026 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants