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

9480-Renamed samesite to sameSite in t.w.h.Request.addCookie. #1036

Conversation

evilham
Copy link
Contributor

@evilham evilham commented Jul 4, 2018

Also improved its API documentation.

Fixes: https://tm.tl/9479 https://tm.tl/9480

Contributor Checklist:

Copy link
Member

@glyph glyph left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I don't have enough time to wait for all the CI machinery (mainly the Appveyor rebuild) so I'm approving contingent upon a passing CI run.

@markrwilliams markrwilliams merged commit 27d19e0 into twisted:trunk Jul 5, 2018
hawkowl pushed a commit that referenced this pull request Jul 5, 2018
…kie-sameSite

Author: evilham

Reviewers: glyph

Fixes: ticket:9480

Renamed samesite to sameSite in t.w.h.Request.addCookie.
danmaas added a commit to spinpunch/twisted that referenced this pull request Jul 13, 2018
* trunk: (85 commits)
  dev0
  Remove extra param from docstring.
  Remove the offending test entirely, per review
  Work around txchecker bug
  Skip CModuleSendmsgTests.test_shortsend
  update readme
  full changelog
  full ver
  Revert "18.7.0rc1 newsfile"
  Revert "changelog"
  changelog
  don't delete this dir
  incremental
  Merge pull request twisted#1036 from evilham/9480-evilham-web-Request-addCookie-sameSite
  Newsfile
  Expand HTTPFactory docstring
  Added newsfragments files for twisted#9479 and twisted#9480.
  Renamed samesite to sameSite in t.w.h.Request.addCookie.
  move 2008-2014 changelogs into historic
  18.7.0rc1 newsfile
  ...
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

3 participants