[PR #132] [MERGED] Validate affiliate and Firefox profile inputs early #129

New issue

Closed

opened 2026-03-02 04:06:55 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 04:06:55 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/FujiwaraChoki/MoneyPrinterV2/pull/132
Author: @FujiwaraChoki
Created: 3/1/2026
Status: ✅ Merged
Merged: 3/1/2026
Merged by: @FujiwaraChoki

Base: main ← Head: fix/afm-input-validation

📝 Commits (1)

88cfc6f Validate affiliate and Firefox profile inputs early

📊 Changes

1 file changed (+48 additions, -16 deletions)

View changed files

📝 src/classes/AFM.py (+48 -16)

📄 Description

Summary

validate Firefox profile paths before launching Selenium in the affiliate marketing flow
validate affiliate links as full HTTP(S) URLs before browser navigation
return clear ValueError messages that explain exactly which input is invalid

Why this change

Current failures surface as low-level Selenium InvalidArgumentException errors that are hard to debug. Early validation turns these into immediate, user-actionable setup errors and avoids unnecessary browser startup work.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/FujiwaraChoki/MoneyPrinterV2/pull/132 **Author:** [@FujiwaraChoki](https://github.com/FujiwaraChoki) **Created:** 3/1/2026 **Status:** ✅ Merged **Merged:** 3/1/2026 **Merged by:** [@FujiwaraChoki](https://github.com/FujiwaraChoki) **Base:** `main` ← **Head:** `fix/afm-input-validation` --- ### 📝 Commits (1) - [`88cfc6f`](https://github.com/FujiwaraChoki/MoneyPrinterV2/commit/88cfc6fd2230db59999cff8850295b71a99d02eb) Validate affiliate and Firefox profile inputs early ### 📊 Changes **1 file changed** (+48 additions, -16 deletions) <details> <summary>View changed files</summary> 📝 `src/classes/AFM.py` (+48 -16) </details> ### 📄 Description ## Summary - validate Firefox profile paths before launching Selenium in the affiliate marketing flow - validate affiliate links as full HTTP(S) URLs before browser navigation - return clear `ValueError` messages that explain exactly which input is invalid ## Why this change Current failures surface as low-level Selenium `InvalidArgumentException` errors that are hard to debug. Early validation turns these into immediate, user-actionable setup errors and avoids unnecessary browser startup work. ## Related issues - #90 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>