-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
[WIP] [knx] Add support for using hardware TPM modules #15326
Open
holgerfriedrich
wants to merge
11
commits into
openhab:main
Choose a base branch
from
holgerfriedrich:pr-knx-tpm
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
d5b0877
to
1d8fa50
Compare
1d8fa50
to
2d18611
Compare
2d18611
to
3921519
Compare
3921519
to
2d5d5a4
Compare
2d5d5a4
to
930bd12
Compare
eafb63d
to
a66fcb5
Compare
a66fcb5
to
8953a91
Compare
8953a91
to
851ed72
Compare
851ed72
to
c0b5c51
Compare
5285dbb
to
8068b88
Compare
8068b88
to
e0d7f3b
Compare
e0d7f3b
to
e86feef
Compare
f25ee6f
to
6a50352
Compare
6a50352
to
d2c9011
Compare
450711e
to
f761f32
Compare
f761f32
to
9ae5b43
Compare
9ae5b43
to
c1d4e24
Compare
c1d4e24
to
1192718
Compare
1192718
to
3e049b4
Compare
3e049b4
to
61038dd
Compare
52024e9
to
b96f07f
Compare
2b4cecf
to
3ab1d62
Compare
0c2ae6e
to
2668c87
Compare
69d187d
to
59243d3
Compare
Add TpmInterface, a class built on top of Tss.Java. This lib is to be included in a special way due to inconsistencies in package creation which makes it incompatible to OSGI. Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
Signed-off-by: Holger Friedrich <[email protected]>
55fd821
to
90955bc
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
enhancement
An enhancement or new feature for an existing add-on
work in progress
A PR that is not yet ready to be merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This as an attempt to store passwords for KNX secure protected by a TPM module.
The password is stored encrypted and can only be decrypted with this specific TPM.
The proof of concept was done on a Rapsberry PI with a LetsTrust TPM module on top.
knx tpm-info
andknx tpm-encrypt <password>
. Output to be used instead of the password. Use space in front on the knx to avoid you password to be stored to console history.Disclaimer: Use at you own risk.
Disclaimer: Storing encrypted passwords does not bring perfect security for the password - anyone who can access your machine can use the TPM as well to decode, RPI is not a secure system, Java is not secure at all. But it is nice that you do not need to worry about disclosing you password in backups or screenshots :-)