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

fix setcookie() #8

Merged
merged 1 commit into from Nov 27, 2016
Merged

Conversation

h2809
Copy link

@h2809 h2809 commented Sep 27, 2016

PHP setcookie()のセキュリティ向上 #7

@thingsym
Copy link
Owner

プルリクありがとうございます。

cookie は、テーマの切り替えの状態をフロントエンドで使うことを想定しているので、
第七引数 httponly は、「false」でお願いします。

@h2809
Copy link
Author

h2809 commented Nov 1, 2016

ご回答有難うございました。
第七引数の件、承知しました。対応いたします。

@h2809
Copy link
Author

h2809 commented Nov 20, 2016

第七引数の件、修正いたしました。
ご確認よろしくお願いします。

@thingsym thingsym merged commit 8f203dc into thingsym:master Nov 27, 2016
@thingsym
Copy link
Owner

対応ありがとうございます。
後日公式にアップデートしますね。

@h2809
Copy link
Author

h2809 commented Nov 29, 2016

マージして頂き、どうもありがとうございました。

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.

None yet

2 participants