dns: removing exceptions from resolver

[alyssawilk]

Risk Level: low Testing: updated tests Docs Changes: n/a Release Notes: n/a https://github.com/envoyproxy/envoy-mobile/issues/176

[repokitteh-read-only[bot]]

As a reminder, PRs marked as draft will not be automatically assigned reviewers, or be handled by maintainer-oncall triage.

Please mark your PR as ready when you want it to be reviewed!

:cat:

Caused by: https://github.com/envoyproxy/envoy/pull/34409 was opened by alyssawilk.

see: more, trace.

[jmarantz]

Yanjun -- can you do a first pass? After your lgtm I'll assign a maintainer.

[yanjunxiang-google]

LGTM modulo CI error.

[jmarantz]

@ravenblackx for non-google review (just realized I don't need a senior maintainer since Alyssa is).

[alyssawilk]

yeah I prefer THROW_OR_SET_VALUE to foo_or_error = function THROW_IF_STATUS_NOT_OK(foo_or_error.status() foo = foo_or_error.value() so I tend to use it when there's setters involved.