Repository navigation
WASM - Check permissions before reading or writing to clipboard - #21573
Conversation
MrJul
added
bug
backport-candidate-12.0.x
|
You can test this PR using the following package version. |
| catch(JSException ex) | ||
| { | ||
| throw new UnauthorizedAccessException(ex.Message); | ||
| } |
There was a problem hiding this comment.
Pass original exception as InnerException
|
You can test this PR using the following package version. |
|
You can test this PR using the following package version. |
|
Updated. Removed explicit permission check, but instead check for |
|
You can test this PR using the following package version. |
|
Addressed @MrJul review. |
|
You can test this PR using the following package version. |
MrJul
left a comment
MrJul
left a comment
There was a problem hiding this comment.
Re-tested, works as expected. LGTM!
|
You can test this PR using the following package version. |
…oniaUI#21573) * check permissions before reading or writing to clipboard * throw UnauthorizedAccessException when permission is not granted for clipboard * handle unauthorized access exceptions for clipboard in textbox * check only for denied errors on clipboard * addressed review * wasm - use marshal methods instead when reading clipboard * log unauthorized read clipboard error * Improve logging * Also log in MaskedTextBox --------- Co-authored-by: Julien Lebosquain
MrJul
added
backported-12.0.x
and removed
backport-candidate-12.0.x
5 participants
What does the pull request do?
Adds checks before reading or writing to clipboard on wasm.
UnauthorizedAccessExceptionwill be thrown if permission is not granted.What is the current behavior?
What is the updated/expected behavior with this PR?
How was the solution implemented (if it's not obvious)?
Checklist
Breaking changes
Obsoletions / Deprecations
Fixed issues
Fixes the cause of #21530 . This PR doesn't help with intercepting clipboard.
Fixes #21568