search

aarond10 / https_dns_proxy

A lightweight DNS-over-HTTPS proxy.
MIT License
775 stars 114 forks source link

Fixes 7 #126

Closed baranyaib90 closed 2 years ago

baranyaib90 commented 2 years ago

Hi, please welcome my next pack of fixes:

  1. gcc-10 and clang-12 warning fixes with github actions
  2. Limiting response body size to avoid dealing with non-standard large invalid responses
  3. Better curl result checking, since the most important curl error code was not checked yet and also the usefull curl error message is printed as well to help users debug issues
  4. HTTP request errors now printed on warning level. (curl API errors are still on error level)

    125

  5. minor changes: condition removed when setting curlpipe option and shutdown on SIGTERM (I can't belive, that this was missing. I did not even noticed 😄)

As always: I have tested my changes in my own server.

I hope, you will like it. Sincerely, Balázs

Spoiler: I have 2 branch WIP in my repo: HTTP/3 support and robot functional testing.

aarond10 commented 2 years ago

Do you want to add yourself to the author list in README.md as well? You've submitted plenty of great improvements.

baranyaib90 commented 2 years ago

Do you want to add yourself to the author list in README.md as well? You've submitted plenty of great improvements.

Thank you for the offering! I will think about it when I will create my next pull request.