Corrected documentation and removed a small debug line

Author: gauffininteractive

src/Server/OneTrueError.Api.Client.Tests/TryTheClient.cs

@@ -20,7 +20,7 @@ public async Task Test()
             {
                 result = await client.QueryAsync(new FindAccountByUserName("admin"));
             }
-            catch (WebException ex)
+            catch (WebException)
             {
             }
 

src/Server/OneTrueError.Api/Core/ApiKeys/Events/ApiKeyCreated.cs

@@ -3,29 +3,66 @@
 
 namespace OneTrueError.Api.Core.ApiKeys.Events
 {
+    /// <summary>
+    /// A new API key has been created.
+    /// </summary>
     public class ApiKeyCreated : ApplicationEvent
     {
-        private readonly int[] _applicationIds;
-
+        /// <summary>
+        ///     Creates a new instance of <see cref="ApiKeyCreated" />.
+        /// </summary>
+        /// <param name="applicationNameForTheAppUsingTheKey">Name of the application that integrates with OTE</param>
+        /// <param name="apiKey">Actual key</param>
+        /// <param name="sharedSecret">Used to authenticate the key</param>
+        /// <param name="applicationIds">Applications that the key is allowed to access</param>
+        /// <param name="createdById">User that created this key</param>
         public ApiKeyCreated(string applicationNameForTheAppUsingTheKey, string apiKey, string sharedSecret,
             int[] applicationIds,
             int createdById)
         {
             if (sharedSecret == null) throw new ArgumentNullException("sharedSecret");
-            _applicationIds = applicationIds;
+            ApplicationIds = applicationIds;
             ApplicationNameForTheAppUsingTheKey = applicationNameForTheAppUsingTheKey;
             ApiKey = apiKey;
             SharedSecret = sharedSecret;
             CreatedById = createdById;
         }
 
+        /// <summary>
+        ///     Serialization constructor
+        /// </summary>
         protected ApiKeyCreated()
         {
         }
 
-        public string ApiKey { get; set; }
-        public string ApplicationNameForTheAppUsingTheKey { get; set; }
-        public int CreatedById { get; set; }
-        public string SharedSecret { get; set; }
+        /// <summary>
+        ///     Actual api key
+        /// </summary>
+        public string ApiKey { get; private set; }
+
+        /// <summary>
+        ///     Applications that the key is allowed to access
+        /// </summary>
+        public int[] ApplicationIds { get; private set; }
+
+        /// <summary>
+        ///     Name of the application that integrates with OTE.
+        /// </summary>
+        /// <remarks>
+        ///     <para>
+        ///         To allow the user to know which key is used for which integration.
+        ///     </para>
+        /// </remarks>
+        public string ApplicationNameForTheAppUsingTheKey { get; private set; }
+
+        /// <summary>
+        ///     Account id of the user that created the key.
+        /// </summary>
+        public int CreatedById { get; private set; }
+
+        /// <summary>
+        ///     Shared secret used to authenticate requests
+        /// </summary>
+        public string SharedSecret { get; private set; }
     }
 }

src/Server/OneTrueError.App/Core/ApiKeys/Events/ApplicationDeletedHandler.cs

@@ -26,6 +26,7 @@ public ApplicationDeletedHandler(IApiKeyRepository repository)
             _repository = repository;
         }
 
+        /// <inheritdoc />
         public async Task HandleAsync(ApplicationDeleted e)
         {
             var apps = await _repository.GetForApplicationAsync(e.ApplicationId);

src/Server/OneTrueError.App/Core/Applications/CommandHandlers/DeleteApplicationHandler.cs

@@ -26,6 +26,7 @@ public DeleteApplicationHandler(IApplicationRepository repository, IEventBus eve
             _eventBus = eventBus;
         }
 
+        /// <inheritdoc/>
         public async Task ExecuteAsync(DeleteApplication command)
         {
             var app = await _repository.GetByIdAsync(command.Id);

src/Server/OneTrueError.App/Core/Notifications/EventHandlers/ApplicationDeletedHandler.cs

@@ -24,6 +24,7 @@ public ApplicationDeletedHandler(IAdoNetUnitOfWork uow)
             _uow = uow;
         }
 
+        /// <inheritdoc />
         public Task HandleAsync(ApplicationDeleted e)
         {
             _uow.ExecuteNonQuery("DELETE FROM UserNotificationSettings WHERE ApplicationId = @id", new { id = e.ApplicationId });

src/Server/OneTrueError.App/Core/Notifications/EventHandlers/CheckForFeedbackNotificationsToSend.cs

@@ -37,6 +37,7 @@ public CheckForFeedbackNotificationsToSend(INotificationsRepository notification
             _queryBus = queryBus;
         }
 
+        /// <inheritdoc/>
         public async Task HandleAsync(FeedbackAttachedToIncident e)
         {
             var settings = await _notificationsRepository.GetAllAsync(-1);

src/Server/OneTrueError.Web/Areas/Receiver/Controllers/ReportController.cs

@@ -43,8 +43,6 @@ public HttpResponseMessage Index()
         [HttpPost, Route("receiver/report/{appKey}")]
         public async Task<HttpResponseMessage> Post(string appKey, string sig)
         {
-            return Request.CreateErrorResponse(HttpStatusCode.NotFound, "No fuck!");
-
             if (HttpContext.Current.Request.InputStream.Length > 20000000)
             {
                 return await KillLargeReportAsync(appKey);