Web hook validation in 2.x+ not present anymore?
@jskeet ya está trabajando en esto.
Desde el 18/10/2022.
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Related to fix #145
I am trying to update my cloud Events sdk to the newest version with the fix included but I can't make the validation work anymore. Is the fix not even relevant anymore?
I used to have this:
public async Task<HttpResponseMessage> HandleSubscriptionValidationEvent(HttpRequestMessage req)
{
if (!await req.IsWebHookValidationRequest())
{
throw new NotSupportedException("The request is not a WebHook validation request!");
}
// Add missing headers (for azure event grid doesnt send these)
if (!req.Headers.Contains("WebHook-Request-Rate"))
{
req.Headers.Add("WebHook-Request-Rate", "*");
}
if (req.Headers.Contains("Webhook-Request-Callback"))
{
req.Headers.Remove("Webhook-Request-Callback");
}
var response = await req.HandleAsWebHookValidationRequest((origin) =>
{
var setting = _settings.WebHookAllowedOrigin ?? Environment.GetEnvironmentVariable("WebHook-Request-Origin");
if (string.IsNullOrEmpty(setting) || setting == "*")
{
return true;
}
else
{
return setting == origin;
}
}, (o) =>
{
return _settings.WebHookAllowedRate ?? Environment.GetEnvironmentVariable("WebHook-Request-Rate") ?? "*";
});
return response;
But the IsWebHookValidationRequest and HandleAsWebHookValidationRequest are gone, how do i do this now?
Sandro
- Lenguaje dominante
- C#
- Estrellas
- 334
- Forks
- 88
- Merge medio
- 6 min
- PR fusionados (30 d)
- 1
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de cloudevents/sdk-csharp
-
Include specversion in CloudEvent.GetPopulatedAttributesQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
cloudevents/sdk-csharp#373 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
cloudevents/sdk-csharp#324 · 1 comentario ·
-
Allow users to signal whether extensions are requiredQuizá libre de nuevo @jskeet la tomó hace 519 días y no hay ningún pull request abierto. Abierto
cloudevents/sdk-csharp#321 · 4 comentarios · 1 asignado ·
-
CloudNative.CloudEvents.RabbitMqAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
cloudevents/sdk-csharp#310 · 6 comentarios ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
cloudevents/sdk-csharp#308 · 5 comentarios · 1 reacción ·
Todos los issues de cloudevents/sdk-csharp
Issues similares
-
[i18n] 安装实例完成后的成功提示未正确本地化Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
PCL-Community/PCL-CE#3658 ·
Los mantenedores suelen responder en 1 día
-
Deploy & Patch-issues opprettes ikke: create-pnd-issues.yml har feilet hver uke siden 2025-09-08Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Altinn/altinn-auth#4359 ·
Los mantenedores suelen responder en 1 día
-
アプリ: チャット 優先: 中 提案
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
yksr-melt/Meltype#243 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Workflows: a workflow stored with null conditions is skipped with an exception instead of runAbiertobug core
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
type/automation type/tech-debt
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día