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

[WebAuthn] Enable concurrent Authenticator requests where supported by the native API #6389

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

feedop
Copy link
Collaborator

@feedop feedop commented Sep 30, 2024

Description of Change

In certain situations it should be possible to make concurrent MakeCredential/GetAssertion calls. Therefore the authenticator being busy is no longer checked in the C# API. Instead, concurrent calls are unconditionally forwarded and it is now up to the native API whether it will honor the request or return an error.

Additionally, this pull request fixes an error in AuthenticatorAssertionResponse data marshalling.

API Changes

  • No public API changes

@github-actions github-actions bot added the API12 Platform : Tizen 9.0 / TFM: net6.0-tizen9.0 label Sep 30, 2024
@TizenAPI-Bot
Copy link
Collaborator

Public API Changed

Please follow the ACR process for the changed API below.

Added: 0, Removed: 0, Changed: 3

Changed

- /// <since_tizen>3</since_tizen
+ /// <since_tizen>none</since_tizen
Tizen.System.BatteryLevelStatus Tizen.System.Battery::Level()

+ /// <privilege>http://tizen.org/privilege/display.state</privilege
/// <since_tizen>5</since_tizen
static System.Void Tizen.System.Power::ReleaseLock(Tizen.System.PowerLock)

+ /// <privilege>http://tizen.org/privilege/display.state</privilege
/// <since_tizen>5</since_tizen
static System.Void Tizen.System.Power::RequestLock(Tizen.System.PowerLock,System.Int32)

Internal API Changed

Added: 4, Removed: 0, Changed: 2

Added

+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.Boolean Tizen.Applications.Notifications.Notification::PairingType()

+ /// <since_tizen>12</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ static System.Void Tizen.Applications.Notifications.NotificationManager::RegisterDoNotDisturbApp()

+ /// <since_tizen>12</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ System.EventHandler Tizen.Applications.Notifications.NotificationManager::DisturbReceived

+ /// <since_tizen>none</since_tizen
+ [EditorBrowsable(EditorBrowsableState.Never)]
+ static System.Threading.Tasks.Task Tizen.Multimedia.WavPlayer::StartAsync(System.String,Tizen.Multimedia.AudioStreamPolicy,System.UInt32,System.Threading.CancellationToken)

Changed

/// <privilege>http://tizen.org/privilege/reboot</privilege
- /// <since_tizen>none</since_tizen
+ /// <since_tizen>9</since_tizen
[EditorBrowsable(EditorBrowsableState.Never)]
static System.Void Tizen.System.Power::PowerOff()

/// <privilege>http://tizen.org/privilege/reboot</privilege
- /// <since_tizen>none</since_tizen
+ /// <since_tizen>9</since_tizen
[EditorBrowsable(EditorBrowsableState.Never)]
static System.Void Tizen.System.Power::Reboot(System.String)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API12 Platform : Tizen 9.0 / TFM: net6.0-tizen9.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants