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

Console.Beep() does not works under RDP #31019

Closed
Roman-Blinkov opened this issue Sep 29, 2019 · 12 comments
Closed

Console.Beep() does not works under RDP #31019

Roman-Blinkov opened this issue Sep 29, 2019 · 12 comments

Comments

@Roman-Blinkov
Copy link

Console.Beep() does not works under RDP

Example:
Console.Beep();

But it works if called with arguments.
Example:
Console.Beep(500, 300);

How to reproduce

Both client and server: Windows 10 x64 1903 Build 18990
Microsoft Remote Desktop from Store: https://www.microsoft.com/store/productId/9WZDNCRFJ3PS

@GrabYourPitchforks GrabYourPitchforks transferred this issue from dotnet/core Sep 29, 2019
@stephentoub
Copy link
Member

If there's an issue here (which seems strange, but I've not tried it), it would seem to be with Windows, rather than with .NET. Both overloads just delegate to the same Win32 function via the same DllImport:
https://github.com/dotnet/corefx/blob/7e9a177824cbefaee8985a9b517ebb0ea2e17a81/src/System.Console/src/System/ConsolePal.Windows.cs#L648

@danmoseley
Copy link
Member

What does Beep(800, 200) produce?

@Roman-Blinkov
Copy link
Author

I am not joking.
I recorded HD video for you. https://1drv.ms/u/s!AvU-sf3z4m8qk-oLkYwB6ou2eO5Arw?e=hqfKwV
You can find code from video here
https://github.com/Roman-Blinkov/Beep

@Roman-Blinkov
Copy link
Author

Roman-Blinkov commented Sep 30, 2019

What does Beep(800, 200) produce?

@danmosemsft nothing! There no sound!

Update: I updated GitHub repository with Console.Beep(800, 200);

@Roman-Blinkov
Copy link
Author

This works

Console.Beep(800, 270);

This not works

Console.Beep(800, 269);

@stephentoub
Copy link
Member

Have you tried just writing a basic C/C++ app that just calls Beep from kernel32?

@Roman-Blinkov
Copy link
Author

I wrote after you asked.

Not works

Beep(800, 200);
std::cout << '\a';
std::cout << '\7';

Works

Beep(800, 270);

@Roman-Blinkov
Copy link
Author

How to transfer issue to correct bug tracker?

@danmoseley
Copy link
Member

@Roman-Blinkov I believe you use the Feedback Hub tool (at least in Windows 10) and choose the category "Developer Platform"

I'm going to close this now but please do post back if you get a resolution.

@danmoseley
Copy link
Member

Although @stephentoub perhaps we should change to (800, 270) ..

@danmoseley
Copy link
Member

@Roman-Blinkov also if you have a support contract then you can report it through there.

@Roman-Blinkov
Copy link
Author

perhaps we should change to (800, 270)

You asked @stephentoub but here my opinion:
I think no, because such change can break something much more important than sound in RDP.

Yesterday I submitted issue to Feedback Hub as you recommended.
Thank you!
Have a good day!

@msftgits msftgits transferred this issue from dotnet/corefx Feb 1, 2020
@msftgits msftgits added this to the 5.0 milestone Feb 1, 2020
@ghost ghost locked as resolved and limited conversation to collaborators Dec 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants