Bind query params only for HTTP GET/DELETE methods

[aldas]

fixes #1670 and continues #1681 (comment) proposed changes

DefaultBinder.Bind() binds now query params to struct only for GET and DELETE methods. For POST/PUT/PATCH query params are not considered anymore. This restores pre v4.1.11 behavior.

[codecov]

Codecov Report

Merging #1727 (65ea019) into master (936c48a) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1727      +/-   ##
==========================================
+ Coverage   85.77%   85.87%   +0.10%     
==========================================
  Files          29       29              
  Lines        2017     2018       +1     
==========================================
+ Hits         1730     1733       +3     
+ Misses        181      180       -1     
+ Partials      106      105       -1     

Impacted Files	Coverage Δ
bind.go	91.30% <100.00%> (+1.30%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 936c48a...65ea019. Read the comment docs.

[aldas]

@lammel have you checked https://github.blog/2020-05-06-new-from-satellite-2020-github-codespaces-github-discussions-securing-code-in-private-repositories-and-more/#discussions maybe it is worth checking. Echo official forum is full of spam and I thinks similar thing in github would be much easier to find and use

[lammel]

@lammel have you checked https://github.blog/2020-05-06-new-from-satellite-2020-github-codespaces-github-discussions-securing-code-in-private-repositories-and-more/#discussions maybe it is worth checking. Echo official forum is full of spam and I thinks similar thing in github would be much easier to find and use

Noted! This is being discussed already.

[lammel]

Nice work. The tests cover all scenarios I could think off, thanks @aldas

[arvenil]

But why? :( What's wrong with query params in POST? :( Is there a way to force echo to parse query param for POST? Or I need to stay on older version?