Skip to content
This repository was archived by the owner on Jul 11, 2023. It is now read-only.

fix(client) resolve FQDNs ending in dots #120

Closed
wants to merge 1 commit into from

Conversation

locao
Copy link
Contributor

@locao locao commented Feb 23, 2021

No description provided.

@locao locao force-pushed the fix/fqdn_ending_in_dots branch from d960b14 to 5c922ad Compare February 23, 2021 12:43
@Tieske
Copy link
Member

Tieske commented Feb 23, 2021

what are the semantics of a name ending with a dot? are they to be treated as fqdn names? so no domain searches etc?

@locao
Copy link
Contributor Author

locao commented Feb 24, 2021

Yes, the dot at the end means the name is unambiguous, there's nothing else to search.

Copy link
Member

@Tieske Tieske left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't look like it is complete. If the trailing . denotes a FQDN, then the iterator (iterating over all the search domains) should be updated/bypassed in that case.

@locao locao force-pushed the fix/fqdn_ending_in_dots branch from 5c922ad to 2c6561c Compare March 8, 2021 16:07
@locao locao force-pushed the fix/fqdn_ending_in_dots branch from 2c6561c to 2c097f5 Compare March 8, 2021 16:11
@locao locao requested a review from Tieske March 8, 2021 16:17
@Tieske
Copy link
Member

Tieske commented Mar 9, 2021

I created an alternative in #122

@locao
Copy link
Contributor Author

locao commented Mar 10, 2021

Superseded by #122

@locao locao closed this Mar 10, 2021
@Tieske Tieske deleted the fix/fqdn_ending_in_dots branch March 10, 2021 16:48
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants