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

⬆️ cookies@0.9.1 #1846

Merged
merged 1 commit into from
Oct 22, 2024
Merged

Conversation

MichalMartinek
Copy link
Contributor

@MichalMartinek MichalMartinek commented Sep 26, 2024

Checklist

  • I have ensured my pull request is not behind the main or master branch of the original repository.
  • I have rebased all commits where necessary so that reviewing this pull request can be done without having to merge it first.
  • I have written a commit message that passes commitlint linting.
  • I have ensured that my code changes pass linting tests.
  • I have ensured that my code changes pass unit tests.
  • I have described my pull request and the reasons for code changes along with context if necessary.

Maintainer Notes

Closes #1561

@kevinpeno
Copy link
Contributor

Hello, is there a specific reason you needed the dependency update? I'm only asking because it appears only one dependency was updated and, if it affects framework operation, it might be important for documentation/semver purposes.

@kevinpeno kevinpeno self-assigned this Oct 20, 2024
@jonathanong jonathanong changed the title Update dependencies ⬆️ cookies@0.9.1 Oct 20, 2024
@jonathanong
Copy link
Member

this would fix this issue: #1561

@kevinpeno kevinpeno force-pushed the feature/update-deps branch from aa27e8b to d40694b Compare October 22, 2024 18:38
@kevinpeno kevinpeno enabled auto-merge (squash) October 22, 2024 18:39
@kevinpeno kevinpeno self-requested a review October 22, 2024 18:39
@kevinpeno kevinpeno merged commit 988ebec into koajs:master Oct 22, 2024
4 checks passed
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.

Koa should strip quotes for returned cookie values
3 participants