search

vrchatapi / vrchatapi-javascript

🟡 VRChat API Library for JavaScript and TypeScript
https://www.npmjs.com/package/vrchat
MIT License
42 stars 17 forks source link

Fix URL Path Issues and Add Helper Class for TOTP and Header Generation #9

Closed ar-Kitty closed 1 month ago

ar-Kitty commented 1 month ago

I fixed some URL path issues in the verify2FA method. The URL path has been changed from OTP to TOTP see here. Additionally, I added a helper class to generate TOTP codes and create headers. Updated Axios to 1.7.2 from the old 0.26.1.

ariesclark commented 1 month ago

This would've never been merged. This repository contains generated code, if you want to make changes, it should be in the specification repository, or as a patch file.

ar-Kitty commented 1 month ago

This would've never been merged. This repository contains generated code, if you want to make changes, it should be in the specification repository, or as a patch file.

Yes, I saw that too, which is a reason why I closed it. There was an issue with the Axios headers because the package is really outdated. Additionally, the Verify2FA method has some nested issues I want to fix (in this merge, I changed the wrong route xD). I'll probably make my own version with some improvements.

C0D3-M4513R commented 1 month ago

10 updates the generator, which updates axios.