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

expires => undef means that expires is not specified. #4

Merged
merged 1 commit into from
Jun 29, 2015
Merged

expires => undef means that expires is not specified. #4

merged 1 commit into from
Jun 29, 2015

Conversation

shogo82148
Copy link
Contributor

c88febd breaks a test of Ark.
ref. ark-framework/ark#10

I think Cookie::Baker should accept expires => undef because of compatibility.
How do you think?

@kazeburo
Copy link
Owner

Thank you.

Original Plack::Response code is this.

push @cookie, "expires=" . $self->_date($val->{expires}) if $val->{expires};

Cookie::Baker broke compatibility. I merge this p-r.

kazeburo added a commit that referenced this pull request Jun 29, 2015
`expires => undef` means that expires is not specified.
@kazeburo kazeburo merged commit 026c5aa into kazeburo:master Jun 29, 2015
@shogo82148
Copy link
Contributor Author

Thank you for merging!

kazeburo added a commit that referenced this pull request Jun 29, 2015
Changelog diff is:

diff --git a/Changes b/Changes
index 54d9fe3..6c3eeb3 100644
--- a/Changes
+++ b/Changes
@@ -2,6 +2,10 @@ Revision history for Perl extension Cookie-Baker

 {{$NEXT}}

+0.06 2015-06-29T05:53:57Z
+
+   - fix compatibility issue. #4 (Thank you shogo82148)
+
 0.05 2014-11-26T05:49:50Z

    - fix bug. accepts expires => 0
@shogo82148 shogo82148 deleted the expires-accepts-undef branch June 29, 2015 05:58
kazeburo added a commit that referenced this pull request Sep 21, 2016
Changelog diff is:

diff --git a/Changes b/Changes
index 6c3eeb3..280b02a 100644
--- a/Changes
+++ b/Changes
@@ -2,6 +2,10 @@ Revision history for Perl extension Cookie-Baker

 {{$NEXT}}

+0.07 2016-09-21T01:35:56Z
+
+   - Allow quoted cookie values, as per RFC 6265. #8
+
 0.06 2015-06-29T05:53:57Z

    - fix compatibility issue. #4 (Thank you shogo82148)
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