Skip to content

security-fixes#1617

Merged
stefangutica merged 4 commits into
developmentfrom
security-fixes
Jul 21, 2026
Merged

security-fixes#1617
stefangutica merged 4 commits into
developmentfrom
security-fixes

Conversation

@stefangutica

Copy link
Copy Markdown
Collaborator

Reasoning

  • Improve security around NFT media processing.
  • Reduce the risk of unsafe redirects and SSRF-style requests.

Proposed Changes

  • Added URL safety checks to validate public versus internal destinations.
  • Updated NFT asset, media, and thumbnail requests to skip redirects.

How to test

  • Verify NFT uploads and thumbnail generation still work for normal public URLs.
  • Confirm unsafe or internal URLs are handled safely and rejected.

@github-actions

github-actions Bot commented Jul 20, 2026

Copy link
Copy Markdown

k6 load testing comparison.
Base Commit Hash: 80d70f1
Target Commit Hash: ae23554

Metric Base Target Diff
AvgMax9095AvgMax9095AvgMax9095
Nodes951.1836551.01180.14199.72174.2539347.6687.8691.99-81.68% ✅+7.65% 🔴-51.23% ✅-53.94% ✅
Mex62.01602.9497.32246.7673.851898.78161.79236.43+19.10% 🔴+214.92% 🔴+66.24% 🔴-4.18% ✅
Accounts63.02730.5992.87255.7572.36688.64144.36235.90+14.82% 🔴-5.74% ✅+55.45% 🔴-7.76% ✅
Pool62.95616.6599.05247.2873.111170.26152.66236.93+16.15% 🔴+89.78% 🔴+54.13% 🔴-4.18% ✅
Tokens84.772144.47107.93116.6498.641237.30107.38136.40+16.36% 🔴-42.30% ✅-0.51% ✅+16.95% 🔴
Transactions575.6236801.0597.63138.31262.2638966.69151.64158.38-54.44% ✅+5.88% 🔴+55.31% 🔴+14.51% 🔴
Blocks82.45659.77138.39338.4995.571875.22237.40303.54+15.91% 🔴+184.22% 🔴+71.55% 🔴-10.32% ✅
Test Run Duration60004.7160002.67

Legend: Avg - Average Response Time, Max - Maximum Response Time, 90 - 90th Percentile, 95 - 95th Percentile
All times are in milliseconds.

@stefangutica
stefangutica merged commit 60014c6 into development Jul 21, 2026
5 of 9 checks passed
@stefangutica
stefangutica deleted the security-fixes branch July 21, 2026 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants