fix: ハイフンを含むリモート絵文字が描画されない

[anatawa12]

What

ハイフンを含むリモート絵文字が描画されないのを修正します。

Why

現行実装ではインポート時などに - を含む絵文字が何も問題なくインポートされ、またインポートを繰り返すことで世の中に数多く存在し、またmfm上では-を含む絵文字を許可しているため、ローカルでは描画されリモートでは正しく描画されないという非対称な状態になっています。
現状絵文字を削除すると過去のノートの描画がすべて壊れてしまうため削除して再追加も現実的ではないため - を含むリモート絵文字を描画できるようにしたいです

Additional info (optional)

Checklist

• Read the contribution guide
• Test working in a local environment
• (If needed) Add story of storybook
• (If needed) Update CHANGELOG.md
• (If possible) Add tests

[github-actions]

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.93%. Comparing base (bba3097) to head (1318270).

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #13715      +/-   ##
===========================================
- Coverage    66.30%   64.93%   -1.37%     
===========================================
  Files          988      985       -3     
  Lines       116273   111734    -4539     
  Branches      5749     4388    -1361     
===========================================
- Hits         77096    72556    -4540     
- Misses       37743    39147    +1404     
+ Partials      1434       31    -1403     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mei23]

APから来る絵文字名はバリデーションしてなくて、なんでも入れ放題ぽい…

misskey/packages/backend/src/core/activitypub/models/ApNoteService.ts

Lines 370 to 401 in bba3097

	public async extractEmojis(tags: IObject | IObject[], host: string): Promise<MiEmoji[]> {
	// eslint-disable-next-line no-param-reassign
	host = this.utilityService.toPuny(host);
	const eomjiTags = toArray(tags).filter(isEmoji);
	const existingEmojis = await this.emojisRepository.findBy({
	host,
	name: In(eomjiTags.map(tag => tag.name.replaceAll(':', ''))),
	});
	return await Promise.all(eomjiTags.map(async tag => {
	const name = tag.name.replaceAll(':', '');
	tag.icon = toSingle(tag.icon);
	const exists = existingEmojis.find(x => x.name === name);
	if (exists) {
	if ((exists.updatedAt == null)
	|| (tag.id != null && exists.uri == null)
	|| (new Date(tag.updated) > exists.updatedAt)
	|| (tag.icon.url !== exists.originalUrl)
	) {
	await this.emojisRepository.update({
	host,
	name,
	}, {
	uri: tag.id,
	originalUrl: tag.icon.url,
	publicUrl: tag.icon.url,
	updatedAt: new Date(),
	});

[mei23]

こうかしら、ややこしい

EmojiとしてDBに追加できるもの (ローカル起因): _まで
EmojiとしてDBに追加できるもの (リモート起因): 制限なし
※文字数は一切検証されてないけどDBに入るのは128文字まで

Emojiをリモート→ローカルコピーした際: リモートの名前のままなので制限なし

投稿時にNoteからNote.emojis: string[]を作成する際: _, -, + まで by mfm.js、100文字まで

APIでNote取得時にpackする際: _まで ←このプルリクで変えてるのはここ
APでリモート向けにrenderする際: 制限なし _, -, + が来うる、id(uri)のエスケープがあやしい

(追記)
リアクション時は_,-,+で、:name@host:が260までReactionテーブルに入る

[anatawa12]

APIでNote取得時にpackする際: _まで ←このプルリクで変えてるのはここ

はい。あっています。　
mfm.js的に+もできるのであればこのプルリクで+も許可しちゃったほうがいいのかな

[mei23]

mfm.js的に+もできるのであればこのプルリクで+も許可しちゃったほうがいいのかな

影響範囲が、絵文字名をURLに入れ込んでいるところを特定して修正する必要がある
になるからあまりここでは手を出さないほうがいいかもだわ。

[syuilo]

🙏🏻