diff options
| author | Matt Montgomery <33811686+ConfusedPolarBear@users.noreply.github.com> | 2020-08-17 16:36:45 -0500 |
|---|---|---|
| committer | Matt Montgomery <33811686+ConfusedPolarBear@users.noreply.github.com> | 2020-08-17 17:13:11 -0500 |
| commit | 5f1a86324170387f12602d77dad7249faf30548f (patch) | |
| tree | 211b6514063773e779254259977807d66f97669c /MediaBrowser.Controller/QuickConnect | |
| parent | c49a357f85edbabab11b61b9d4a2938bdb8f3df9 (diff) | |
Apply suggestions from code review
Diffstat (limited to 'MediaBrowser.Controller/QuickConnect')
| -rw-r--r-- | MediaBrowser.Controller/QuickConnect/IQuickConnect.cs | 12 |
1 files changed, 5 insertions, 7 deletions
diff --git a/MediaBrowser.Controller/QuickConnect/IQuickConnect.cs b/MediaBrowser.Controller/QuickConnect/IQuickConnect.cs index fd7e973f6..959a2d771 100644 --- a/MediaBrowser.Controller/QuickConnect/IQuickConnect.cs +++ b/MediaBrowser.Controller/QuickConnect/IQuickConnect.cs @@ -1,6 +1,5 @@ using System; using MediaBrowser.Model.QuickConnect; -using Microsoft.AspNetCore.Http; namespace MediaBrowser.Controller.QuickConnect { @@ -15,9 +14,9 @@ namespace MediaBrowser.Controller.QuickConnect int CodeLength { get; set; } /// <summary> - /// Gets or sets the string to prefix internal access tokens with. + /// Gets or sets the name of internal access tokens. /// </summary> - string TokenNamePrefix { get; set; } + string TokenName { get; set; } /// <summary> /// Gets the current state of quick connect. @@ -48,9 +47,8 @@ namespace MediaBrowser.Controller.QuickConnect /// <summary> /// Initiates a new quick connect request. /// </summary> - /// <param name="friendlyName">Friendly device name to display in the request UI.</param> /// <returns>A quick connect result with tokens to proceed or throws an exception if not active.</returns> - QuickConnectResult TryConnect(string friendlyName); + QuickConnectResult TryConnect(); /// <summary> /// Checks the status of an individual request. @@ -62,10 +60,10 @@ namespace MediaBrowser.Controller.QuickConnect /// <summary> /// Authorizes a quick connect request to connect as the calling user. /// </summary> - /// <param name="request">HTTP request object.</param> + /// <param name="userId">User id.</param> /// <param name="code">Identifying code for the request.</param> /// <returns>A boolean indicating if the authorization completed successfully.</returns> - bool AuthorizeRequest(HttpRequest request, string code); + bool AuthorizeRequest(Guid userId, string code); /// <summary> /// Expire quick connect requests that are over the time limit. If <paramref name="expireAll"/> is true, all requests are unconditionally expired. |