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

TXT: join text records #127

Merged
merged 1 commit into from Apr 14, 2023
Merged

TXT: join text records #127

merged 1 commit into from Apr 14, 2023

Conversation

b4ldr
Copy link
Member

@b4ldr b4ldr commented Feb 17, 2023

TXT records that are larger then 255 characters are split into chunks. This means that the result of a TXT record is always Array[Array[String]].

This 255 character limit is more a detail about DNS which is not very useful when presenting data to higher interfaces. as such join the inner elements so we have a more normalise Array[String] return_type

This is a backwards incompatible breaking change

Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

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

This makes sense

TXT records that are larger then 255 characters are split into chunks.
This means that the result of a TXT record is always
Array[Array[String]].

This 355 character limit is more a detail about DNS which is not very
useful when presenting data to higher interfaces. as such join the inner
elements so we have a more normalise Array[String] return_type

This is a backwards incompatible breaking change
@b4ldr b4ldr merged commit 5f7e473 into master Apr 14, 2023
4 checks passed
@b4ldr b4ldr deleted the join_txt_records branch April 14, 2023 09:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants