Skip to content

Add trusted device to the events.UserMetadata proto#20079

Merged
codingllama merged 2 commits into
masterfrom
codingllama/dt-usermetadata
Jan 11, 2023
Merged

Add trusted device to the events.UserMetadata proto#20079
codingllama merged 2 commits into
masterfrom
codingllama/dt-usermetadata

Conversation

@codingllama
Copy link
Copy Markdown
Contributor

Add the trusted device to UserMetadata, thus enabling audit logs to contain the device in use.

https://github.com/gravitational/teleport.e/issues/514

@codingllama
Copy link
Copy Markdown
Contributor Author

Commit 1026d30cd3afc882947cd9e71ae1e63bcb5743d0 simply runs make grpc{,-teleterm} on master, as there are non-generated changes present. Similar to #20070, but I've also included make grpc-teleterm. (FYI @Tener)

I've elected to split devicetrust.proto into separate files, so imports can be more granular. Makes little difference here, but apparently old habits die hard.

Copy link
Copy Markdown
Contributor

@espadolini espadolini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicated because gogo doesn't play well with protoc-gen-go.

🥲

@github-actions github-actions Bot removed the request for review from Joerger January 11, 2023 18:16
@codingllama codingllama force-pushed the codingllama/dt-usermetadata branch from 887e3aa to 9a01989 Compare January 11, 2023 18:32
@codingllama codingllama enabled auto-merge (squash) January 11, 2023 18:32
@codingllama codingllama force-pushed the codingllama/dt-usermetadata branch 4 times, most recently from e10c930 to e4e0663 Compare January 11, 2023 20:58
@codingllama codingllama force-pushed the codingllama/dt-usermetadata branch from e4e0663 to eddeaff Compare January 11, 2023 21:12
@codingllama codingllama merged commit 62dbe3e into master Jan 11, 2023
@codingllama codingllama deleted the codingllama/dt-usermetadata branch January 11, 2023 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants