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

URL書式チェックを追加 #4

Merged
merged 3 commits into from
Aug 5, 2012
Merged

URL書式チェックを追加 #4

merged 3 commits into from
Aug 5, 2012

Conversation

ackintosh
Copy link
Contributor

URLの書式チェック機能を追加しました!
ご確認お願いいたしますm(_ _)m

@dounokouno
Copy link
Owner

pull requestありがとうございます!
現在、少々立て込んでおりまして、確認までしばらくお待ちくださいませ。

それでは、取り急ぎお返事まで。

@ackintosh
Copy link
Contributor Author

了解です ^^
お忙しいところすいません!

@dounokouno
Copy link
Owner

いくつかのURLにて検証をしてみたところ、URLと判断される場合と判断されない場合が確認されました。

URLと判断される場合

URLと判断されない場合

「どのような文字列を正しいURLとするか」(日本語ドメインや日本語のディレクトリ名、ファイル名などの半角英数字以外の文字を含むURLをどのように扱うか等)といった議論もあるかと思いますが、最低限、上記は正しいURLとして判別したいなと思っておりますが、いかがでしょうか?

ご確認をお願いいたします。

ackintosh added 2 commits July 8, 2012 19:24
アンダースコアやクエリ文字列がエラーになってしまうのを修正
@ackintosh
Copy link
Contributor Author

ご指摘いただきありがとうございます。
さっそく修正いたしました。

dounokouno added a commit that referenced this pull request Aug 5, 2012
URL書式チェックを追加
@dounokouno dounokouno merged commit 3f8963c into dounokouno:master Aug 5, 2012
@dounokouno
Copy link
Owner

お返事が遅くなってしまい申し訳ありません…。
(やっと時間がとれました><)

確認をしましたところ、
特に問題ございませんでしたので、
マージいたしました。

引き続きWikiや動作サンプルページに、
追加いただいたURLチェック機能のことを記載していきたいと思います。

この度はPull requestをいただきありがとうございました。

また、大変恐縮でございますが、
1点お伺いしたいことがございます。

ackintosh様に追加いただいたコードを拝見しますと、
$_POST[$v] = mb_convert_kana($_POST[$v], 'a');
とした後に、
$_POST[$v] = delete_crlf($_POST[$v]);
とされていますが、
これはmb_convert_kana関数実行後に改行コードが入ってしまう場合があるためでしょうか?

TransmitMail内ではmb_convert_kana関数をいくつか使用しており、
他の箇所にも影響があるものかと思い、ご質問をさせていただきました。

以上でございます。
それでは、どうぞよろしくお願いいたします。

@ackintosh
Copy link
Contributor Author

マージしていただきありがとうございます。
(ご多用のところ本当にありがとうございます!)

$_POST[$v] = delete_crlf($_POST[$v]);

につきましては、単一のURLを入力値として想定しているため改行コードは不要と考えました。

自身の最善を尽くすよう努力しておりますが、恥ずかしながらまだまだ未熟なところがございますので
実装の不備等ございましたら是非ご指摘いただけますと幸いです!

@dounokouno
Copy link
Owner

ackintosh様

お返事ありがとうございます。

単一のURLを入力値として想定しているため改行コードは不要と考えました。

おっしゃる通り、URLのチェックに改行コードは不要ですね。
mb_convert_kana関数は特に関係無さそうで安心いたしました。

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

Successfully merging this pull request may close these issues.

2 participants