Re: [PATCH gwproxy v5 2/2] dnsparser: Add dns parser code

[Ahmad Gani <reyuki@gnuweeb.org>]

On Sat, Aug 23, 2025 at 2:52 AM Ammar Faizi <ammarfaizi2@gnuweeb.org> wrote:
> On Sat, Aug 23, 2025 at 02:09:34AM +0700, Alviro Iskandar Setiawan wrote:
> > (( The Jackson Hole's speech was very dovish, congrats if you went long ;-) ))
>
> Lol. No need to bring the portofolio discussion to this thread. Ahmad
> won't care about it anyway.

Yeah, I might be interested if I were rich and have a truckload of money :-)

> > My suggestion would be to create a new option in the configure file to
> > select what resolver type to be used. Let's mark this new feature as
> > experimental and keep the old feature for now. If your feature has
> > become stable enough, we can move forward abandoning getaddrinfo_a()
> > entirely.
>
> Agree with your idea. The same with io_uring feature in gwproxy. I
> assume the io_uring feature is still experimental and it is not enabled
> by default. We will probably never delete epoll code, forever. But the
> old DNS code may be deleted later if this one goes well.

I think it should be fine to have old DNS code removed if the raw DNS
resolver feature is only applied when it's completed and tested.

You can still keep the old DNS feature in the master branch that is meant
to be stable (?)

--
Ahmad Gani