-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migrate to Open-Source Tokens #594
Comments
@ngardiner, do you want to do the registration and get TWCManager a token? We shouldn't duplicate. |
Hm, it appears you still have to provide fixed URLs in the registration. That may be a barrier we need to think through. |
The user auth process looks pretty straight forward and I can't see why it would need to interact with an additional service. Maybe the URLs can be filled with arbitrary values during registration and it would still work? Could easily try with a https://webhook.site/ URL or so and see if anything arrives there. Found your thread with Tesla btw. and am linking it here as a reference for others. |
Tesla has just added the option of Open-Source Tokens to the fleet API, where applications can get client IDs that do not have an associated client secret. That's exactly what we need here, and relieves customers of running their own integration.
The migration looks pretty straightforward and might not even require code changes, but we should a) register TWCManager, b) validate that assumption, and c) update documentation.
The text was updated successfully, but these errors were encountered: