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.

Sign up for GitHub

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

Jump to bottom

Some WTS_* constants are missing #1403

Closed
vitkuz573 opened this issue Dec 9, 2022 · 3 comments
Closed

Some WTS_* constants are missing #1403

vitkuz573 opened this issue Dec 9, 2022 · 3 comments
Assignees
@mikebattista
Labels
missing api Some documented API is missing from the metadata

Comments

@vitkuz573
Copy link

vitkuz573 commented Dec 9, 2022 •
edited
Loading

Missing constants:

WTS_CURRENT_SERVER_HANDLE
WTS_CURRENT_SERVER_NAME
WTS_CURRENT_SERVER
WTS_SECURITY_CURRENT_GUEST_ACCESS
WTS_SECURITY_USER_ACCESS
WTS_SECURITY_CURRENT_USER_ACCESS
WTS_SECURITY_ALL_ACCESS
@vitkuz573 vitkuz573 changed the title WTS_CURRENT_* constants are missing WTS_CURRENT_* constants are missing Dec 9, 2022
@vitkuz573 vitkuz573 changed the title WTS_CURRENT_* constants are missing WTS_CURRENT_SERVER_* constants are missing Dec 9, 2022
@vitkuz573 vitkuz573 changed the title WTS_CURRENT_SERVER_* constants are missing Some WTS_* constants are missing Dec 12, 2022
@mikebattista mikebattista self-assigned this Dec 12, 2022
@mikebattista mikebattista added the missing api Some documented API is missing from the metadata label Dec 12, 2022
mikebattista added a commit that referenced this issue Dec 14, 2022
@mikebattista
Added WTS_SECURITY_FLAGS for #1403.
ff822dc
@chenss3 chenss3 closed this as completed Jan 19, 2023
@vitkuz573
Copy link
Author

But the constants WTS_CURRENT_ were not added. I see only WTS_SECURITY_FLAGS.

@riverar riverar reopened this Jan 20, 2023
mikebattista added a commit that referenced this issue Jan 24, 2023
@mikebattista
Added WTS_CURRENT_SERVER constants for #1403.
c7f4a60
@mikebattista
Copy link
Collaborator

Can you check how WTS_CURRENT_SERVER_NAME was defined in the commit above? I believe the HANDLEs are correct but not sure about WTS_CURRENT_SERVER_NAME given it's used as both LPSTR and LPWSTR for the -A and -W variants, and it's defined as just NULL in the headers.

@riverar riverar mentioned this issue Jan 29, 2023
Closed
@mikebattista
Copy link
Collaborator

Closing out. Please reactivate or file a new issue if there are any usability issues with how the constants are defined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mikebattista mikebattista

Labels
missing api Some documented API is missing from the metadata
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@riverar @mikebattista @vitkuz573 @chenss3