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

Direct-tcpip #1487

Closed
russjones opened this issue Nov 30, 2017 · 7 comments
Closed

Direct-tcpip #1487

russjones opened this issue Nov 30, 2017 · 7 comments

Comments

@russjones
Copy link
Contributor

Problem

When using the recording proxy can establish a direct-tcpip connection to another node and then directly communicate with it bypassing session logging. Teleport should support an option to disable direct-tcpip.

@russjones russjones added this to the 2.4.0 milestone Nov 30, 2017
@klizhentas
Copy link
Contributor

klizhentas commented Nov 30, 2017

Per OpenSSH protocol spec: https://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.bin/ssh/PROTOCOL.certkeys?annotate=HEAD
will be called:

permit-port-forwarding and should be part of existing SSH options in our RBAC ssh options key map.

@kontsevoy
Copy link
Contributor

It's a bit unclear to me, what are we doing with this issue? Sounds like a legit use of SSH and folks should realize that session recording cannot possibly record other traffic. Update the docs?

@klizhentas
Copy link
Contributor

no, add the flag to support turning permit-port-forwarding off in the RBAC like we did for pty

@knisbet
Copy link
Contributor

knisbet commented Nov 30, 2017

Silly question, but even without being able to record the session, do we audit log the other traffic, like the user setup the following tunnel, or used SCP to copy a file of x bytes to the destination?

@klizhentas
Copy link
Contributor

some of it in audit events, yes

@kontsevoy
Copy link
Contributor

kontsevoy commented Nov 30, 2017

makes sense, I didn't realize we haven't done it yet.
here's the updated role definition with a new flag:

kind: role
version: v3
metadata:
  name: admin
spec:
  # SSH options used for user sessions 
  options:
    # port_forwarding controls either users are allowed to forward ports
    port_forwarding: true

@klizhentas
Copy link
Contributor

looks good to me

hatched pushed a commit that referenced this issue Jan 30, 2023
* Refactor DownloadScript Screens (#1367)
* Tentatively implement Create Database screen (#1372)
* Refactor TestConnection Screens (#1375)
* Database Tweaks and Add ons (#1412)
* Implement mutual TLS screen (#1418)
* Add all db options to db selector (#1441)
* Tweaks based on design review and regression fixes (#1433)
* Implement the IAM policy screen (#1459)
* Add database service checker and various db tweaks (#1481)
* Temp remove db service checker until bug is fixed (#1495)

Co-authored-by: Ryan Clark <ryan.clark@goteleport.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants