Web hook validation in 2.x+ not present anymore?
@jskeet ci sta già lavorando.
Dal 18/10/2022.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
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
- Lingua principale
- C#
- Stelle
- 334
- Fork
- 88
- Merge medio
- 6m
- PR unite (30g)
- 1
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di cloudevents/sdk-csharp
-
Include specversion in CloudEvent.GetPopulatedAttributesForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 68/100
cloudevents/sdk-csharp#373 · 1 commento ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
cloudevents/sdk-csharp#324 · 1 commento ·
-
Allow users to signal whether extensions are requiredForse di nuovo libera @jskeet l’ha presa 518 giorni fa e non c’è nessuna pull request aperta. Aperta
cloudevents/sdk-csharp#321 · 4 commenti · 1 assegnatario ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
cloudevents/sdk-csharp#310 · 6 commenti ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
cloudevents/sdk-csharp#308 · 5 commenti · 1 reazione ·
Tutte le issue di cloudevents/sdk-csharp
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
stryker-mutator/stryker-net#3892 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
MobiFlight/MobiFlight-Connector#3419 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Kryptos-FR/MarkView.Avalonia#105 ·
I maintainer di solito rispondono entro 1 giorno
-
[辞書]Aperta提案 辞書
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
microsoft/fluentui-blazor#5410 ·
I maintainer di solito rispondono entro 1 giorno