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

SC_CheckError::HTML_TAG_CHECK() 数字を含むタグを正しく扱えない #129

Closed
seasoftjapan opened this issue Jul 20, 2017 · 7 comments
Milestone

Comments

@seasoftjapan
Copy link
Contributor

代替案

-        preg_match_all('/<\/?([a-z]+)/i', $this->arrParam[$keyname], $match);
+        preg_match_all('/<\/?(\w+)/i', $this->arrParam[$keyname], $match);

もっと良い正規表現が、どこかに転がっていそうな予感もする。

@ryo-endo ryo-endo added this to the 2.17.0 milestone Oct 26, 2017
@nobuhiko
Copy link
Contributor

数字を含むタグってなんでしょう?

@so-amuamu
Copy link

h1~h6 ですかね?それ以外には無いと思う

@nobuhiko
Copy link
Contributor

あー
これをもし直すとmtb_allowed_tagも直さないとダメになりますね

@so-amuamu so-amuamu removed the fix-me label Aug 16, 2018
@so-amuamu
Copy link

でもh1~h6は通ると思うんですよね
どちらにしろ、mtb_allowed_tagの初期値は見直さないといかんですね
そして、それらが通るか確認すべきかな

@nobuhiko
Copy link
Contributor

現状は、 h1 -> h として通っていてmtb_allowed_tagにも h が登録されているので、ある意味意図した挙動なのかなと思います

@seasoftjapan
Copy link
Contributor Author

どちらにしろ、mtb_allowed_tagの初期値は見直さないといかんですね

#224 に展開しました。

seasoftjapan added a commit to seasoftjapan/eccube-2_13 that referenced this issue Aug 18, 2018
正規表現を考えた際の参考 http://w3c.github.io/html-reference/syntax.html#syntax-elements (本当は、W3C より、ブラウザーの実装を参考にすべきかも。)
ryo-endo added a commit that referenced this issue Aug 30, 2018
[fix] refs #129 SC_CheckError::HTML_TAG_CHECK() 数字を含むタグを正しく扱えない
@nanasess
Copy link
Contributor

#225 で修正済み

ji-eunsoo pushed a commit that referenced this issue Feb 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants