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

[ BUG ] - unable to add RDP_AUDIO_INPUT in RDP subprotocol for a service declaration #48

Closed
raphaelbkt opened this issue Dec 16, 2024 · 6 comments
Assignees
Labels
bug Something isn't working

Comments

@raphaelbkt
Copy link

raphaelbkt commented Dec 16, 2024

This is a Terraform provider problem. We are using version 0.13.0 however I did not find any correction for later release.

If I declare a service containing RDP_AUDIO_OUTPUT everything works as expected:

resource "wallix-bastion_device_service" "wodd2-dc-ws-co2-id-2124-h_xxxrdp_audio" {
    device_id         = wallix-bastion_device.wodd2-dc-ws-co2-id-2124-h.device_name
    service_name      = "xxxrdp_audio"
    ## service_name      = wallix-bastion_connection_policy.xxxrdp.connection_policy_name
    connection_policy = wallix-bastion_connection_policy.xxxrdp.connection_policy_name
    port              = "3389"
    protocol          = "RDP"
    subprotocols      = [ "RDP_SMARTCARD", "RDP_AUDIO_OUTPUT" ]
    global_domains    = [ wallix-bastion_domain.xxx.domain_name ]
}

But If I add the sub-protocol RDP_AUDIO_INPUT I have an error at the terraform apply:

resource "wallix-bastion_device_service" "wodd2-dc-ws-co2-id-2124-h_xxxrdp_audio" {
    device_id         = wallix-bastion_device.wodd2-dc-ws-co2-id-2124-h.device_name
    service_name      = "xxxrdp_audio"
    ## service_name      = wallix-bastion_connection_policy.xxxrdp.connection_policy_name
    connection_policy = wallix-bastion_connection_policy.xxxrdp.connection_policy_name
    port              = "3389"
    protocol          = "RDP"
    subprotocols      = [ "RDP_SMARTCARD", "RDP_AUDIO_INPUT", "RDP_AUDIO_OUTPUT" ]
    global_domains    = [ wallix-bastion_domain.xxx.domain_name ]
}

The error is:

│ 1038: resource "wallix-bastion_device_service" "wodd2-dc-ws-co0-id-2124-h_xxxrdp_audio" {
│ Error: subprotocols RDP_AUDIO_INPUT not valid for RDP service
│   with wallix-bastion_device_service.wodd2-dc-ws-co1-id-2124-h_xxxrdp_audio,
│   on services.tf line 1049, in resource "wallix-bastion_device_service" "wodd2-dc-ws-co1-id-2124-h_xxxrdp_audio":

And the service is not configured on the bastion host Wallix.

Expected behavior
The service should be provisionned on the Wallix bastion host with the subprotocols RDP_AUDIO_INPUT and RDP_AUDIO_OUTPUT activated.

Desktop (please complete the following information):
We are using Terraform provider v0.13.0 and Wallix Bastion 10.0.9

Thanks

@raphaelbkt raphaelbkt added the bug Something isn't working label Dec 16, 2024
@raphaelbkt
Copy link
Author

If I make the modification in the GUI it works.
Problem is at the next pipeline reapply it removes ths subprotocol RDP_AUDIO_INPUT.

@raphaelbkt
Copy link
Author

I can see that it is not listed in the file: https://github.com/wallix/terraform-provider-wallix-bastion/blob/main/bastion/resource_device_service.go

func rdpSubProtocolsValid() []string {
	return []string{
		"RDP_CLIPBOARD_UP",
		"RDP_CLIPBOARD_DOWN",
		"RDP_CLIPBOARD_FILE",
		"RDP_PRINTER",
		"RDP_COM_PORT",
		"RDP_DRIVE",
		"RDP_SMARTCARD",
		"RDP_AUDIO_OUTPUT",
	}
}

@bsimonWallix bsimonWallix self-assigned this Dec 16, 2024
@bsimonWallix
Copy link
Collaborator

Thanks @raphaelbkt, I just updated the list for ssh and RDP.
If you are able to fix issues, do not hesitate to contribute to the code by creating a PR.

@raphaelbkt
Copy link
Author

Ok I'll do it next time !
Thanks

@bsimonWallix
Copy link
Collaborator

We are evaluating if we can do some other change to publish it by thursday.

@bsimonWallix
Copy link
Collaborator

Closing this issue, we released the v0.14.2 that fix this.

Best regards,

raphaelbkt added a commit to raphaelbkt/terraform-provider-wallix-bastion that referenced this issue Dec 23, 2024
correct wallix#48

There was RDP_AUDIO_OUTPUT and the list was missong RDP_AUDIO_INPUT.

Thanks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants