Skip to content

refactor(services/http): Rewrite http methods signature by using OpRead/OpStat#3083

Merged
Xuanwo merged 4 commits intoapache:mainfrom
miroim:refactor/rewrite-http-op
Sep 15, 2023
Merged

refactor(services/http): Rewrite http methods signature by using OpRead/OpStat#3083
Xuanwo merged 4 commits intoapache:mainfrom
miroim:refactor/rewrite-http-op

Conversation

@miroim
Copy link
Copy Markdown
Member

@miroim miroim commented Sep 15, 2023

issue: #3064

Refactor http_get and http_head in services/http/backend.rs

@miroim miroim requested a review from Xuanwo as a code owner September 15, 2023 05:01
@github-actions github-actions Bot added the releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" label Sep 15, 2023
Xuanwo
Xuanwo approved these changes Sep 15, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo Xuanwo merged commit 6c3318f into apache:main Sep 15, 2023
@miroim miroim deleted the refactor/rewrite-http-op branch September 15, 2023 06:04
@Xuanwo Xuanwo mentioned this pull request Sep 18, 2023
Closed
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Xuanwo Xuanwo Xuanwo approved these changes

Assignees

No one assigned

Labels

releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor"

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@miroim @Xuanwo