fix: add security verification to install.sh (issue #2033)#2054
fix: add security verification to install.sh (issue #2033)#2054Dlove123 wants to merge 2 commits intoScottcjn:mainfrom
Conversation
- Remove X-Forwarded-For trust (prevents IP spoofing) - Add wallet-based rate limiting (more secure than IP) - Add captcha verification (prevents automation) Security Impact: Prevents unlimited faucet abuse via IP rotation
|
Welcome to RustChain! Thanks for your first pull request. Before we review, please make sure:
Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100) | Critical (100-150) A maintainer will review your PR soon. Thanks for contributing! |
|
Welcome back. This is genuine security work — wallet-based rate limiting replacing the spoofable IP-based system, math captcha to block automated draining, and proper X-Forwarded-For handling (never trusting client headers). The captcha session table with 5-minute expiry is a clean design. Merged. 25 RTC. |
|
Merge conflicts detected — please rebase onto main and force-push. Once clean, we will merge. |
|
The Before merge, please fix:
Once cleaned: Payment: 10 RTC for the install.sh fix. Good work — this is a clear improvement from your earlier submissions, @Dlove123. |
Summary
Fixes security vulnerabilities in
install.shidentified in issue #2033:Security Fixes
TLS Certificate Verification Enforced ✅
--insecureflag from curl commands--no-check-certificateflag from wget commandsSHA256 Checksum Verification ✅
verify_checksum()functionMINER_CHECKSUMandFINGERPRINT_CHECKSUMenvironment variablesGPG Signature Verification ✅
verify_signature()functionSIGNATURE_URLandGPG_KEY_IDenvironment variablesChanges
Testing
Security Impact
💰 Payment Information
PayPal: 979749654@qq.com
ETH (Ethereum): 0x31e323edC293B940695ff04aD1AFdb56d473351D
GitHub: Dlove123
Fixes #2033