Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

"View on Tumblr" links should go to blog's domain/subdomain if possible #32

Open
the-blank-x opened this issue Feb 25, 2024 · 5 comments
Labels
enhancement New feature or request

Comments

@the-blank-x
Copy link
Contributor

the-blank-x commented Feb 25, 2024

There are three types of blogs:

I prefer browsing blogs with their custom theme over using Priviblur (which I prefer over using the crappy interface at www.tumblr.com), so when I land on a new blog while on Priviblur, I try to navigate to https://<blog>.tumblr.com to see if it has a custom theme. However, Priviblur's "View on Tumblr" links points to the subdomain/domain-less versions.

I've already made Priviblur use the domain or subdomain versions here, but I'm making this issue first to see if this would be accepted upstream.

@syeopite
Copy link
Owner

Sounds good to me though I'm against this being the default behavior. Feel free to make a PR once settings are added, which should hopefully be before the end of the month if nothing goes wrong.

@syeopite syeopite added the enhancement New feature or request label Mar 13, 2024
@the-blank-x
Copy link
Contributor Author

Sure, though I wonder if you could let instances pick their default settings (akin to Nitter)

@syeopite
Copy link
Owner

Yeah instances will definitely be able to configure the default settings.

@syeopite
Copy link
Owner

I do wonder if there are any security risks to users for Priviblur directly linking (which can be taken as endorsing) to a blog's subdomain considering that there have been a few cases in the past where blogs were using malicious scripts within their custom themes.

Those are usually caught pretty quickly by Tumblr but it is still a risk factor that exists for custom themes but not with Tumblr's dashboard layout.

@the-blank-x
Copy link
Contributor Author

the-blank-x commented Jun 11, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants