Skip to content

TRANSIP: Audit records#2008

Merged
tlimoncelli merged 2 commits intoitstoragesvc:masterfrom
cafferata:transip-audit-records
Jan 29, 2023
Merged

TRANSIP: Audit records#2008
tlimoncelli merged 2 commits intoitstoragesvc:masterfrom
cafferata:transip-audit-records

Conversation

@cafferata
Copy link
Copy Markdown
Contributor

Checked the Audit records through the TransIP API.

POST /v6/domains/cafferata.dev/dns

rejectif.MxNull

Request

{
  "dnsEntry": {
    "name": "@",
    "expire": 86400,
    "type": "MX",
    "content": ""
  }
}

Response

{
  "error": "the content of an MX record must be in the format \"priority target\": @ 86400 MX \nrecord content cannot be empty: @ 86400 MX "
}

rejectif.TxtHasBackticks

{
  "dnsEntry": {
    "name": "@",
    "expire": 86400,
    "type": "TXT",
    "content": "`"
  }
}

Response

{
  "error": "the content of a TXT record cannot contain invalid characters: @ 86400 TXT `"
}

rejectif.TxtHasTrailingSpace

{
  "dnsEntry": {
    "name": "@",
    "expire": 86400,
    "type": "TXT",
    "content": "ms=123456 "
  }
}

Response

{
    "error": "whitespace at the end or start of the content of a record is not allowed: @ 86400 TXT ms=123456 "
}

rejectif.TxtIsEmpty

{
  "dnsEntry": {
    "name": "@",
    "expire": 86400,
    "type": "TXT",
    "content": ""
  }
}

Response

{
  "error": "record content cannot be empty: @ 86400 TXT "
}

@tlimoncelli
Copy link
Copy Markdown
Contributor

rejectif.MxNull

Despite the name "MxNull", the target should be .. This is similar to the SRV record which uses a target of . to mean "No service".

Thus, in your API call:

    "content": "."

See if that works.

@cafferata
Copy link
Copy Markdown
Contributor Author

POST /v6/domains/cafferata.dev/dns

rejectif.MxNull

{
    "dnsEntry": {
        "name": "@",
        "expire": 86400,
        "type": "MX",
        "content": "."
    }
}

Response

{
    "error": "the content of an MX record must be in the format \"priority target\": @ 86400 MX ."
}

@cafferata cafferata force-pushed the transip-audit-records branch from f169f47 to 422b613 Compare January 28, 2023 17:33
@tlimoncelli tlimoncelli merged commit fd7e642 into itstoragesvc:master Jan 29, 2023
@cafferata cafferata deleted the transip-audit-records branch January 29, 2023 18:05
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.

2 participants