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

If domains last_check in SQL is not number a vague error is given #4450

Closed
cmouse opened this issue Sep 10, 2016 · 3 comments · Fixed by #7854
Closed

If domains last_check in SQL is not number a vague error is given #4450

cmouse opened this issue Sep 10, 2016 · 3 comments · Fixed by #7854
Labels

Comments

@cmouse
Copy link
Contributor

cmouse commented Sep 10, 2016

If last_check value for a domain contains non-numeric value other than NULL, an error is given.

Example value

INSERT INTO "domains" VALUES(1,'vpn.gwad.fi','NULL','NULL','NATIVE',NULL,NULL);

Will produce errors when doing DNSUPDATE, such as

Sep 10 11:22:23 gw-fw01 pdns_server: Sep 10 11:22:23 Exception building answer packet for vpn.gwad.fi/SOA (stoul) sending out servfail

or

pdnsutil rectify-all-zones
Error: stoul
@Habbie
Copy link
Member

Habbie commented Feb 6, 2017

A user is reporting stoul errors with bindbackend as well, which makes sense - there is no reason for this issue to be limited to gSQL.

@Habbie
Copy link
Member

Habbie commented Mar 26, 2018

A user is reporting stoul errors with bindbackend as well, which makes sense - there is no reason for this issue to be limited to gSQL.

I should have written more words here - I no longer recall why I said this makes sense for bindbackend to have similar issues.

@Habbie
Copy link
Member

Habbie commented May 14, 2019

Current master on rectify-all-zones: Error: stoul; (invalid argument during std::stoul); data was "NULL"

@Habbie Habbie added this to the auth-4.2.0-rc2 milestone May 14, 2019
Habbie added a commit to Habbie/pdns that referenced this issue May 27, 2019
Habbie added a commit to Habbie/pdns that referenced this issue May 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants