Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

StubResolver::lookup_srv: TTL is not exposed #461

Open
WhyNotHugo opened this issue Dec 3, 2024 · 0 comments
Open

StubResolver::lookup_srv: TTL is not exposed #461

WhyNotHugo opened this issue Dec 3, 2024 · 0 comments

Comments

@WhyNotHugo
Copy link
Contributor

I'm using StubResolver::lookup_srv to resolve the location of CalDav/CardDav servers via rfc6764.

My implementation works fine so far, but the lookup_srv does not return the TTL, so my application cannot know how long it should cache these results.

Currently, lookup_srv returns Result<Option<FoundSrvs>, SrvError>.

I believe that the FoundSrvs could have an additional property describing the TTL of the result. This change would be backwards compatible, since it would merely add a new field to a returned field.

I'm not sure what the best format is to return the TTL. Returning the raw integer means that the caller needs to immediately convert it into a timestamp, but might be a sensible choice to keep domain's implementation and API the simplest.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant