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

tedge.toml does not include device-id under [device] #531

Closed
mbay-ODW opened this issue Oct 27, 2021 · 2 comments
Closed

tedge.toml does not include device-id under [device] #531

mbay-ODW opened this issue Oct 27, 2021 · 2 comments
Labels
feature Change request wontwork This will not be worked on

Comments

@mbay-ODW
Copy link

Is your feature request related to a problem? Please describe.
Currently the device id is not stored within the tedge.toml which might be needed by other plugins or modules.
Describe the solution you'd like
Write the device id into the toml
Describe alternatives you've considered
I can get the device id also from tedge config list but there is no reason why only the device id is missing on the toml.

@mbay-ODW mbay-ODW added the feature Change request label Oct 27, 2021
@mbay-ODW mbay-ODW changed the title Teige.toml does not include device-id under [device] tedge.toml does not include device-id under [device] Oct 27, 2021
@rina23q
Copy link
Member

rina23q commented Oct 28, 2021

Hi, actually it's as we designed.
tedge gets device.id from your device certificate's CN. We don't make device.id configurable by tedge config. The purpose is to print it by tedge config list is to help user to check the value in an easy way.

@itsyitsy itsyitsy added invalid wontwork This will not be worked on labels Nov 1, 2021
@itsyitsy
Copy link
Contributor

itsyitsy commented Nov 1, 2021

As @rina23q pointed out rightfully this is expected behaviour and done on purpose. We do not want to have inconsistency regarding device-id and therefore the device certificate is and will be the only place where the device-id gets stored.

@itsyitsy itsyitsy closed this as completed Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Change request wontwork This will not be worked on
Projects
None yet
Development

No branches or pull requests

3 participants