Skip to content

feat(http): add cookie feature for client#512

Merged
yukiiiteru merged 36 commits intomainfrom
feat(http)/cookie_jar
Nov 4, 2024
Merged

feat(http): add cookie feature for client#512
yukiiiteru merged 36 commits intomainfrom
feat(http)/cookie_jar

Conversation

@StellarisW
Copy link
Copy Markdown
Member

Motivation

add cookie feature for client

Solution

@StellarisW StellarisW requested review from a team as code owners October 24, 2024 08:22
@yukiiiteru
Copy link
Copy Markdown
Contributor

可以先去了解一下 Cookies 的作用和使用方式

Comment thread volo-http/src/client/cookie.rs Outdated
@@ -0,0 +1,88 @@
use std::sync::RwLock;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

用异步锁

Comment thread volo-http/src/client/mod.rs Outdated
Comment thread volo-http/src/client/mod.rs Outdated
Comment thread volo-http/src/utils/cookie/store.rs Outdated
Comment thread volo-http/src/utils/cookie/store.rs Outdated
Comment thread volo-http/src/utils/cookie/store.rs Outdated
@yukiiiteru
Copy link
Copy Markdown
Contributor

提交前先用 scripts/selftest.sh 跑一下吧

Comment thread volo-http/src/request.rs Outdated
Comment thread volo-http/src/response.rs Outdated
Comment thread volo-http/src/client/mod.rs Outdated
Comment thread volo-http/src/client/cookie.rs
Comment thread volo-http/src/client/cookie.rs Outdated
Comment thread volo-http/src/response.rs Outdated
Comment thread volo-http/src/client/mod.rs Outdated
Comment thread volo-http/src/client/cookie.rs Outdated
Comment thread volo-http/src/utils/cookie/store.rs Outdated
yukiiiteru
yukiiiteru previously approved these changes Nov 1, 2024
@yukiiiteru yukiiiteru merged commit e282982 into main Nov 4, 2024
@yukiiiteru yukiiiteru deleted the feat(http)/cookie_jar branch November 4, 2024 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants