ansible / eda-server

Event Driven Ansible for AAP
Apache License 2.0
66 stars 42 forks source link

Change webhook to eventstream #1011

Closed mkanoor closed 3 weeks ago

ttuffin commented 3 weeks ago

Hi @mkanoor, as discussed lets include the following change in this PR as well:

diff --git a/src/aap_eda/core/enums.py b/src/aap_eda/core/enums.py
index 8c7e1c5..1b9a0ba 100644
--- a/src/aap_eda/core/enums.py
+++ b/src/aap_eda/core/enums.py
@@ -164,14 +164,14 @@ class WebhookCredentialType(DjangoStrEnum):
     HMAC = "HMAC Webhook"
     BASIC = "Basic Webhook"
     TOKEN = "Token Webhook"
-    OAUTH2 = "Oauth2 Webhook"
-    OAUTH2_JWT = "Oauth2 JWT Webhook"
+    OAUTH2 = "OAuth2 Webhook"
+    OAUTH2_JWT = "OAuth2 JWT Webhook"
     ECDSA = "ECDSA Webhook"
     MTLS = "mTLS Webhook"

 class CustomWebhookCredentialType(DjangoStrEnum):
-    GITLAB = "GITLAB Webhook"
+    GITLAB = "GitLab Webhook"
     GITHUB = "GitHub Webhook"
-    SNOW = "Service Now Webhook"
+    SNOW = "ServiceNow Webhook"
     DYNATRACE = "Dynatrace Webhook"
rooftopcellist commented 3 weeks ago

@Alex-Izquierdo @ttuffin Here is the associated operator PR:

sonarcloud[bot] commented 3 weeks ago

Quality Gate Passed Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
1.6% Duplication on New Code

See analysis details on SonarCloud