As described in gravitational/webapps.e#177, we want to replace the db
tab with just two sections:
* "Connect with CLI" which will show the command to use in terminal
* "Connect with GUI" which links to our documentation
After gravitational/teleport#11720 gets merged, the "Connect with CLI"
section will be massively simplified: it'll be basically just something like:
psql postgres://localhost:12345
Moreover, @smallinsky suggested that tsh should be responsible for creating
those CLI connection commands. We should also do this in the future as
it'll let us support new protocols as soon as they land in tsh, without
us having to touch Teleterm codebase, for the most part.
* Mention that `yarn build-term` needs to be run first before attempting
to run the app in dev mode (already had a couple of people who had
problems with setting up the app because they didn't run this first).
* Mention the assumption about both repos living in the same folder.
* Move the architecture diagram to the end of the file. Most people
reading the README are not doing it for the diagram, but build
instructions.
* Explain when gRPC files need to be recompiled.
This reverts commit 3956e71664.
Turns out that for development, we need to use arm64 version of native
deps. The build server is going to use x64 anyway, as per the reverted
commit, but when making manual demo builds, we'll have to remember to
use x64 for Teleterm and tsh.
Our build system doesn't support arm64 for Mac releases yet (see issue
gravitational/teleport#4226 for more information). Because of that, for
the preview release we're likely going to have only the x64 version of
Teleterm.
This means that the shipped version of tsh should also be the x64 version.
I tried to change electron-builder's config to use x64 for macOS, but the
config options don't seem to work. I tried `mac.defaultArch` as well as
changing `mac.target` in various ways but `electron-builder install-app-deps`
just doesn't pick up those options. Both were set through
`packages/teleterm/package.json`
This commit removes a bunch of unused protobufs and also updates some of
those that got updated on the teleterm branch in the teleport repo.
The biggest change from all of them is that LoginRequest now has oneof
on Sso and Local. [1] This is because a login request should have either
Sso or Local params, but not both at the same time.
The previous implementation called both `setSso` and `setLogin` on the same
object. This no longer works with the use of `oneof`, because calling `setLogin`
after `setSso` would clear the `Sso` params. [2]
[1] https://github.com/gravitational/teleport/pull/10286#discussion_r821743202
[2] https://developers.google.com/protocol-buffers/docs/proto3#oneof_features
Grzegorz Zdunek