Skip to content

UN-3815 [FIX] Validate webhook URLs in one place, at both sinks - #2214

Open
athul-rs wants to merge 20 commits into
mainfrom
UN-3794-webhook-egress
Open

UN-3815 [FIX] Validate webhook URLs in one place, at both sinks#2214
athul-rs wants to merge 20 commits into
mainfrom
UN-3794-webhook-egress

Merge remote-tracking branch 'origin/main' into UN-3794-webhook-egress

c394f51
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Sep 4, 2026 in 49s

Annotations

Check warning on line 51 in unstract/core/src/unstract/core/network/ssrf.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Fix the syntax of this issue suppression comment.

See more on https://sonarcloud.io/project/issues?id=Zipstack_unstract&issues=AaBtYfcRqippGUiyXngh&open=AaBtYfcRqippGUiyXngh&pullRequest=2214

Check warning on line 82 in unstract/core/src/unstract/core/network/ssrf.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this redundant Exception class; it derives from another which is already caught.

See more on https://sonarcloud.io/project/issues?id=Zipstack_unstract&issues=AZ_10cKtQxgd4b4M9spU&open=AZ_10cKtQxgd4b4M9spU&pullRequest=2214

Check failure on line 153 in unstract/core/src/unstract/core/network/ssrf.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 18 to the 15 allowed.

See more on https://sonarcloud.io/project/issues?id=Zipstack_unstract&issues=AZ_10cKtQxgd4b4M9spV&open=AZ_10cKtQxgd4b4M9spV&pullRequest=2214