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

docs: add example for RelativeDname::from_slice #220

Merged
merged 1 commit into from
Aug 24, 2023

Conversation

WhyNotHugo
Copy link
Contributor

@WhyNotHugo WhyNotHugo commented Aug 23, 2023

Mostly to make it clear that b"_submissions.tcp" is not valid input here.

@partim
Copy link
Member

partim commented Aug 23, 2023

Thank you for the PR! Since this indeed seems to be a recurring mistake, perhaps we should add a sentence on all of these (also Dname<_>) that the slice needs to in wire format, i.e., no dots but each label prefixed with its length?

partim

This comment was marked as duplicate.

@WhyNotHugo WhyNotHugo force-pushed the examples branch 2 times, most recently from b1610b9 to fd10a26 Compare August 23, 2023 09:12
@WhyNotHugo
Copy link
Contributor Author

WhyNotHugo commented Aug 23, 2023

I've included an example for domain::base::name::Dname::from_slice. I'm not sure where else this same advice is applicable.

@WhyNotHugo WhyNotHugo force-pushed the examples branch 4 times, most recently from ac3c809 to 8e065ed Compare August 23, 2023 10:16
Mostly to make it clear the examples like the following are not valid:

    RelativeDname::from_slice(b"_submissions._tcp");
@partim
Copy link
Member

partim commented Aug 24, 2023

I think that’s about it?

@partim partim merged commit cb12b52 into NLnetLabs:main Aug 24, 2023
12 checks passed
@WhyNotHugo WhyNotHugo deleted the examples branch August 24, 2023 14:32
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

Successfully merging this pull request may close these issues.

2 participants