Skip to content
This repository was archived by the owner on May 8, 2026. It is now read-only.
This repository was archived by the owner on May 8, 2026. It is now read-only.

QuicChannelBootstrap requires handler != null or streamHandler != null, seems unecessary. #770

Description

@hirshi001

Like the title says, in QuicChannelBootstrap::connect there is the following check

public Future<QuicChannel> connect(Promise<QuicChannel> promise) {
        if (handler == null && streamHandler == null) {
            throw new IllegalStateException("handler and streamHandler not set");
        }

This check doesn't seem to make sense, why is it only an illegal state if both handler and streamHandler are null? What about if only one was null?

Also, why would it matter if either of them is null, as a handler can always be added to the pipeline after creation?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions