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

Fix task quantum unmarshall error from bind stats channel xml output #166

Merged
merged 2 commits into from
Mar 22, 2023
Merged

Fix task quantum unmarshall error from bind stats channel xml output #166

merged 2 commits into from
Mar 22, 2023

Conversation

bschoenbach
Copy link
Contributor

This change fixes broken xml parse (unmarshall of possible negative task quantum attribute) within bind v9.18.12+ on statistics channel.

@SuperQ
Copy link
Contributor

SuperQ commented Mar 22, 2023

It would be useful to update the test fixtures to add -1 for this field somewhere.

@bschoenbach
Copy link
Contributor Author

It would be useful to update the test fixtures to add -1 for this field somewhere.

done i put some negative values for v3 and v2

@SuperQ SuperQ merged commit 143595a into prometheus-community:master Mar 22, 2023
SuperQ added a commit that referenced this pull request Mar 22, 2023
* [BUGFIX] Fix nmarshall error for negative values #166

Signed-off-by: SuperQ <[email protected]>
@SuperQ SuperQ mentioned this pull request Mar 22, 2023
SuperQ added a commit that referenced this pull request Mar 22, 2023
* [BUGFIX] Fix unmarshall error for negative values #166

Signed-off-by: SuperQ <[email protected]>
SuperQ added a commit that referenced this pull request Mar 22, 2023
* [BUGFIX] Fix unmarshall error for negative values #166

Signed-off-by: SuperQ <[email protected]>
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