Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@chuff
Copy link
Contributor

@chuff chuff commented Aug 5, 2025

No description provided.

@yuzawa-san
Copy link
Contributor

@chuff can you please describe how this broke and why this works in the pr description

@chuff
Copy link
Contributor Author

chuff commented Oct 30, 2025

Yeah, basically when DE, IA, NE, NH, NJ, TN was released, USNAT was updated to accommodate 4 additional values for sensitive data processing and 1 additional value for known child sensitive data consents. Since those 2 fields are not at the end of the string, to maintain backwards compatibility, a check needs to be done to see if the string to be parsed was created before or after this change. Since string length is a consistent identifier, I chose that option and injected default values into the string at the correct locations for the newer values.

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.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.