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

addon_info->description을 포함하는 페이지가 escape로 인해 br 태그가 그대로 노출되는 이슈 #2303

Closed
Xvezda opened this issue Sep 19, 2018 · 3 comments
Labels
Milestone

Comments

@Xvezda
Copy link
Contributor

Xvezda commented Sep 19, 2018

$addon_info->description 을 포함하는 템플릿 코드가 이번 보안패치로 인해서
기존의 nl2br이 escape 되어, 깨져보이는 이슈가 있는것으로 보입니다.

해당 변수에 noescape 필터를 적용하거나 기존 개행을 무시하는 패치가 필요 할 것으로 생각됩니다.

@ghost ghost self-assigned this Sep 19, 2018
@ghost ghost added the type/bug label Sep 19, 2018
@ghost ghost added this to the 1.11.0 milestone Sep 19, 2018
@ghost
Copy link

ghost commented Sep 19, 2018

@Xvezda 감사합니다.
문제가 될 만한 페이지를 계속 찾고 있습니다.

nl2br에 대해서도 다시 확인해보도록 하겠습니다.

@Xvezda
Copy link
Contributor Author

Xvezda commented Sep 19, 2018

@bnu 다른 메뉴에서도 상세보기 등으로 열어볼때 비슷한 증상이 발생하는것으로 보아 코어에서
info.xml 을 파싱해서 출력해주는 부분을 전반적으로 검토해보는게 좋을 것 같습니다.

screen

@ghost
Copy link

ghost commented Sep 19, 2018

@Xvezda 네. 레이아웃, 위젯, 애드온 등 유사 목록을 표시하는 곳은 수정했습니다.(커밋 전)

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant