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

guacamole_connection_ssh -> terminal_type -> xterm-25color is bad value #15

Open
ArnaultMICHEL opened this issue Jan 5, 2024 · 3 comments

Comments

@ArnaultMICHEL
Copy link

Hi

I found a potential little mistake on a value checked for guacamole_connection_ssh -> terminal_type : xterm-25color

  • it should be xterm-256color

  • CLI htop is not working properly as it use $TERM env var

Terraform Version

Terraform v1.4.5
on linux_amd64

Guacamole provider version

techbeck03/guacamole v1.4.1

Affected Resource

  • guacamole_connection_ssh

Terraform Configuration Files

resource "guacamole_connection_ssh" "conn_ssh" {
  ...
  parameters {
    ..
    terminal_type = "xterm-25color"
  }
}

Expected Behavior

allow xterm-256color value instead of xterm-25color for terminal_type parameter

Actual Behavior

terraform apply is OK, but the $TERM is not correct for some linux CLI tools

arnault.michel@test-host:~$ htop                      
Error opening terminal: xterm-25color.
arnault.michel@test-host:~$ echo $TERM                
xterm-25color

Steps to Reproduce

Please list the steps required to reproduce the issue, for example:

  1. terraform apply
  2. open the SSH connection to the linux VM
  3. launch htop

References

please change the documentation

@Thomas-McKanna
Copy link

This is definitely a bug. Even in the Gucacamole UI, you can see it should be xterm-256color. Unfortunately, this bug means that xterm-256color cannot be set with this Terraform provider, since the incorrect value seems to be simply ignored by Guacamole (so terminal type ends up being set to nothing).

image

@Thomas-McKanna
Copy link

Opened #16 to fix this. It is really straightforward.

@techBeck03
Copy link
Owner

techBeck03 commented Nov 30, 2024 via email

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

3 participants