Update image URLs in info.py#61
Update image URLs in info.py#61rimimondal4545-cloud wants to merge 1 commit intoLearningBotsOfficial:mainfrom
Conversation
📝 WalkthroughWalkthroughDefault image URLs for Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
| START_IMG = (environ.get('START_IMG', 'https://files.catbox.moe/ithgp4.jpg')).split() | ||
| FORCESUB_IMG = environ.get('FORCESUB_IMG', 'https://i.ibb.co/ZNC1Hnb/ad3f2c88a8f2.jpg') | ||
| REFER_PICS = (environ.get("REFER_PICS", "https://envs.sh/PSI.jpg")).split() | ||
| PAYPICS = (environ.get('PAYPICS', 'https://envs.sh/_kA.jpg')).split() | ||
| PAYPICS = (environ.get('PAYPICS', 'https://files.catbox.moe/kecx57.jpg')).split() |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
git ls-files | head -20Repository: LearningBotsOfficial/movies
Length of output: 515
🏁 Script executed:
find . -name "info.py" -type f 2>/dev/nullRepository: LearningBotsOfficial/movies
Length of output: 80
🏁 Script executed:
#!/bin/bash
# Verify URL accessibility for the mentioned image URLs
echo "=== Verifying Image URLs ==="
echo ""
echo "Checking START_IMG URL..."
START_URL="https://files.catbox.moe/ithgp4.jpg"
START_RESPONSE=$(curl -s -o /dev/null -w "%{http_code}" -L "$START_URL" --max-time 5)
START_CONTENT=$(curl -s -I -L "$START_URL" --max-time 5 2>&1)
echo "START_IMG ($START_URL)"
echo "HTTP Status: $START_RESPONSE"
echo "Response headers:"
echo "$START_CONTENT" | head -5
echo ""
echo "Checking PAYPICS URL..."
PAY_URL="https://files.catbox.moe/kecx57.jpg"
PAY_RESPONSE=$(curl -s -o /dev/null -w "%{http_code}" -L "$PAY_URL" --max-time 5)
PAY_CONTENT=$(curl -s -I -L "$PAY_URL" --max-time 5 2>&1)
echo "PAYPICS ($PAY_URL)"
echo "HTTP Status: $PAY_RESPONSE"
echo "Response headers:"
echo "$PAY_CONTENT" | head -5
echo ""
echo "Checking FORCESUB_IMG URL..."
FORCESUB_URL="https://i.ibb.co/ZNC1Hnb/ad3f2c88a8f2.jpg"
FORCESUB_RESPONSE=$(curl -s -o /dev/null -w "%{http_code}" -L "$FORCESUB_URL" --max-time 5)
echo "FORCESUB_IMG ($FORCESUB_URL)"
echo "HTTP Status: $FORCESUB_RESPONSE"
echo ""
echo "Checking REFER_PICS URL..."
REFER_URL="https://envs.sh/PSI.jpg"
REFER_RESPONSE=$(curl -s -o /dev/null -w "%{http_code}" -L "$REFER_URL" --max-time 5)
echo "REFER_PICS ($REFER_URL)"
echo "HTTP Status: $REFER_RESPONSE"Repository: LearningBotsOfficial/movies
Length of output: 721
🏁 Script executed:
cat -n ./info.py | sed -n '65,80p'Repository: LearningBotsOfficial/movies
Length of output: 1398
Image URLs for catbox.moe are confirmed accessible and serving valid image content.
The catbox.moe URLs for START_IMG and PAYPICS (lines 71, 74) are accessible and return valid JPEG images. However, REFER_PICS (line 73) still references envs.sh, which returns a 502 Bad Gateway error and should be migrated to a working hosting provider.
🤖 Prompt for AI Agents
In info.py around lines 71 to 74, the default for REFER_PICS points to envs.sh
which returns 502; replace that default with a working image host (e.g., a
catbox.moe or other reliable URL) and keep the .split() usage consistent with
START_IMG/PAYPICS so the code receives a list; update the literal default string
to the new working URL and verify the chosen URL returns 200 and serves an
image.
| #--------------------------------------------------------------- | ||
| #--------------------------------------------------------------- | ||
| START_IMG = (environ.get('START_IMG', 'https://i.ibb.co/qpxpGmC/image.jpg https://i.ibb.co/DQ35zLZ/image.jpg')).split() | ||
| START_IMG = (environ.get('START_IMG', 'https://files.catbox.moe/ithgp4.jpg')).split() |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
find . -name "info.py" -type fRepository: LearningBotsOfficial/movies
Length of output: 80
🏁 Script executed:
git log --oneline -5 -- "info.py"Repository: LearningBotsOfficial/movies
Length of output: 107
🏁 Script executed:
git show HEAD:info.py | sed -n '60,85p'Repository: LearningBotsOfficial/movies
Length of output: 1796
🏁 Script executed:
rg "START_IMG" -B 2 -A 2Repository: LearningBotsOfficial/movies
Length of output: 3434
🏁 Script executed:
git show HEAD~1:info.py | sed -n '60,85p'Repository: LearningBotsOfficial/movies
Length of output: 107
🏁 Script executed:
git diff HEAD~1 HEAD -- info.py | head -50Repository: LearningBotsOfficial/movies
Length of output: 99
🏁 Script executed:
git log --all --oneline -- info.pyRepository: LearningBotsOfficial/movies
Length of output: 384
🏁 Script executed:
rg "random.choice\(START_IMG\)" -B 3 -A 3Repository: LearningBotsOfficial/movies
Length of output: 1327
🏁 Script executed:
git show c9b39c4:info.py | sed -n '60,85p'Repository: LearningBotsOfficial/movies
Length of output: 1843
Behavior change: START_IMG reduced from 2 images to 1.
The previous default provided two space-separated image URLs which .split() converted to a 2-element list. The new default provides only one URL, resulting in a 1-element list. The codebase uses random.choice(START_IMG) in two places (plugins/commands.py), which was designed for selecting from multiple start images. With only one image, the random selection feature is effectively disabled.
This was a regression of the intentional feature added in commit c9b39c4 ("You can now place multiple start images"). Restore the multi-image default or clarify if single-image behavior is now intended.
🤖 Prompt for AI Agents
In info.py around line 71, the START_IMG default was changed to a single URL
which makes .split() produce a 1-element list and disables the intended
random-choice behavior; restore the original multi-image default by changing the
environ.get default string back to a space-separated list of the original two
start image URLs so .split() yields multiple entries (or, if single-image
behavior is intended, update the two places that call random.choice(START_IMG)
to use deterministic selection instead).
1 participant
Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.