Skip to content

SmartCard ascii variants#40287

Merged
ibeckermayer merged 5 commits into
masterfrom
isaiah/scard-ascii-variants
Apr 15, 2024
Merged

SmartCard ascii variants#40287
ibeckermayer merged 5 commits into
masterfrom
isaiah/scard-ascii-variants

Conversation

@ibeckermayer
Copy link
Copy Markdown
Contributor

@ibeckermayer ibeckermayer commented Apr 5, 2024

Depends on

Adds handling for ascii variants of smartcard calls as well as handling for non-infinite timeouts.

A user in the community slack tested this out for me.

changelog: Adds support for the ascii variants of smartcard calls

@ibeckermayer ibeckermayer requested a review from zmb3 April 10, 2024 16:07
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from AntonAM April 12, 2024 02:25
@ibeckermayer ibeckermayer added this pull request to the merge queue Apr 15, 2024
Merged via the queue into master with commit bff733d Apr 15, 2024
@ibeckermayer ibeckermayer deleted the isaiah/scard-ascii-variants branch April 15, 2024 20:10
@public-teleport-github-review-bot
Copy link
Copy Markdown

@ibeckermayer See the table below for backport results.

Branch Result
branch/v15 Failed

ibeckermayer pushed a commit that referenced this pull request Apr 15, 2024
* Current hash corresponds to IronRDP branch  (should be changed before merging).

* Updates ScardBackend message handling code to account for ascii variants

* Return a timeout when we receive a non-infinite timeout status change call

Includes minor refactor to clean up the code a bit.

* updates to latest IronRDP master hash
github-merge-queue Bot pushed a commit that referenced this pull request Apr 16, 2024
* Current hash corresponds to IronRDP branch  (should be changed before merging).

* Updates ScardBackend message handling code to account for ascii variants

* Return a timeout when we receive a non-infinite timeout status change call

Includes minor refactor to clean up the code a bit.

* updates to latest IronRDP master hash
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants