Skip to content

Increase attachment remote_url limit#688

Open
alphatownsman wants to merge 1 commit into
jointakahe:mainfrom
neodb-social:attachment-url-limit
Open

Increase attachment remote_url limit#688
alphatownsman wants to merge 1 commit into
jointakahe:mainfrom
neodb-social:attachment-url-limit

Conversation

@alphatownsman

Copy link
Copy Markdown
Contributor

Current limit 500 is too short for some threads post.

Raise it to 2500 based on this

{
	attachment: {
		height: 1280,
		type: 'Video',
		url: 'https://scontent.cdninstagram.com/o1/v/t16/f1/m69/GGt50Rg-JZwdl6oCAGPnidPSuQV_bkYLAAAF?_nc_ht=scontent.cdninstagram.com&_nc_cat=104&vs=234542236356205_3726029553&_nc_vs=HBksFQIYOnBhc3N0aHJvdWdoX2V2ZXJzdG9yZS9HR3Q1MFJnLUpad2RsNm9DQUdQbmlkUFN1UVZfYmtZTEFBQUYVAALIAQAVAhg6cGFzc3Rocm91Z2hfZXZlcnN0b3JlL0dPYTAwQmlUMVE3bU5la0dBSkh4WUFSbHVxUnBia1lMQUFBRhUCAsgBACgAGAAbAYgHdXNlX29pbAExFQAAJo6VqYKW4YBAFQIoAkMzLBdAN3eNT987ZBgSZGFzaF9iYXNlbGluZV8xX3YxEQB16gcA&ccb=9-4&oh=00_AfBK85KEGCDt8QZ1KLuerI4o7qI8S8CvwKGrrODNxdwx9Q&oe=659A1CFF&_nc_sid=1d576d&_nc_rid=92c86d303e',
		width: 720
	},
	attributedTo: 'https://www.threads.net/ap/users/mosseri/',
	cc: 'https://www.threads.net/ap/users/mosseri/followers/',
	content: '<p>From this week’s ask me anything on Instagram about recent low quality Threads recommendations. We’re on it.</p>',
	id: 'https://www.threads.net/ap/users/mosseri/post/18015967712009007/',
	published: '2024-01-05T15:24:25-0800',
	tag: [],
	to: 'as:Public',
	type: 'Note',
	url: 'https://www.threads.net/@mosseri/post/C1vJ1MoPOTc'
}

@andrewgodwin

Copy link
Copy Markdown
Member

Can you provide a URL of a specific post that breaches the limit?

@alphatownsman

Copy link
Copy Markdown
Contributor Author

@andrewgodwin there is one in the json in the description. Is that what you look for?

@andrewgodwin

Copy link
Copy Markdown
Member

I guess that'll do - I wanted something authoritative but I trust that that's a direct copy. It'd be nice to have a test for ingesting threads posts, is all.

@AstraLuma AstraLuma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

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.

3 participants