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

템플릿에 auto-escape (htmlspecialchars 자동 적용) 기능 추가 #2300

Merged
4 commits merged into from
Sep 17, 2018

Conversation

ghost
Copy link

@ghost ghost commented Sep 17, 2018

#1267 PR이 제공하는 auto-escape 기능 추가

@kijin 님의 코드 제공을 기반으로 하며, 추가로 rhymix/rhymix#727 개선 사항을 포함합니다.

@ghost ghost self-assigned this Sep 17, 2018
@ghost ghost changed the title Feature/autoescape 템플릿에 auto-escape (htmlspecialchars 자동 적용) 기능 추가 Sep 17, 2018
@ghost ghost merged commit 5ce08f1 into xpressengine:feature/autoescape Sep 17, 2018
ghost pushed a commit that referenced this pull request Sep 17, 2018
- Rhymix에서 가져 옴
- copyright [Rhymix](https://github.com/rhymix/rhymix) Developers and Contributors
ghost pushed a commit that referenced this pull request Sep 17, 2018
- Rhymix에 패치된 727 PR의 코드를 가져옴
- rhymix/rhymix#727
- 코드 제공 : @kijin
@kijin
Copy link
Contributor

kijin commented Sep 17, 2018

이 PR에 포함되었거나 이 PR을 레퍼런스하는 커밋 내용 중 예전에 제가 (라이믹스의 일부로) GPL 라이선스 하에 배포했던 것이 있습니다. XE와 라이믹스의 상호 호환성 유지 및 보안 강화에 도움이 되도록 하기 위해, 아래에 해당하는 코드는 LGPL 라이선스 하에 재배포하는 것을 허가합니다.^^

  • TemplateHandler 클래스의 변경내역으로서 autoescape 및 필터 문법 지원에 필요한 코드
  • 그 밖에 문자열을 escape 처리하는 데 도움이 되는 전역함수로서 라이믹스의 common/functions.php에 포함되어 있으며 라이믹스에서 가져왔다는 사실을 분명히 표시한 코드

ghost pushed a commit that referenced this pull request Sep 17, 2018
@ghost
Copy link
Author

ghost commented Sep 18, 2018

@kijin 감사합니다. 👍

@ghost ghost added this to the 1.11.0 milestone Oct 10, 2018
dorami added a commit to daolcms/daolcms that referenced this pull request Oct 12, 2018
dorami added a commit to daolcms/daolcms that referenced this pull request Oct 13, 2018
dorami added a commit to daolcms/daolcms that referenced this pull request Nov 7, 2018
dorami added a commit to daolcms/daolcms that referenced this pull request Nov 7, 2018
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant