Skip to content

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

Merged
4 commits merged into
xpressengine:feature/autoescapefrom
kijin:feature/autoescape
Sep 17, 2018
Merged

템플릿에 auto-escape (htmlspecialchars 자동 적용) 기능 추가#2300
4 commits merged into
xpressengine:feature/autoescapefrom
kijin:feature/autoescape

Conversation

@ghost

@ghost ghost commented Sep 17, 2018

Copy link
Copy Markdown

#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

kijin commented Sep 17, 2018

Copy link
Copy Markdown
Contributor

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

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

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

ghost commented Sep 18, 2018

Copy link
Copy Markdown
Author

@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