Skip to content

shortcode: Add py.md as alternative to cutt.ly#3001

Open
Kraptor123 wants to merge 5 commits into
recloudstream:masterfrom
Kraptor123:short-code
Open

shortcode: Add py.md as alternative to cutt.ly#3001
Kraptor123 wants to merge 5 commits into
recloudstream:masterfrom
Kraptor123:short-code

Conversation

@Kraptor123

@Kraptor123 Kraptor123 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor

cutt.ly is currently blocked in Turkiye, so users are getting connection errors when trying to load repos with shortcodes. I wrapped the request in a try-catch and added py.md as a fallback. Now if cutt.ly fails, it just tries py.md instead.

@fire-light42 fire-light42 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The code and motivation are both good, but this option opens up vulnerabilities in the code system. For example what if someone maliciously registers a short code for a popular repository with py.md?

I think the best option is to have a different prefix for py.md links, or something to visibly distinguish them from known short codes. Different users should not get different repositories with the same short code.

@Kraptor123 Kraptor123 requested a review from fire-light42 July 4, 2026 21:10
@Kraptor123

Kraptor123 commented Jul 4, 2026

Copy link
Copy Markdown
Contributor Author

i can add all known repos to my account if thats okey so users can access every repo without any problem.

@Kraptor123 Kraptor123 changed the title shortcode: Add py.md as a fallback if cutt.ly fails shortcode: Add py.md as alternative to cutt.ly Jul 4, 2026
@fire-light42

Copy link
Copy Markdown
Collaborator

I like the idea of "!", which we can easily migrate away from in the future if we decide to remove cutt.ly.

However, creating all short codes leaves your account as another single point of failure. I recommend that you instead communicate with fellow repository maintainers to let them create their own short codes before I merge this.

@fire-light42 fire-light42 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good, idea is good, just want some feedback from other developers.

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.

2 participants