-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add tool for getting data and anonymize it #94
Comments
Note to self:
The VIN, location, alias, license plate, etc... should be randomised to prevent a security or privacy leak. The printout to the terminal can be reduced to a minimum, only when an error/exception occurs and the name of the file (or files when multiple files are present in the same account) to which the data is written. This small program can at the same time also be a good example program on how the mytoyota library can be used to retrieve the information. We can thus also include it in an 'examples' directory. Another advantage of this small program is that if someone else wants to use the information from the MyT servers, he can call this program from the commandline, and then load the .json file in his own program. For this benefit some additional commandline arguments could be helpful, like:
And the program returns 0 on success, and 1 if an error/exception has occured. |
I think it would be cool to have a central log management, as suggested in this PR: #296
|
@DurgNomis-drol In the original description of this PR, you are referring to that more testing is needed to get all the different information values from the MyT endpoints, for all different car models.
I do observe that this will be a common problem, not only for HVAC but also for other sensors and engine types.
This may be off topic for this PR, but shouldn't we start thinking about a separate function (or tool) that just requests the data and then anonimizes the data (stripping vin, user and location data), and makes it possible for the user to share it with us?
Originally posted by @joro75 in #93 (comment)
The text was updated successfully, but these errors were encountered: