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

エスケープせずにDescriptionを出力、Update info-card.pug #11108

Merged
merged 1 commit into from
Jul 6, 2023
Merged

エスケープせずにDescriptionを出力、Update info-card.pug #11108

merged 1 commit into from
Jul 6, 2023

Conversation

EdamAme-x
Copy link
Contributor

@EdamAme-x EdamAme-x commented Jul 4, 2023

What

HTMLのタグがエスケープされず、HTMLelementとして表示されるように。

Why

HTMLのタグがエスケープされ、
misskey-hub.netのサーバー一覧で、iframeで読み込む際にタグがそのまま出力される状況が発生していた。
eg

Misskey.io は、Misskey HQが運営する地球で生まれた分散マイクロブログSNSです。Fediverse(様々なSNSで構成される宇宙)の中に存在するため、他のSNSと相互に繋がっています。 <br> 暫し都会の喧騒から離れて、新しいインターネットにダイブしてみませんか。<br> <br> <strong>現在、新規登録時に利用できないユーザー名で登録しようとするとメール認証をした後に失敗する不具合が確認されています。お手数ですが、「始める」を押す前にユーザー名の下に「利用できます」と表示されていることをご確認ください。</strong><br> <br> <a href="https://go.misskey.io/support" target="_blank">お問い合わせはこちら<br>https://go.misskey.io/support</a><br> <br> <br> &copy; 2023 MisskeyHQ<br> Powered by Misskey

pugにおける仕様に則り、!=に変更、エスケープを行わないように。

Additional info (optional)

部分的な修正ですので、issueにおけるreviewはスキップさせて頂きました。

Checklist

HTMLのタグがエスケープされ、
misskey-hub.netのサーバー一覧で、iframeで読み込む際にタグがそのまま出力される状況が発生していた。
pugにおける仕様に則り、!=に変更、エスケープを行わないように。
@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Jul 4, 2023
@codecov
Copy link

codecov bot commented Jul 4, 2023

Codecov Report

Merging #11108 (f91e72f) into develop (61e7eb8) will decrease coverage by 0.28%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop   #11108      +/-   ##
===========================================
- Coverage    77.48%   77.21%   -0.28%     
===========================================
  Files          909      737     -172     
  Lines        91645    70142   -21503     
  Branches      7572     7052     -520     
===========================================
- Hits         71014    54158   -16856     
+ Misses       20631    15984    -4647     

see 189 files with indirect coverage changes

@EdamAme-x
Copy link
Contributor Author

@syuilo Please confirm..

@syuilo syuilo merged commit d2f8ed9 into misskey-dev:develop Jul 6, 2023
17 checks passed
@syuilo
Copy link
Member

syuilo commented Jul 6, 2023

🙏

slofp pushed a commit to Secineralyr/misskey.dream that referenced this pull request Jul 21, 2023
HTMLのタグがエスケープされ、
misskey-hub.netのサーバー一覧で、iframeで読み込む際にタグがそのまま出力される状況が発生していた。
pugにおける仕様に則り、!=に変更、エスケープを行わないように。
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants