handlers icon indicating copy to clipboard operation
handlers copied to clipboard

Upgrade httpsnoop v1.0.1 => v1.0.3

Open malt3 opened this issue 2 years ago • 6 comments

Fixes #234

Summary of Changes

  1. Upgrade httpsnoop

malt3 avatar Aug 17 '22 15:08 malt3

Can you please fix the CI pipeline failure issue? I will also take a look if updating the httpsnoop version is a breaking change or not.

Thank you

amustaque97 avatar Aug 17 '22 19:08 amustaque97

Can you please fix the CI pipeline failure issue?

Here you go https://github.com/gorilla/handlers/pull/236

malt3 avatar Aug 18 '22 07:08 malt3

Hi @malt3, oops I should write it briefly. You need to fix the pipeline in the current PR. Here is the required diff to understand file changes better

diff --git a/handlers_go18_test.go b/handlers_go18_test.go
index d8e6321..ec227cc 100644
--- a/handlers_go18_test.go
+++ b/handlers_go18_test.go
@@ -1,3 +1,4 @@
+//go:build go1.8
 // +build go1.8
 
 package handlers

If you find it difficult to understand. Save the above diff output in a file let's say patch.txt and apply the patch git apply patch.txt. Add a commit message and push the changes. You can close other PR #236

Indeed this is not a breaking change, we can request the actual maintainer to merge this PR.

amustaque97 avatar Aug 21 '22 11:08 amustaque97

It is very weird for me to resolve existing problems in this unrelated PR. I also question wether you are an actual maintainer of this project. Maintainers can edit the PR so feel free to apply the exact patch you proposed to me yourself.

malt3 avatar Aug 21 '22 11:08 malt3

It is very weird for me to resolve existing problems in this unrelated PR.

I agree but keeping in mind that the original maintainer is quite busy and it's better to accommodate that minor change in the same PR.

I also question whether you are an actual maintainer of this project. Maintainers can edit the PR so feel free to apply the exact patch you proposed to me yourself.

No, I'm not the actual maintainer that's why I'm requesting you. 🙂

amustaque97 avatar Aug 21 '22 11:08 amustaque97

I see. :smile:

Sorry for making this complicated initially.

malt3 avatar Aug 21 '22 12:08 malt3

Completed via https://github.com/gorilla/handlers/pull/241

coreydaley avatar Aug 17 '23 01:08 coreydaley