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

Verification information does not mention 1 MB response limit #1401

Closed
cassidyjames opened this issue Feb 9, 2024 · 2 comments · Fixed by #1402
Closed

Verification information does not mention 1 MB response limit #1401

cassidyjames opened this issue Feb 9, 2024 · 2 comments · Fixed by #1402
Labels
user guide User-facing documentation

Comments

@cassidyjames
Copy link
Contributor

When encouraging accounts to get verified, I've seen more than once some get tripped up by the 1 MB response limit. We should probably clarify exactly what that limitation is.

@andypiper andypiper added the user guide User-facing documentation label Feb 9, 2024
@andypiper
Copy link
Sponsor Member

I am guessing that this is a limit on the size of the HTTP response returned when the URL to be verified is called by the Mastodon server? (I used to see something similar in relation to the Twitter Cards crawler breaking if the HTTP response contained a huge HTML document with tons of inline CSS and JS). Interesting! Yes, definitely worth adding this to the docs if so.

@cassidyjames
Copy link
Contributor Author

cassidyjames commented Feb 9, 2024

Yep, I forget the exact details, but if it might be something like verification used to fail completely if the response was >1MB, but it was updated to truncate the response at 1MB and not fail outright? I'm not 100% sure.

You can actually see this failing here for the bare https://endlessos.org/ link: https://mastodon-link-debugger.vercel.app/profile/@[email protected]

So adding a note here https://github.com/mastodon/documentation/blob/main/content/en/user/profile.md#link-verification-verification is probably a good idea?

#1402

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
user guide User-facing documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants