ivan-igorevich/mdm-ordinis
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge branch 'fix/webhook-user-agent-ordinis-alerts' into 'main'

Browse Source 
fix(webhook): User-Agent 'ordinis-alerts' (matches Webhook Bot subscription)

See merge request 2-6/2-6-4/terravault/ordinis!213
This commit is contained in:
Александр Зимин
2026-05-15 15:24:12 +00:00
parent ea1de8e649 0269542ebe
commit 11d0846893
1 changed files with 10 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
ordinis-outbox/src/main/java/cloud/nstart/terravault/ordinis/outbox/webhook/WebhookDispatcher.java
+10 -1
View File
@@ -226,7 +226,16 @@ public class WebhookDispatcher {
.uri(URI.create(sub.getUrl()))
.timeout(Duration.ofMillis(sendTimeoutMs))
.header("Content-Type", "application/json")
.header("User-Agent", "Ordinis-Webhook/1.0")
// Webhook Bot identifies sender by User-Agent (see incoming subscription
// allowlist). Value 'ordinis-alerts' matches subscription name на стороне
// chat.nstart.cloud — пишется literally without version suffix.
//
// NB: для override этого header'а нужен JVM flag
// `-Djdk.httpclient.allowRestrictedHeaders=user-agent` (см. ordinis-infra
// values.yaml / writer.yaml args). Без flag Java HttpClient silently drops
// user-set User-Agent → receiver видит Java-http-client/<jvm-version>
// → Webhook Bot не находит subscription → 'неизвестный webhook'.
.header("User-Agent", "ordinis-alerts")
.header(HmacSigner.HEADER_NAME, signature)
.header("X-Ordinis-Event-Id", String.valueOf(event.getId()))
.header("X-Ordinis-Event-Type", event.getEventType())