Skip to content
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

TeslaMate encryption of API tokens #141

Merged
merged 7 commits into from
May 8, 2022
Merged

Conversation

tobiasehlert
Copy link
Owner

Adds support for AES_256_GCM encrypted API tokens implemented by PR teslamate-org/teslamate#2360 in TeslaMate.

@tobiasehlert tobiasehlert added enhancement New feature or request go Pull requests that update Go code help wanted Extra attention is needed WIP Work in progress labels Jan 26, 2022
@tobiasehlert tobiasehlert force-pushed the feature-encrypt-api-tokens branch from 8804cdc to a4aa6fe Compare April 28, 2022 06:20
Repository owner deleted a comment from sonarqubecloud bot Apr 28, 2022
@noocyte
Copy link

noocyte commented May 6, 2022

@tobiasehlert What is left before this can be merged?

Co-authored-by: Leland Sindt <[email protected]>
@tobiasehlert
Copy link
Owner Author

@tobiasehlert What is left before this can be merged?

@noocyte, the problem is that the decryption doesn't work and I was kind of stuck.
@LelandSindt seems to have solved the decryption.. trying his code in 51306bf to see if it works :)

@tobiasehlert tobiasehlert linked an issue May 8, 2022 that may be closed by this pull request
@sonarqubecloud
Copy link

sonarqubecloud bot commented May 8, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@tobiasehlert tobiasehlert removed help wanted Extra attention is needed WIP Work in progress labels May 8, 2022
@tobiasehlert tobiasehlert merged commit 3f9e8c4 into main May 8, 2022
@tobiasehlert tobiasehlert deleted the feature-encrypt-api-tokens branch May 8, 2022 10:29
tobiasehlert added a commit that referenced this pull request May 9, 2022
richard1122 pushed a commit to richard1122/teslamateapi that referenced this pull request Jun 26, 2022
richard1122 added a commit to richard1122/teslamateapi that referenced this pull request Jun 26, 2022
richard1122 added a commit to richard1122/teslamateapi that referenced this pull request Jun 26, 2022
richard1122 added a commit to richard1122/teslamateapi that referenced this pull request Jun 26, 2022
@pkuehnel
Copy link

pkuehnel commented Jul 15, 2022

@tobiasehlert Could you please mention in your new Release notes, that you have to add the encryption key to your service section as well?

@tobiasehlert
Copy link
Owner Author

@pkuehnel I did mention that in the beginning and updated the readme with the ENCRYPTION_KEY parameter.
But I can make it more clear :)

@tobiasehlert
Copy link
Owner Author

@pkuehnel, have I done it more clear in the release note of v1.15.0?

@pkuehnel
Copy link

@tobiasehlert Much better. Thanks :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not able to use POST commands
3 participants