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 test not passing in 32-bit architectures (fixes #2032) #2033

Merged
merged 1 commit into from
Feb 11, 2023

Conversation

kraptor
Copy link
Contributor

@kraptor kraptor commented Feb 11, 2023

Some architectures cannot hold values after
2038 year.

This commit fixes the following tests, adjusting
the expiring date for the cookies to a maximum
year value of 2037:

  • test_set_cookie
  • test_expires_on_set_cookie

The starting point for contributions should usually be a discussion

Simple documentation typos may be raised as stand-alone pull requests, but otherwise please ensure you've discussed your proposal prior to issuing a pull request.

This will help us direct work appropriately, and ensure that any suggested changes have been okayed by the maintainers.

@kraptor
Copy link
Contributor Author

kraptor commented Feb 11, 2023

This should fix #2032

Some architectures cannot hold values after
2038 year.

This commit fixes the following tests, adjusting
the expiring date for the cookies to a maximum
year value of 2037:

* test_set_cookie
* test_expires_on_set_cookie
@Kludex
Copy link
Member

Kludex commented Feb 11, 2023

Thank you! We are going to make a release on the 14th - in case it matters.

@Kludex Kludex merged commit 5771a78 into encode:master Feb 11, 2023
aminalaee pushed a commit that referenced this pull request Feb 13, 2023
Some architectures cannot hold values after
2038 year.

This commit fixes the following tests, adjusting
the expiring date for the cookies to a maximum
year value of 2037:

* test_set_cookie
* test_expires_on_set_cookie
@aminalaee
Copy link
Member

why am I mentioned here? 😄

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.

3 participants