Merge pull request #31 from DevExcelerate/feature/new-team-request

Calin Lupas · web-flow · commit f5b18ec5a3ab · 2025-04-30T08:23:29.000-04:00
Feature/new team request
diff --git a/src/DevExcelerateApi/DevExcelerateApi.csproj b/src/DevExcelerateApi/DevExcelerateApi.csproj
@@ -10,14 +10,14 @@
 	<PackageReference Include="Azure.Extensions.AspNetCore.Configuration.Secrets" Version="1.4.0" />
 	<PackageReference Include="Azure.Identity" Version="1.13.2" />
 	<PackageReference Include="Azure.Security.KeyVault.Keys" Version="4.7.0" />
-	<PackageReference Include="Microsoft.Azure.Cosmos" Version="3.48.0" />
-	<PackageReference Include="Microsoft.IdentityModel.JsonWebTokens" Version="8.8.0" />
+	<PackageReference Include="Microsoft.Azure.Cosmos" Version="3.49.0" />
+	<PackageReference Include="Microsoft.IdentityModel.JsonWebTokens" Version="8.9.0" />
 	<PackageReference Include="Newtonsoft.Json" Version="13.0.3" />
 	<PackageReference Include="Octokit" Version="14.0.0" />
 	<PackageReference Include="Octokit.Webhooks.AspNetCore" Version="2.4.1" />
-	<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.8.0" />
+	<PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.9.0" />
 	<PackageReference Include="YamlDotNet" Version="16.3.0" />
-	<PackageReference Include="Microsoft.Data.SqlClient" Version="6.0.1" />
+	<PackageReference Include="Microsoft.Data.SqlClient" Version="6.0.2" />
 	<PackageReference Include="Dapper" Version="2.1.66" />
   </ItemGroup>
   
diff --git a/src/DevExcelerateApi/Program.cs b/src/DevExcelerateApi/Program.cs
@@ -78,10 +78,7 @@ public static async Task Main(string[] args)
                 .RequireAuthorization();
 
             // Start the service
-            Task runTask = app.RunAsync();
-
-            // Wait for the service to complete.
-            await runTask;
+            await app.RunAsync();
         }
     }
 }
diff --git a/src/DevExcelerateApi/Services/DevExIssuesEventProcessorService.cs b/src/DevExcelerateApi/Services/DevExIssuesEventProcessorService.cs
@@ -8,6 +8,7 @@
 using Octokit.Webhooks.Events;
 using Octokit.Webhooks.Events.IssueComment;
 using Octokit.Webhooks.Events.Issues;
+using Octokit.Webhooks.Models;
 using static DevExcelerateApi.Parsers.GitHubIssueFormParser;
 
 namespace DevExcelerateApi.Services
@@ -53,6 +54,13 @@ public async Task SaveAndProcessIssueEventAsync(WebhookHeaders headers, IssuesEv
 
         public async Task SaveAndProcessIssueCommentEventAsync(WebhookHeaders headers, IssueCommentEvent issueCommentEvent, IssueCommentAction action)
         {
+            // Validate if the issue comment is from a user vs bot
+            if (issueCommentEvent?.Sender?.Type == UserType.Bot)
+            {
+                _logger.LogInformation("Bypassing GitHub Webhook event for Issue# {number} with action - {action}...", issueCommentEvent?.Issue.Number, issueCommentEvent?.Action);
+                return;
+            }
+
             _logger.LogInformation("Starting the process of the Issue Comment event for Issue# {number} with action - {action}...", (issueCommentEvent?.Issue.Number), issueCommentEvent?.Action);
 
             await _gitHubWebhookRepository.SaveWebhookAsync(headers, issueCommentEvent!).ConfigureAwait(false);
@@ -114,13 +122,15 @@ private async Task ValidateIssue(IssuesEvent issuesEvent)
                     // Check ticket status => From Approved to Validated or Failed
                     if ((issuesEvent?.Issue?.Labels?.Any(i => i.Name.IsIssueApproved())).GetValueOrDefault())
                     {
-                        await _repositoryService.ValidateCreateRepository(issuesEvent!, _gitHubOptions.UseInventoryWithPullRequest);
+                        // Check if the issue form is valid
+                        await _repositoryService.ValidateCreateRepository(issuesEvent!, createPullRequest: false);
                     }
                     else if ((issuesEvent?.Issue?.Labels?.Any(i => i.Name.IsIssueValidated())).GetValueOrDefault())
                     {
                         if (_gitHubOptions.UseInventoryWithPullRequest)
                         {
-                            await _repositoryService.ValidateCreateRepository(issuesEvent!);
+                            // IF validated, create a pull request
+                            await _repositoryService.ValidateCreateRepository(issuesEvent!, createPullRequest: true);
                         }
                         else
                         {
diff --git a/src/DevExcelerateApi/Services/DevExPullRequestEventProcessorService.cs b/src/DevExcelerateApi/Services/DevExPullRequestEventProcessorService.cs
@@ -44,7 +44,7 @@ private async Task ValidatePullRequestApproval(PullRequestEvent pullRequestEvent
         {
             var prNumber = (pullRequestEvent?.PullRequest.Number).GetValueOrDefault();
 
-            _logger.LogInformation("Pull request #{prNumber} is {State} with merge commit SHA {MergeCommitSha}.", prNumber, pullRequestEvent?.PullRequest.State, pullRequestEvent?.PullRequest.MergeCommitSha);
+            _logger.LogInformation("Pull request #{prNumber} is {State} with merge commit SHA {MergeCommitSha}.", prNumber, pullRequestEvent?.PullRequest.State.Value, pullRequestEvent?.PullRequest.MergeCommitSha);
 
             // Closed & Merged
             if (pullRequestEvent?.PullRequest.State == PullRequestState.Closed 
diff --git a/src/DevExcelerateApi/Services/DevExWebhookEventProcessorService.cs b/src/DevExcelerateApi/Services/DevExWebhookEventProcessorService.cs
@@ -6,7 +6,6 @@
 using Octokit.Webhooks.Events.IssueComment;
 using Octokit.Webhooks.Events.Issues;
 using Octokit.Webhooks.Events.PullRequest;
-using Octokit.Webhooks.Models;
 
 namespace DevExcelerateApi.Services
 {
@@ -36,20 +35,26 @@ public class DevExWebhookEventProcessorService(
 
         private readonly IList<IssuesAction> _allowedIssuesActions =
         [
-            IssuesAction.Opened,
+            //IssuesAction.Opened, // When we open an issue, it will always be labeled with the OPENED label
             IssuesAction.Edited,
-            IssuesAction.Closed,
+            //IssuesAction.Closed,
             IssuesAction.Reopened,
             IssuesAction.Labeled
         ];
 
+        private readonly IList<IssueCommentAction> _allowedIssueCommentActions =
+        [
+            IssueCommentAction.Created,
+            IssueCommentAction.Edited            
+        ];
+
         private readonly IList<PullRequestAction> _allowedPullRequestActions =
         [
-            PullRequestAction.Opened,
-            PullRequestAction.Edited,
-            PullRequestAction.Labeled,
+            //PullRequestAction.Opened,
+            //PullRequestAction.Edited,
+            //PullRequestAction.Labeled,
             PullRequestAction.Closed,
-            PullRequestAction.Reopened            
+            //PullRequestAction.Reopened
         ];
 
         public override Task ProcessWebhookAsync(IDictionary<string, StringValues> headers, string body)
@@ -58,18 +63,21 @@ public override Task ProcessWebhookAsync(IDictionary<string, StringValues> heade
             {
                 ArgumentNullException.ThrowIfNull(headers);
                 ArgumentNullException.ThrowIfNull(body);
-
+                 
                 var webhookHeaders = WebhookHeaders.Parse(headers);
 
                 // Bypass if not an implemented event
                 if (_allowedWebhookEvents.Contains(webhookHeaders?.Event!))
                 {
-                    return base.ProcessWebhookAsync(headers, body);
+                    // Not supported by Octokit.Webhooks
+                    if (!body.Contains("\"action\":\"typed\"")) 
+                    {
+                        return base.ProcessWebhookAsync(headers, body);
+                    }
                 }
 
                 _logger.LogDebug("Bypassing GitHub Webhook event {Event}", webhookHeaders?.Event);
                 return Task.CompletedTask;
-
             }
             catch (Exception ex)
             {
@@ -80,48 +88,40 @@ public override Task ProcessWebhookAsync(IDictionary<string, StringValues> heade
 
         protected override Task ProcessIssuesWebhookAsync(WebhookHeaders headers, IssuesEvent issuesEvent, IssuesAction action)
         {
-            // Only process issues events in the onboarding repository
-            // Skip if the Issues action is not supported
-            if (_gitHubOptions?.RepoOnboarding != issuesEvent?.Repository?.Name || !_allowedIssuesActions.Contains(action))
-            {
-                _logger.LogDebug("Bypassing GitHub Webhook event for Issue# {Number} with action: {Action} in repository {RepoName}", issuesEvent?.Issue?.Number, issuesEvent?.Action, issuesEvent?.Repository?.Name);
-                return Task.CompletedTask;
-            }
-
-            if (issuesEvent?.Issue?.State?.Value == IssueState.Closed)
+            // Only process issues events in the onboarding repository & for the allowed actions            
+            if (_gitHubOptions?.RepoOnboarding == issuesEvent?.Repository?.Name && _allowedIssuesActions.Contains(action))
             {
-                _logger.LogDebug("Bypassing closed Issue# {Number} with action: {Action}.", issuesEvent?.Issue?.Number, issuesEvent?.Action);
-                return Task.CompletedTask;
+                return _devExIssuesEventProcessorService.SaveAndProcessIssueEventAsync(headers, issuesEvent!, action);
             }
 
-            return _devExIssuesEventProcessorService.SaveAndProcessIssueEventAsync(headers, issuesEvent!, action);
+            // Skip if the Issues action is not supported
+            _logger.LogDebug("Bypassing GitHub Webhook event for Issue# {Number} with action: {Action} in repository {RepoName}", issuesEvent?.Issue?.Number, issuesEvent?.Action, issuesEvent?.Repository?.Name);
+            return Task.CompletedTask;            
         }
 
         protected override Task ProcessIssueCommentWebhookAsync(WebhookHeaders headers, IssueCommentEvent issueCommentEvent, IssueCommentAction action)
         {
-            // Only process issue comment events in the onboarding repository
-            if (_gitHubOptions?.RepoOnboarding == issueCommentEvent?.Repository?.Name)
+            // Only process issue comment events in the onboarding repository & for the allowed actions
+            if (_gitHubOptions?.RepoOnboarding == issueCommentEvent?.Repository?.Name && _allowedIssueCommentActions.Contains(action))
             {
                 return _devExIssuesEventProcessorService.SaveAndProcessIssueCommentEventAsync(headers, issueCommentEvent!, action);
             }
 
             _logger.LogDebug("Bypassing GitHub Webhook event for Issue# {Number} with action: {Action} in repository {RepoName}", issueCommentEvent?.Issue?.Number, issueCommentEvent?.Action, issueCommentEvent?.Repository?.Name);
             return Task.CompletedTask;
-
         }
     
         protected override Task ProcessPullRequestWebhookAsync(WebhookHeaders headers, PullRequestEvent pullRequestEvent, PullRequestAction action)
         {
-            // Only process issues events in the onboarding repository
-            // Skip if the PR action is not supported
+            // Only process issues events in the onboarding repository & for the allowed actions
             if (_gitHubOptions?.RepoInventory == pullRequestEvent?.Repository?.Name && _allowedPullRequestActions.Contains(action))
             {
                 return _devExPullRequestEventProcessorService.SaveAndProcessPullRequestEventAsync(headers, pullRequestEvent!, action);
             }
 
+            // Skip if the PR action is not supported
             _logger.LogDebug("Bypassing GitHub Webhook event for PR# {Number} with action: {Action} in repository {RepoName}", pullRequestEvent?.PullRequest?.Number, pullRequestEvent?.Action, pullRequestEvent?.Repository?.Name);
             return Task.CompletedTask;
-
         }
     }
 }
diff --git a/src/DevExcelerateApi/Services/IRepositoryService.cs b/src/DevExcelerateApi/Services/IRepositoryService.cs
@@ -4,7 +4,7 @@ namespace DevExcelerateApi.Services
 {
     public interface IRepositoryService
     {
-        Task ValidateCreateRepository(IssuesEvent issuesEvent, bool createPullRequest = true);
+        Task ValidateCreateRepository(IssuesEvent issuesEvent, bool createPullRequest);
 
         Task SaveRepository(IssuesEvent issuesEvent);
 
diff --git a/src/DevExcelerateApi/Services/RepositoryService.cs b/src/DevExcelerateApi/Services/RepositoryService.cs
@@ -24,7 +24,7 @@ public class RepositoryService(
         private readonly GitHubOptions _gitHubOptions = gitHubOptions.Value;
         private readonly IStringLocalizer<RepositoryService> _stringLocalizer = stringLocalizer;
 
-        public async Task ValidateCreateRepository(IssuesEvent issuesEvent, bool createPullRequest = true)
+        public async Task ValidateCreateRepository(IssuesEvent issuesEvent, bool createPullRequest)
         {
             var gitHubClient = await _gitHubClientFactory.GetInstallationAsync(issuesEvent?.Installation?.Id);
             var issueNumber = (issuesEvent?.Issue.Number).GetValueOrDefault();
diff --git a/src/DevExcelerateApp/DevExcelerateApp.csproj b/src/DevExcelerateApp/DevExcelerateApp.csproj
@@ -10,10 +10,10 @@
     <PackageReference Include="Azure.Extensions.AspNetCore.Configuration.Secrets" Version="1.4.0" />
     <PackageReference Include="Azure.Identity" Version="1.13.2" />
     <PackageReference Include="Azure.Security.KeyVault.Keys" Version="4.7.0" />
-    <PackageReference Include="Microsoft.IdentityModel.JsonWebTokens" Version="8.8.0" />
+    <PackageReference Include="Microsoft.IdentityModel.JsonWebTokens" Version="8.9.0" />
     <PackageReference Include="Octokit" Version="14.0.0" />
     <PackageReference Include="Octokit.Webhooks.AspNetCore" Version="2.4.1" />
-    <PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.8.0" />
+    <PackageReference Include="System.IdentityModel.Tokens.Jwt" Version="8.9.0" />
   </ItemGroup>
 
   <ItemGroup>

Original file line number	Diff line number	Diff line change
`@@ -78,10 +78,7 @@ public static async Task Main(string[] args)`
`78`	`78`	`.RequireAuthorization();`
`79`	`79`
`80`	`80`	`// Start the service`
`81`		`- Task runTask = app.RunAsync();`
`82`		`-`
`83`		`- // Wait for the service to complete.`
`84`		`- await runTask;`
	`81`	`+ await app.RunAsync();`
`85`	`82`	`}`
`86`	`83`	`}`
`87`	`84`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@`
`8`	`8`	`using Octokit.Webhooks.Events;`
`9`	`9`	`using Octokit.Webhooks.Events.IssueComment;`
`10`	`10`	`using Octokit.Webhooks.Events.Issues;`
	`11`	`+using Octokit.Webhooks.Models;`
`11`	`12`	`using static DevExcelerateApi.Parsers.GitHubIssueFormParser;`
`12`	`13`
`13`	`14`	`namespace DevExcelerateApi.Services`
`@@ -53,6 +54,13 @@ public async Task SaveAndProcessIssueEventAsync(WebhookHeaders headers, IssuesEv`
`53`	`54`
`54`	`55`	`public async Task SaveAndProcessIssueCommentEventAsync(WebhookHeaders headers, IssueCommentEvent issueCommentEvent, IssueCommentAction action)`
`55`	`56`	`{`
	`57`	`+ // Validate if the issue comment is from a user vs bot`
	`58`	`+ if (issueCommentEvent?.Sender?.Type == UserType.Bot)`
	`59`	`+ {`
	`60`	`+ _logger.LogInformation("Bypassing GitHub Webhook event for Issue# {number} with action - {action}...", issueCommentEvent?.Issue.Number, issueCommentEvent?.Action);`
	`61`	`+ return;`
	`62`	`+ }`
	`63`	`+`
`56`	`64`	`_logger.LogInformation("Starting the process of the Issue Comment event for Issue# {number} with action - {action}...", (issueCommentEvent?.Issue.Number), issueCommentEvent?.Action);`
`57`	`65`
`58`	`66`	`await _gitHubWebhookRepository.SaveWebhookAsync(headers, issueCommentEvent!).ConfigureAwait(false);`
`@@ -114,13 +122,15 @@ private async Task ValidateIssue(IssuesEvent issuesEvent)`
`114`	`122`	`// Check ticket status => From Approved to Validated or Failed`
`115`	`123`	`if ((issuesEvent?.Issue?.Labels?.Any(i => i.Name.IsIssueApproved())).GetValueOrDefault())`
`116`	`124`	`{`
`117`		`- await _repositoryService.ValidateCreateRepository(issuesEvent!, _gitHubOptions.UseInventoryWithPullRequest);`
	`125`	`+ // Check if the issue form is valid`
	`126`	`+ await _repositoryService.ValidateCreateRepository(issuesEvent!, createPullRequest: false);`
`118`	`127`	`}`
`119`	`128`	`else if ((issuesEvent?.Issue?.Labels?.Any(i => i.Name.IsIssueValidated())).GetValueOrDefault())`
`120`	`129`	`{`
`121`	`130`	`if (_gitHubOptions.UseInventoryWithPullRequest)`
`122`	`131`	`{`
`123`		`- await _repositoryService.ValidateCreateRepository(issuesEvent!);`
	`132`	`+ // IF validated, create a pull request`
	`133`	`+ await _repositoryService.ValidateCreateRepository(issuesEvent!, createPullRequest: true);`
`124`	`134`	`}`
`125`	`135`	`else`
`126`	`136`	`{`
Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ private async Task ValidatePullRequestApproval(PullRequestEvent pullRequestEvent`
`44`	`44`	`{`
`45`	`45`	`var prNumber = (pullRequestEvent?.PullRequest.Number).GetValueOrDefault();`
`46`	`46`
`47`		`- _logger.LogInformation("Pull request #{prNumber} is {State} with merge commit SHA {MergeCommitSha}.", prNumber, pullRequestEvent?.PullRequest.State, pullRequestEvent?.PullRequest.MergeCommitSha);`
	`47`	`+ _logger.LogInformation("Pull request #{prNumber} is {State} with merge commit SHA {MergeCommitSha}.", prNumber, pullRequestEvent?.PullRequest.State.Value, pullRequestEvent?.PullRequest.MergeCommitSha);`
`48`	`48`
`49`	`49`	`// Closed & Merged`
`50`	`50`	`if (pullRequestEvent?.PullRequest.State == PullRequestState.Closed`
Original file line number	Diff line number	Diff line change
`@@ -6,7 +6,6 @@`
`6`	`6`	`using Octokit.Webhooks.Events.IssueComment;`
`7`	`7`	`using Octokit.Webhooks.Events.Issues;`
`8`	`8`	`using Octokit.Webhooks.Events.PullRequest;`
`9`		`-using Octokit.Webhooks.Models;`
`10`	`9`
`11`	`10`	`namespace DevExcelerateApi.Services`
`12`	`11`	`{`
`@@ -36,20 +35,26 @@ public class DevExWebhookEventProcessorService(`
`36`	`35`
`37`	`36`	`private readonly IList<IssuesAction> _allowedIssuesActions =`
`38`	`37`	`[`
`39`		`- IssuesAction.Opened,`
	`38`	`+ //IssuesAction.Opened, // When we open an issue, it will always be labeled with the OPENED label`
`40`	`39`	`IssuesAction.Edited,`
`41`		`- IssuesAction.Closed,`
	`40`	`+ //IssuesAction.Closed,`
`42`	`41`	`IssuesAction.Reopened,`
`43`	`42`	`IssuesAction.Labeled`
`44`	`43`	`];`
`45`	`44`
	`45`	`+ private readonly IList<IssueCommentAction> _allowedIssueCommentActions =`
	`46`	`+ [`
	`47`	`+ IssueCommentAction.Created,`
	`48`	`+ IssueCommentAction.Edited`
	`49`	`+ ];`
	`50`	`+`
`46`	`51`	`private readonly IList<PullRequestAction> _allowedPullRequestActions =`
`47`	`52`	`[`
`48`		`- PullRequestAction.Opened,`
`49`		`- PullRequestAction.Edited,`
`50`		`- PullRequestAction.Labeled,`
	`53`	`+ //PullRequestAction.Opened,`
	`54`	`+ //PullRequestAction.Edited,`
	`55`	`+ //PullRequestAction.Labeled,`
`51`	`56`	`PullRequestAction.Closed,`
`52`		`- PullRequestAction.Reopened`
	`57`	`+ //PullRequestAction.Reopened`
`53`	`58`	`];`
`54`	`59`
`55`	`60`	`public override Task ProcessWebhookAsync(IDictionary<string, StringValues> headers, string body)`
`@@ -58,18 +63,21 @@ public override Task ProcessWebhookAsync(IDictionary<string, StringValues> heade`
`58`	`63`	`{`
`59`	`64`	`ArgumentNullException.ThrowIfNull(headers);`
`60`	`65`	`ArgumentNullException.ThrowIfNull(body);`
`61`		`-`
	`66`	`+`
`62`	`67`	`var webhookHeaders = WebhookHeaders.Parse(headers);`
`63`	`68`
`64`	`69`	`// Bypass if not an implemented event`
`65`	`70`	`if (_allowedWebhookEvents.Contains(webhookHeaders?.Event!))`
`66`	`71`	`{`
`67`		`- return base.ProcessWebhookAsync(headers, body);`
	`72`	`+ // Not supported by Octokit.Webhooks`
	`73`	`+ if (!body.Contains("\"action\":\"typed\""))`
	`74`	`+ {`
	`75`	`+ return base.ProcessWebhookAsync(headers, body);`
	`76`	`+ }`
`68`	`77`	`}`
`69`	`78`
`70`	`79`	`_logger.LogDebug("Bypassing GitHub Webhook event {Event}", webhookHeaders?.Event);`
`71`	`80`	`return Task.CompletedTask;`
`72`		`-`
`73`	`81`	`}`
`74`	`82`	`catch (Exception ex)`
`75`	`83`	`{`
`@@ -80,48 +88,40 @@ public override Task ProcessWebhookAsync(IDictionary<string, StringValues> heade`
`80`	`88`
`81`	`89`	`protected override Task ProcessIssuesWebhookAsync(WebhookHeaders headers, IssuesEvent issuesEvent, IssuesAction action)`
`82`	`90`	`{`
`83`		`- // Only process issues events in the onboarding repository`
`84`		`- // Skip if the Issues action is not supported`
`85`		`- if (_gitHubOptions?.RepoOnboarding != issuesEvent?.Repository?.Name \|\| !_allowedIssuesActions.Contains(action))`
`86`		`- {`
`87`		`- _logger.LogDebug("Bypassing GitHub Webhook event for Issue# {Number} with action: {Action} in repository {RepoName}", issuesEvent?.Issue?.Number, issuesEvent?.Action, issuesEvent?.Repository?.Name);`
`88`		`- return Task.CompletedTask;`
`89`		`- }`
`90`		`-`
`91`		`- if (issuesEvent?.Issue?.State?.Value == IssueState.Closed)`
	`91`	`+ // Only process issues events in the onboarding repository & for the allowed actions`
	`92`	`+ if (_gitHubOptions?.RepoOnboarding == issuesEvent?.Repository?.Name && _allowedIssuesActions.Contains(action))`
`92`	`93`	`{`
`93`		`- _logger.LogDebug("Bypassing closed Issue# {Number} with action: {Action}.", issuesEvent?.Issue?.Number, issuesEvent?.Action);`
`94`		`- return Task.CompletedTask;`
	`94`	`+ return _devExIssuesEventProcessorService.SaveAndProcessIssueEventAsync(headers, issuesEvent!, action);`
`95`	`95`	`}`
`96`	`96`
`97`		`- return _devExIssuesEventProcessorService.SaveAndProcessIssueEventAsync(headers, issuesEvent!, action);`
	`97`	`+ // Skip if the Issues action is not supported`
	`98`	`+ _logger.LogDebug("Bypassing GitHub Webhook event for Issue# {Number} with action: {Action} in repository {RepoName}", issuesEvent?.Issue?.Number, issuesEvent?.Action, issuesEvent?.Repository?.Name);`
	`99`	`+ return Task.CompletedTask;`
`98`	`100`	`}`
`99`	`101`
`100`	`102`	`protected override Task ProcessIssueCommentWebhookAsync(WebhookHeaders headers, IssueCommentEvent issueCommentEvent, IssueCommentAction action)`
`101`	`103`	`{`
`102`		`- // Only process issue comment events in the onboarding repository`
`103`		`- if (_gitHubOptions?.RepoOnboarding == issueCommentEvent?.Repository?.Name)`
	`104`	`+ // Only process issue comment events in the onboarding repository & for the allowed actions`
	`105`	`+ if (_gitHubOptions?.RepoOnboarding == issueCommentEvent?.Repository?.Name && _allowedIssueCommentActions.Contains(action))`
`104`	`106`	`{`
`105`	`107`	`return _devExIssuesEventProcessorService.SaveAndProcessIssueCommentEventAsync(headers, issueCommentEvent!, action);`
`106`	`108`	`}`
`107`	`109`
`108`	`110`	`_logger.LogDebug("Bypassing GitHub Webhook event for Issue# {Number} with action: {Action} in repository {RepoName}", issueCommentEvent?.Issue?.Number, issueCommentEvent?.Action, issueCommentEvent?.Repository?.Name);`
`109`	`111`	`return Task.CompletedTask;`
`110`		`-`
`111`	`112`	`}`
`112`	`113`
`113`	`114`	`protected override Task ProcessPullRequestWebhookAsync(WebhookHeaders headers, PullRequestEvent pullRequestEvent, PullRequestAction action)`
`114`	`115`	`{`
`115`		`- // Only process issues events in the onboarding repository`
`116`		`- // Skip if the PR action is not supported`
	`116`	`+ // Only process issues events in the onboarding repository & for the allowed actions`
`117`	`117`	`if (_gitHubOptions?.RepoInventory == pullRequestEvent?.Repository?.Name && _allowedPullRequestActions.Contains(action))`
`118`	`118`	`{`
`119`	`119`	`return _devExPullRequestEventProcessorService.SaveAndProcessPullRequestEventAsync(headers, pullRequestEvent!, action);`
`120`	`120`	`}`
`121`	`121`
	`122`	`+ // Skip if the PR action is not supported`
`122`	`123`	`_logger.LogDebug("Bypassing GitHub Webhook event for PR# {Number} with action: {Action} in repository {RepoName}", pullRequestEvent?.PullRequest?.Number, pullRequestEvent?.Action, pullRequestEvent?.Repository?.Name);`
`123`	`124`	`return Task.CompletedTask;`
`124`		`-`
`125`	`125`	`}`
`126`	`126`	`}`
`127`	`127`	`}`
Original file line number	Diff line number	Diff line change
`@@ -4,7 +4,7 @@ namespace DevExcelerateApi.Services`
`4`	`4`	`{`
`5`	`5`	`public interface IRepositoryService`
`6`	`6`	`{`
`7`		`- Task ValidateCreateRepository(IssuesEvent issuesEvent, bool createPullRequest = true);`
	`7`	`+ Task ValidateCreateRepository(IssuesEvent issuesEvent, bool createPullRequest);`
`8`	`8`
`9`	`9`	`Task SaveRepository(IssuesEvent issuesEvent);`
`10`	`10`
Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ public class RepositoryService(`
`24`	`24`	`private readonly GitHubOptions _gitHubOptions = gitHubOptions.Value;`
`25`	`25`	`private readonly IStringLocalizer<RepositoryService> _stringLocalizer = stringLocalizer;`
`26`	`26`
`27`		`- public async Task ValidateCreateRepository(IssuesEvent issuesEvent, bool createPullRequest = true)`
	`27`	`+ public async Task ValidateCreateRepository(IssuesEvent issuesEvent, bool createPullRequest)`
`28`	`28`	`{`
`29`	`29`	`var gitHubClient = await _gitHubClientFactory.GetInstallationAsync(issuesEvent?.Installation?.Id);`
`30`	`30`	`var issueNumber = (issuesEvent?.Issue.Number).GetValueOrDefault();`