Page MenuHomePhabricator

HTTP 500 from Horizon when creating long DNS records
Open, Needs TriagePublic

Description

Initially discovered in T87338
To reproduce the problem, I tried to make a TXT record called designatetesting.testlabs.wmflabs.org., comment Alex testing https://phabricator.wikimedia.org/T87338#4274206, value abcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyzabcdefghijklmnopqrstuvwxyz
Horizon through an HTTP 500 straight back at me and the UI showed "Error: Unable to create the record set.".
This was fine as soon as I reduced the value to 256 characters. More than that and it errored.

Event Timeline

Krenair created this task.Aug 28 2018, 10:45 PM
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 28 2018, 10:45 PM
Krenair updated the task description. (Show Details)

Note: This might be a problem with our Designate SQL schema, with the Horizon upstream code, or with the Designate upstream code.

This prevents you from adding DKIM records with 2048 bit RSA keys.

Before reporting upstream we should at least dig the error out of the logs actually.

Krenair edited projects, added Cloud-VPS; removed Horizon.Sep 7 2018, 3:08 PM
Krenair edited projects, added Horizon; removed Cloud-VPS.
Paladox added a subscriber: Paladox.Sep 7 2018, 3:17 PM

Brooke sent me the schema for the Designate records table, which includes this:

| Field                   | Type                                       | Null | Key | Default | Extra |
| data                    | text                                       | NO   |     | NULL    |       |

But MySQL TEXT is supposed to allow for 64KB so that shouldn't be the problem.

bd808 moved this task from Backlog to Upstream on the Horizon board.Nov 10 2019, 11:13 PM