Skip to content

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

Open
athul-rs wants to merge 11 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 11 commits into
mainfrom
UN-3794-webhook-egress

UN-3815 [FIX] Propagate the unstract-core idna dependency to every lo…

90fce36
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Aug 12, 2026 in 1m 35s

Annotations

Check warning on line 325 in workers/notification/tasks.py

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Replace this generic exception class with a more specific one.

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

Check failure on line 146 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 17 to the 15 allowed.

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

Check warning on line 90 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