Skip to content

Conversation

@Mygod
Copy link
Owner

@Mygod Mygod commented Jan 14, 2026

Fixes #1. Supersedes #3.

@Mygod Mygod added the enhancement New feature or request label Jan 14, 2026
@FDeghy
Copy link

FDeghy commented Jan 14, 2026

Queries reach the server, but the server does not respond to the queries and Server-failure is received on the client side after timeout at dns server.

@patterniha
Copy link

patterniha commented Jan 14, 2026

#1 (comment)

so we don't need this option at all :)

@Mygod
Copy link
Owner Author

Mygod commented Jan 14, 2026

Good. Regarding #5 (comment), it appears that picoquic requires a minimum packet length for initial packet so that's why it didn't work: (PICOQUIC_ENFORCED_INITIAL_MTU = 129)

    /* Create a connection context if the CI is acceptable */
    if (packet_length < PICOQUIC_ENFORCED_INITIAL_MTU) {
        /* Unexpected packet. Reject, drop and log. */
        ret = PICOQUIC_ERROR_INITIAL_TOO_SHORT;
    }

@Mygod Mygod closed this Jan 14, 2026
@Mygod Mygod reopened this Jan 18, 2026
@Mygod
Copy link
Owner Author

Mygod commented Jan 19, 2026

Unfortunately, the 101-byte limit leaves only a 56-byte MTU, rendering running QUIC (especially for the handshake) quite difficult. This will be a difficult change to implement on this codebase.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

customize max sub domain length

4 participants